OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-2206-gfb03b13

Gerrit Code Review gerrit@openafs.org
Fri, 24 Feb 2012 04:28:52 -0800 (PST)


The following commit has been merged in the master branch:
commit 46e85dd4682a3273e15bbf96c4d5492087555cab
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Thu Feb 23 13:02:13 2012 -0600

    salvager: Do not fork for single VG salvage
    
    Currently we always fork a child in the salvager in order to salvage a
    volume group. I believe this is in order to protect SEGV, exit(), etc
    in one salvage operation from preventing salvaging anything else. When
    salvaging a single volume group, though, there appears to be little
    benefit.
    
    In addition, we need to keep the VG salvaging code in the same process
    as the cleanup code for single-volume salvages, so we can know which
    volumes were deleted by SalvageVolumeGroup, so we know which volumes
    to bring back online. So, do not fork for the singleVolumeNumber case.
    
    Note that for DAFS, we already never fork for the entire salvage
    operation when salvaging an individual volume group. So, this is
    effectively a non-DAFS-only change.
    
    Change-Id: I2b646b9ff932a673c714bc27ac32ed41cc3e22e6
    Reviewed-on: http://gerrit.openafs.org/6787
    Reviewed-by: Derrick Brashear <shadow@dementix.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>

 src/vol/vol-salvage.c |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)

-- 
OpenAFS Master Repository