diff options
author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2007-12-01 17:47:58 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 17:55:41 -0500 |
commit | ede9f3b186bc3eb0fce084bdcab500efc3721a80 (patch) | |
tree | 78d421c79f18e3d56e9ab079459081d3d8dc42d8 | |
parent | 407ef1de03e87225d75a9bed271f35ea6880f5f1 (diff) |
[TCP]: Unite identical code from two seqno split blocks
Bogus seqno compares just mislead, the code is identical for
both sides of the seqno compare (and was even executed just
once because of return in between).
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/ipv4/tcp_input.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c index 87111084280..d313dea361a 100644 --- a/net/ipv4/tcp_input.c +++ b/net/ipv4/tcp_input.c | |||
@@ -1246,8 +1246,7 @@ static int tcp_sacktag_one(struct sk_buff *skb, struct tcp_sock *tp, | |||
1246 | if (dup_sack && (sacked & TCPCB_RETRANS)) { | 1246 | if (dup_sack && (sacked & TCPCB_RETRANS)) { |
1247 | if (after(TCP_SKB_CB(skb)->end_seq, tp->undo_marker)) | 1247 | if (after(TCP_SKB_CB(skb)->end_seq, tp->undo_marker)) |
1248 | tp->undo_retrans--; | 1248 | tp->undo_retrans--; |
1249 | if (!after(TCP_SKB_CB(skb)->end_seq, tp->snd_una) && | 1249 | if (sacked & TCPCB_SACKED_ACKED) |
1250 | (sacked & TCPCB_SACKED_ACKED)) | ||
1251 | *reord = min(fack_count, *reord); | 1250 | *reord = min(fack_count, *reord); |
1252 | } | 1251 | } |
1253 | 1252 | ||
@@ -1310,10 +1309,6 @@ static int tcp_sacktag_one(struct sk_buff *skb, struct tcp_sock *tp, | |||
1310 | 1309 | ||
1311 | if (after(TCP_SKB_CB(skb)->seq, tcp_highest_sack_seq(tp))) | 1310 | if (after(TCP_SKB_CB(skb)->seq, tcp_highest_sack_seq(tp))) |
1312 | tp->highest_sack = skb; | 1311 | tp->highest_sack = skb; |
1313 | |||
1314 | } else { | ||
1315 | if (dup_sack && (sacked & TCPCB_RETRANS)) | ||
1316 | *reord = min(fack_count, *reord); | ||
1317 | } | 1312 | } |
1318 | 1313 | ||
1319 | /* D-SACK. We can detect redundant retransmission in S|R and plain R | 1314 | /* D-SACK. We can detect redundant retransmission in S|R and plain R |