diff options
author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2009-02-28 04:44:32 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-03-02 11:00:11 (GMT) |
commit | d0af4160d19ff2849386140881e729f9ba86f2aa (patch) | |
tree | d9582acc750f249839602316a129eddb59e21acb /net/ipv4/inet_lro.c | |
parent | 02276f3c962fd408fa9d441251067845f948bfcf (diff) | |
download | linux-d0af4160d19ff2849386140881e729f9ba86f2aa.tar.xz |
tcp: remove redundant code from tcp_mark_lost_retrans
Arnd Hannemann <hannemann@nets.rwth-aachen.de> noticed and was
puzzled by the fact that !tcp_is_fack(tp) leads to early return
near the beginning and the later on tcp_is_fack(tp) was still
used in an if condition. The later check was a left-over from
RFC3517 SACK stuff (== !tcp_is_fack(tp) behavior nowadays) as
there wasn't clear way how to handle this particular check
cheaply in the spirit of RFC3517 (using only SACK blocks, not
holes + SACK blocks as with FACK). I sort of left it there as
a reminder but since it's confusing other people just remove
it and comment the missing-feature stuff instead.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Cc: Arnd Hannemann <hannemann@nets.rwth-aachen.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/inet_lro.c')
0 files changed, 0 insertions, 0 deletions