diff options
| author | Eric Dumazet <[email protected]> | 2014-10-30 17:32:34 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2014-10-30 23:58:30 +0000 |
| commit | 39bb5e62867de82b269b07df900165029b928359 (patch) | |
| tree | f0cbefb0eee4eda2697c4e2fd6e0ad3ee7331f43 /net/ipv4/tcp_output.c | |
| parent | gre: Use inner mac length when computing tunnel length (diff) | |
| download | kernel-39bb5e62867de82b269b07df900165029b928359.tar.gz kernel-39bb5e62867de82b269b07df900165029b928359.zip | |
net: skb_fclone_busy() needs to detect orphaned skb
Some drivers are unable to perform TX completions in a bound time.
They instead call skb_orphan()
Problem is skb_fclone_busy() has to detect this case, otherwise
we block TCP retransmits and can freeze unlucky tcp sessions on
mostly idle hosts.
Signed-off-by: Eric Dumazet <[email protected]>
Fixes: 1f3279ae0c13 ("tcp: avoid retransmits of TCP packets hanging in host queues")
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'net/ipv4/tcp_output.c')
| -rw-r--r-- | net/ipv4/tcp_output.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index 3af21296d967..a3d453b94747 100644 --- a/net/ipv4/tcp_output.c +++ b/net/ipv4/tcp_output.c @@ -2126,7 +2126,7 @@ bool tcp_schedule_loss_probe(struct sock *sk) static bool skb_still_in_host_queue(const struct sock *sk, const struct sk_buff *skb) { - if (unlikely(skb_fclone_busy(skb))) { + if (unlikely(skb_fclone_busy(sk, skb))) { NET_INC_STATS_BH(sock_net(sk), LINUX_MIB_TCPSPURIOUS_RTX_HOSTQUEUES); return true; |