OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1278-gcd326b2

Gerrit Code Review gerrit@openafs.org
Tue, 7 Jun 2011 08:41:28 -0700 (PDT)


The following commit has been merged in the master branch:
commit cd326b2f54c3397468807c32ce0834f73c9d5d1b
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date:   Sun Jun 5 18:41:24 2011 -0400

    rx: Add RX_CALL_ACKALL_SENT flag and rxi_SendAck processing
    
    3cd3715e608b801b4848399e42cb47464e6e3cc3 modified rxi_ReceiveDataPacket
    to send an ACKALL whenever RX_CALL_RECEIVE_DONE is set on the call.
    This produced the potential for a race with ACKs that set the
    firstPacket value to 'rnext' when the receive queue for the call
    has yet to be emptied.  From the perspective of receiver the ACK
    was already processed and does not require a response since the
    previously received ACKALL acknowledged the delivery of all data
    packets to the application.  When sending ACKs after ACKALL it is
    therefore required that firstPacket be set to the sequence number
    after the last unprocessed packet in the receive queue.
    
    Thanks to Simon Wilkinson for his extensive assistance in identifying
    the problem and the development of this patchset.
    
    Change-Id: I3bdf0c8f297b1d91b1a2bf3284adfeb9301874eb
    Reviewed-on: http://gerrit.openafs.org/4798
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>

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

-- 
OpenAFS Master Repository