OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0pre6-21-g8103a9a

[Gerrit Code Review]

The following commit has been merged in the openafs-stable-1_6_x branch:
commit cbd4ef0161c20de025cadba98bcc78f63b76de2c
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Fri Jun 17 20:35:59 2011 +0100

    rx: Remove incorrect backoff code
    
    The ACK packet handling routine contains code which causes the
    RTT to backoff if the selective ACK response indicates that there is
    a missing packet. The comment justifies this code as being in line
    with Phil Karn's work on TCP.
    
    However, the TCP behaviour is that we backoff when we enter resend. Both
    TCP and RX have difficulty computing RTTs for resent packets due to the
    ambiguous ACK problem. Whilst RX is slightly better than TCP in this
    regard, we can't always tell whether an ACK refers to the original, or
    resent packet, so resent packets are unable to contribute to the RTT.
    This means that if the RTT ends up too low for the connection, and we
    start resending every packet, the RTT will never grow to account for
    this, as we never feed it any packet samples.
    
    Karn's solution to this was to backoff (double) the RTT value when we
    resend a packet, and then to not drop it back down until we receive an
    ACK that we can count. This means that we will always get a new sample
    for the connection, and the RTT will grow again.
    
    The original author confirms that the current behaviour in RX is
    incorrect, so simply remove it with this patchset.
    
    Reviewed-on: http://gerrit.openafs.org/4860
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    (cherry picked from commit b65944973a24e9365dc1ff118ded4c3a1e25f782)
    
    Change-Id: I4d2766d98883dad4f27ff4c52e2a03a49733f89f
    Reviewed-on: http://gerrit.openafs.org/4911
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/rx/rx.c |   18 ------------------
 src/rx/rx.h |    1 -
 2 files changed, 0 insertions(+), 19 deletions(-)

-- 
OpenAFS Master Repository