OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1788-g4552dc5

Gerrit Code Review
Mon, 24 Oct 2011 13:14:22 -0700 (PDT)

The following commit has been merged in the master branch:
commit 4552dc552687267fce3c7a6a9c7f4a1e9395c8e5
Author: Andrew Deason <>
Date:   Mon Aug 29 17:41:31 2011 -0500

    Currently VRequestSalvage_r takes a flag,
    VOL_SALVAGE_INVALIDATE_HEADER, which causes the header for the
    specified volume to be freed (via FreeVolumeHeader). This is almost
    never safe to do, since there may be other users of the specified
    volume that can be accessing the volume header at the same time.
    There is also no reason to invalidate the header at the time of the
    VRequestSalvage_r call, since the header must be invalidated when we
    detach the volume (other utilities may change header information). So,
    if there are any problems in the future because we do not invalidate
    the header at the time of VRequestSalvage_r, it is the fault of the
    detachment/offlining logic.
    So, remove VOL_SALVAGE_INVALIDATE_HEADER and all of its users. Take
    this opportunity to correctly document the VRequestSalvage_r headers
    in the VRequestSalvage_r comment, as it was previously missing the
    Change-Id: I65c4cc0e81171aafa7b78ae9e1f503edb2f3f2c3
    Reviewed-by: Derrick Brashear <>
    Tested-by: BuildBot <>

 src/vol/volume.c |   39 ++++++++++++---------------------------
 src/vol/volume.h |    3 +--
 2 files changed, 13 insertions(+), 29 deletions(-)

OpenAFS Master Repository