From: Neal Cardwell Date: Wed, 16 Nov 2011 08:58:02 +0000 (+0000) Subject: tcp: use DSACKs that arrive when packets_out is 0 X-Git-Tag: upstream/snapshot3+hdmi~8450^2~466 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5628adf1a0ff39b9e76e1a8e1c94dadabfd46914;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git tcp: use DSACKs that arrive when packets_out is 0 The bug: Senders ignored DSACKs after recovery when there were no outstanding packets (a common scenario for HTTP servers). The change: when there are no outstanding packets (the "no_queue" goto label), call tcp_fastretrans_alert() in order to use DSACKs to undo congestion window reductions. Other patches in this series will provide other changes that are necessary to fully fix this problem. Signed-off-by: Neal Cardwell Acked-by: Ilpo Järvinen Signed-off-by: David S. Miller --- diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c index f772aaa..b49e418 100644 --- a/net/ipv4/tcp_input.c +++ b/net/ipv4/tcp_input.c @@ -3788,6 +3788,10 @@ static int tcp_ack(struct sock *sk, const struct sk_buff *skb, int flag) return 1; no_queue: + /* If data was DSACKed, see if we can undo a cwnd reduction. */ + if (flag & FLAG_DSACKING_ACK) + tcp_fastretrans_alert(sk, pkts_acked, newly_acked_sacked, + is_dupack, flag); /* If this ack opens up a zero window, clear backoff. It was * being used to time the probes, and is probably far higher than * it needs to be for normal retransmission.