OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3104-gbcb077a

Gerrit Code Review
Mon, 12 Nov 2012 10:32:17 -0800 (PST)

The following commit has been merged in the master branch:
commit 0eaa0d1baa8b8fe115301f188ce32176acc7b065
Author: Michael Meffie <>
Date:   Tue Oct 30 10:22:40 2012 -0400

    vol: allow non-dafs volume utils to attach with V_READONLY again
    Allow non-fileserver, non-dafs, programs to attach volumes with the
    V_READONLY mode again. This was lost during the code changes for
    The caller sends a fssync request to the fileserver, which updates the
    on-disk contents of the volume headers, before the caller reads the
    volume headers, allowing the caller to have the most recent info about
    the volume. The fileserver still has the volume in use.
    Later in the attachment process, the inUse check is skipped for the case
    of a non-fileserver process which is attaching the volume using the
    V_READONLY mode, otherwise the attachment would incorrectly mark the
    volume as needing to be salvaged.
    Note: The mode checks in VMustCheckOutVolume() are correct. We must
    checkout the volume when attaching with the V_READONLY mode. This
    fix updates the VShouldCheckInUse(), in which an additional
    exception was added to cover the case for V_READONLY mode from a non-
    fileserver process.
    Note: A check is added in the dafs version of attach to avoid overwriting the
    inUse field when a volume utility is attaching a volume in V_READONLY mode.
    Currently, V_READONLY is not used by dafs, but this was done to avoid future
    Change-Id: I09d3af1e74087d0627399392e662e5075d41cdeb
    Reviewed-by: Andrew Deason <>
    Reviewed-by: Derrick Brashear <>
    Tested-by: BuildBot <>

 src/vol/volume.c        |    2 +-
 src/vol/volume_inline.h |   25 ++++++++++++++++++++-----
 2 files changed, 21 insertions(+), 6 deletions(-)

OpenAFS Master Repository