diff options
| author | Ilpo Järvinen <[email protected]> | 2009-02-28 04:44:32 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2009-03-02 11:00:11 +0000 |
| commit | d0af4160d19ff2849386140881e729f9ba86f2aa (patch) | |
| tree | d9582acc750f249839602316a129eddb59e21acb /net/ipv4/tcp_timer.c | |
| parent | tcp: fix corner case issue in segmentation during rexmitting (diff) | |
| download | kernel-d0af4160d19ff2849386140881e729f9ba86f2aa.tar.gz kernel-d0af4160d19ff2849386140881e729f9ba86f2aa.zip | |
tcp: remove redundant code from tcp_mark_lost_retrans
Arnd Hannemann <[email protected]> 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 <[email protected]>
Cc: Arnd Hannemann <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'net/ipv4/tcp_timer.c')
0 files changed, 0 insertions, 0 deletions