net: tcp: Remove the RETRAN flag
authorBarret Rhoden <brho@cs.berkeley.edu>
Thu, 20 Jul 2017 15:46:46 +0000 (11:46 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Fri, 21 Jul 2017 15:56:27 +0000 (11:56 -0400)
commit34d65403cdd0218ee8724514a4e646fc81e27387
tree273ef7bce3c3f816c1319438676d42a66de0a79d
parent34b57464c3109f77f5f9dd4257dda089c7a5b3d3
net: tcp: Remove the RETRAN flag

Use tcb->snd.recovery instead.  RETRAN actually only applied to one pass
through update(), so I don't know what exactly it was doing.

Regarding the RTTM, we're not supposed to use ACKs from retrans.  The
RETRAN flag would do that for one update() call, but not for any others.
Using snd.recovery does the trick.

Note that we will get SACKs for new data (beyond recovery_pt) during
recovery, and these will not be used to adjust the RTT.  That's probably
fine, and overall better than the RETRAN flag.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/src/net/tcp.c