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