Rx: raise rx_minPeerTimeout to 20ms
authorJeffrey Altman <jaltman@your-file-system.com>
Sat, 2 Oct 2010 03:47:11 +0000 (23:47 -0400)
committerDerrick Brashear <shadow@dementia.org>
Sat, 2 Oct 2010 17:18:09 +0000 (10:18 -0700)
commitb62cacf889a6a231a97951baad6594e4066880bd
treee93aaa7fc7ae2cb24a962548324d22581cda5ff4
parent3cd3715e608b801b4848399e42cb47464e6e3cc3
Rx: raise rx_minPeerTimeout to 20ms

At 2ms it is possible for the packet we are sending to be resent
just about immediately as the retryTime computation occurs before
the send takes place and not afterwards.  If the network send blocks,
the retryTime may have already expired.

We do not want rx_minPeerTimeout to be too large though because the
value will end up being used as the backoff time period when the
actual RTT for the connection is less than the rx_minPeerTimeout
value.  Extensive testing shows 20ms to be an adequate compromise
that avoids the vast majority of unnecessary resends without
unnecessarily slowing down the connection if a packet is in fact
lost.

Change-Id: I87248ec1cff6a4d0862da0239d4c203024a783b6
Reviewed-on: http://gerrit.openafs.org/2881
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/rx/rx_globals.h