diff options
| author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2008-09-21 06:24:49 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-09-21 06:24:49 +0200 |
| commit | f0ceb0ed86b4792a4ed9d3438f5f7572e48f9803 (patch) | |
| tree | 0bc0525bb8bb80fc6b8d93b48d5726a69b8e8727 /net | |
| parent | tcp: reorganize retransmit code loops (diff) | |
| download | linux-f0ceb0ed86b4792a4ed9d3438f5f7572e48f9803.tar.xz linux-f0ceb0ed86b4792a4ed9d3438f5f7572e48f9803.zip | |
tcp: remove retransmit_skb_hint clearing from failure
This doesn't much sense here afaict, probably never has. Since
fragmenting and collapsing deal the hints by themselves, there
should be very little reason for the rexmit loop to do that.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/ipv4/tcp_output.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index b5b4ddcdda41..f900fae8b87e 100644 --- a/net/ipv4/tcp_output.c +++ b/net/ipv4/tcp_output.c @@ -2096,10 +2096,8 @@ begin_fwd: if (sacked & (TCPCB_SACKED_ACKED|TCPCB_SACKED_RETRANS)) continue; - if (tcp_retransmit_skb(sk, skb)) { - tp->retransmit_skb_hint = NULL; + if (tcp_retransmit_skb(sk, skb)) return; - } NET_INC_STATS_BH(sock_net(sk), mib_idx); if (skb == tcp_write_queue_head(sk)) |