OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1123-g89cdb34

Gerrit Code Review gerrit@openafs.org
Tue, 26 Apr 2011 17:37:02 -0700 (PDT)


The following commit has been merged in the master branch:
commit 44801d157e6fd0933f37f0fc6451e37bdca285f3
Author: Marc Dionne <marc.c.dionne@gmail.com>
Date:   Sat Apr 16 14:19:57 2011 -0400

    ubik: always hold DB lock for urecovery_ResetState()
    
    ubik_ResetState requires callers to hold the DB lock, since it modifies
    urecovery_state.  All callers of ubeacon_AmSyncSite outside of the beacon
    package hold the DB lock, but calls from the beacon thread do not, and
    can't block on getting the DB lock if we're sync site.
    
    Add a beacon internal version of ubeacon_AmSyncSite that skips the
    call to ResetState, and have the callers take the DB lock and call
    ResetState themselves if needed.  They can take the lock in this case
    because we know we're not the sync site.  Refactor the exported
    ubeacon_AmSyncSite in terms of this new function.
    
    Change-Id: I88b231010dd52adf6e43a17802e83d12568afc6b
    Reviewed-on: http://gerrit.openafs.org/4490
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/ubik/beacon.c |   67 +++++++++++++++++++++++++++++++++-------------------
 1 files changed, 42 insertions(+), 25 deletions(-)

-- 
OpenAFS Master Repository