OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_18a-20-g16c68a7

Gerrit Code Review gerrit@openafs.org
Thu, 8 Nov 2012 06:29:29 -0800 (PST)


The following commit has been merged in the openafs-devel-1_7_x branch:
commit 27acd671c85e1fc84a000c8c26c20284980ad7f5
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Fri Oct 26 14:55:02 2012 +0100

    rx: Remove duplicate out of order ACK check
    
    Once we've moved the congestion window, there's no going back. So
    any ACK packets that attempt to move the window backwards by including
    a 'firstPacket' value earlier than the current window position must
    be ignored.
    
    However, we check (and ignore) these packets twice. Once in
    rxi_ReceivePacket, which only checks in the client side case, and again
    in rxi_ReceiveAckPacket, which has a more complete check that runs for
    both client and server connections.
    
    Remove the identical check from rxi_ReceivePacket in a continuing effort
    to clean up this bit of code.
    
    Reviewed-on: http://gerrit.openafs.org/8299
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    (cherry picked from commit 8d359e6dff5317698597e77f0a1dd5ba2bfb569a)
    
    Change-Id: I29c72136d02a16381e33d8836f35528ba7b1f055
    Reviewed-on: http://gerrit.openafs.org/8396
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>

 src/rx/rx.c |   25 +------------------------
 1 files changed, 1 insertions(+), 24 deletions(-)

-- 
OpenAFS Master Repository