Skip to content

Commit

Permalink
tcp: more aggressive skipping
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Ilpo Järvinen authored and David S. Miller committed Nov 25, 2008
1 parent e1aa680 commit e8bae27
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion net/ipv4/tcp_input.c
Original file line number Diff line number Diff line change
Expand Up @@ -1402,7 +1402,7 @@ static struct sk_buff *tcp_sacktag_skip(struct sk_buff *skb, struct sock *sk,
if (skb == tcp_send_head(sk))
break;

if (!before(TCP_SKB_CB(skb)->end_seq, skip_to_seq))
if (after(TCP_SKB_CB(skb)->end_seq, skip_to_seq))
break;

*fack_count += tcp_skb_pcount(skb);
Expand Down

0 comments on commit e8bae27

Please sign in to comment.