OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_18a-13-g6af838e
Gerrit Code Review
gerrit@openafs.org
Thu, 8 Nov 2012 06:28:03 -0800 (PST)
The following commit has been merged in the openafs-devel-1_7_x branch:
commit 6af838e28f0cae3e37c4814c85ba3ab2de2efc87
Author: Simon Wilkinson <sxw@your-file-system.com>
Date: Thu Oct 25 11:27:33 2012 +0100
rx: Don't double check conn->call
We currently have
call = conn->call[channel]
if (call) {
...
} else {
call = conn->call[channel]
if (call) {
...
}
}
As we don't drop (or acquire) any locks between the first and the
second check of call, there's no way that the result can be different
from the first time we checked. So just get rid of the uneccessary
code, and reindent the following block to match.
Reviewed-on: http://gerrit.openafs.org/8290
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
(cherry picked from commit e93b6dca2f593e479499122eedef1e65452aaaac)
Change-Id: I5f3283a76945be92790baf58aafc32c501666b97
Reviewed-on: http://gerrit.openafs.org/8390
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/rx/rx.c | 66 +++++++++++++++++++++++++++-------------------------------
1 files changed, 31 insertions(+), 35 deletions(-)
--
OpenAFS Master Repository