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