OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-devel-1_5_77-93-gfb8a204

Gerrit Code Review gerrit@openafs.org
Mon, 25 Oct 2010 17:49:59 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit fb8a2049e355dc9a4cd6e7264a2d18d61447e2af
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Sun Oct 10 08:04:41 2010 -0400

    rx: Don't malloc the xmit list
    
    Building the transmit list happens in a time critical section of
    code. Using malloc to allocate the list which holds the packets to
    be transmitted slows down this critical section. Instead, just
    allocate the space as part of the call structure.
    
    Locking of xmitList is somewhat tricksy, as the call->lock is
    dropped over calls to sendmsg(). However, the xmitList is protected
    by the TQ_BUSY call flag, which prevents multiple threads from
    usign the transmit queue, and hence the xmitList, simultaneously.
    
    Reviewed-on: http://gerrit.openafs.org/2957
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    (cherry picked from commit 09aec1539dbe4e4c72ec7acadf2d25e6f93c5982)
    Change-Id: I589fa248a124a15248e23b6154e193e4d136fde8
    Reviewed-on: http://gerrit.openafs.org/3102
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/rx/rx.c |   26 ++++++--------------------
 src/rx/rx.h |    7 +++++--
 2 files changed, 11 insertions(+), 22 deletions(-)

-- 
OpenAFS Master Repository