diff options
author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2008-11-25 00:12:28 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-11-25 00:12:28 -0500 |
commit | e8bae275d9354104f7ae24a48a90d1a6286e7bd9 (patch) | |
tree | 90f4bb2abd9eb31b3faa6393f1e164ac48b57238 | |
parent | e1aa680fa40e7492260a09cb57d94002245cc8fe (diff) |
tcp: more aggressive skipping
I knew already when rewriting the sacktag that this condition
was too conservative, change it now since it prevent lot of
useless work (especially in the sack shifter decision code
that is being added by a later patch). This shouldn't change
anything really, just save some processing regardless of the
shifter.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/ipv4/tcp_input.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c index 8085704863fb..3f26599ddc88 100644 --- a/net/ipv4/tcp_input.c +++ b/net/ipv4/tcp_input.c | |||
@@ -1402,7 +1402,7 @@ static struct sk_buff *tcp_sacktag_skip(struct sk_buff *skb, struct sock *sk, | |||
1402 | if (skb == tcp_send_head(sk)) | 1402 | if (skb == tcp_send_head(sk)) |
1403 | break; | 1403 | break; |
1404 | 1404 | ||
1405 | if (!before(TCP_SKB_CB(skb)->end_seq, skip_to_seq)) | 1405 | if (after(TCP_SKB_CB(skb)->end_seq, skip_to_seq)) |
1406 | break; | 1406 | break; |
1407 | 1407 | ||
1408 | *fack_count += tcp_skb_pcount(skb); | 1408 | *fack_count += tcp_skb_pcount(skb); |