diff options
author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2008-09-21 00:25:15 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-09-21 00:25:15 -0400 |
commit | ef9da47c7cc64d69526331f315e76b5680d4048f (patch) | |
tree | 97407b83c7c339dbe1fd0948c30f35e1680562ec /net/ipv4/tcp_input.c | |
parent | f0ceb0ed86b4792a4ed9d3438f5f7572e48f9803 (diff) |
tcp: don't clear retransmit_skb_hint when not necessary
Most importantly avoid doing it with cumulative ACK. Not clearing
means that we no longer need n^2 processing in resolution of each
fast recovery.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_input.c')
-rw-r--r-- | net/ipv4/tcp_input.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c index d017aed6edd7..44a4fffc2cc3 100644 --- a/net/ipv4/tcp_input.c +++ b/net/ipv4/tcp_input.c | |||
@@ -2925,7 +2925,9 @@ static int tcp_clean_rtx_queue(struct sock *sk, int prior_fackets) | |||
2925 | 2925 | ||
2926 | tcp_unlink_write_queue(skb, sk); | 2926 | tcp_unlink_write_queue(skb, sk); |
2927 | sk_wmem_free_skb(sk, skb); | 2927 | sk_wmem_free_skb(sk, skb); |
2928 | tcp_clear_all_retrans_hints(tp); | 2928 | tcp_clear_retrans_hints_partial(tp); |
2929 | if (skb == tp->retransmit_skb_hint) | ||
2930 | tp->retransmit_skb_hint = NULL; | ||
2929 | } | 2931 | } |
2930 | 2932 | ||
2931 | if (skb && (TCP_SKB_CB(skb)->sacked & TCPCB_SACKED_ACKED)) | 2933 | if (skb && (TCP_SKB_CB(skb)->sacked & TCPCB_SACKED_ACKED)) |