OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_1-89-ga7ed553

[Gerrit Code Review]

The following commit has been merged in the openafs-devel-1_7_x branch:
commit a7ed5536e142f62b28094274647e66fed40d14cd
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Mon Aug 29 17:41:31 2011 -0500

    DAFS: Remove VOL_SALVAGE_INVALIDATE_HEADER
    
    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
    VOL_SALVAGE_NO_OFFLINE flag.
    
    Change-Id: I5d0451a7326997b38a466ec427c2e12e0cea4ae6
    Reviewed-on: http://gerrit.openafs.org/5319
    Reviewed-by: Derrick Brashear <shadow@dementix.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    (cherry-picked from 4552dc552687267fce3c7a6a9c7f4a1e9395c8e5)
    Reviewed-on: http://gerrit.openafs.org/5919
    Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>
    Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>

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

-- 
OpenAFS Master Repository