OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1276-gff30293

Gerrit Code Review gerrit@openafs.org
Tue, 7 Jun 2011 07:57:39 -0700 (PDT)


The following commit has been merged in the master branch:
commit ff30293624a7e31d585b9e8f2e1a439cf87b6524
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date:   Sun Jun 5 16:02:46 2011 -0400

    rx: do not rxi_AckAll for one data packet call
    
    rxi_ReceiveDataPacket() calls rxi_AckAll() when the call reaches
    the RX_CALL_RECEIVE_DONE state to permit the caller to empty the
    transmit queue.  That reduces the memory consumption of the caller
    and avoids unnecessary retransmits which the call is in process.
    
    If the call data consists of a single packet it is possible that
    Ping ACK packets sent as part of connection establishment could
    race with the ACKALL and be delivered out of order.  If the Ping
    ACK is delivered second, it will be ignored by the peer forcing
    a two second delay in connection establishment.  To avoid the race
    do not send an ACKALL for a single packet call.
    
    Change-Id: I69d967b3b2e9ee77636ca12bc7ade4896bb8a071
    Reviewed-on: http://gerrit.openafs.org/4799
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>

 src/rx/rx.c |   16 +++++++++++-----
 1 files changed, 11 insertions(+), 5 deletions(-)

-- 
OpenAFS Master Repository