OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0b-119-g4ff4ec8

Gerrit Code Review gerrit@openafs.org
Tue, 8 Nov 2011 19:40:42 -0800 (PST)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit 4ff4ec8ac94a75385d919f9031abaec31aca9b26
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Wed Oct 12 09:47:14 2011 -0400

    rx: Don't clear the receive queue when out of packets
    
    We can end up discarding a receive queue that's been soft acked,
    effectively taking back soft acks we sent. Whilst the RX
    documentation says that a client can drop soft acked packets at
    will, our RX implementation assumes that if the final packet in
    a call has been soft acked, we won't clear the queue. If a client
    clears the queue in this situation, the call will hang.
    
    What *should* happen is that we should take necessary locks,
    confirm that we have not soft-acked all of the packets in a flow,
    and then discard, or, if we're just going to discard, error the
    call.
    
    Reviewed-on: http://gerrit.openafs.org/5603
    Reviewed-by: Derrick Brashear <shadow@dementix.org>
    Tested-by: Derrick Brashear <shadow@dementix.org>
    (cherry-picked from f722a75fe21963d64217375f1f7bbb0eb14befb4)
    
    Change-Id: Ie2f14a0ed7b6d258a7f456387df44ac95fd1748c
    Reviewed-on: http://gerrit.openafs.org/5820
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementix.org>

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

-- 
OpenAFS Master Repository