OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1121-gb2e6b42

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


The following commit has been merged in the master branch:
commit b2e6b426e6351da8b13e1d8b55be220fc98df5e5
Author: Marc Dionne <marc.c.dionne@gmail.com>
Date:   Sat Apr 16 12:56:05 2011 -0400

    ubik: set UBIK_RECLABELDB before propagating version
    
    Quoting Jeffrey Hutzelman:
    
    In udisk_commit(), when committing the first write transaction
    after becoming sync site, the database is relabelled.  In this
    case, the UBIK_RECLABELDB recovery state bit should be set before
    propagating the label change to other servers, rather than after.
    
    This is because ContactQuorum_DISK_Setversion() will
    release the database lock, at which point the recovery state may
    be cleared by urecovery_ResetState() running in another thread.
    It is important that a relabelling which occurs before recovery
    state is cleared not result in the UBIK_RECLABELDB recovery state
    bit being set after; otherwise, the server may fail to correctly
    relabel the database the next time it becomes sync site.
    
    Change-Id: I9753a24c84cf45cdbb11a1d8b7ab262fbe487204
    Reviewed-on: http://gerrit.openafs.org/4489
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/ubik/disk.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository