OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_66-108-gd393aab

Gerrit Code Review gerrit@openafs.org
Mon, 30 Nov 2009 09:01:09 -0800 (PST)


The following commit has been merged in the master branch:
commit d393aabca577917b107afdd42efb40cc2fdac50c
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Mon Oct 26 14:09:41 2009 -0500

    Correct duplicate special inodes while salvaging
    
    Right now when the salvager encounters duplicate special inodes for a
    volume, it refuses to salvage the volume, presumably because it does not
    know which inodes to use when recreating the volume header.
    
    However, this can cause the confusing state where the fileserver and
    various volume utilities have no problem with a volume, but the salvager
    refuses to salvage it and marks the volume as needing salvage. When
    salvaging, if we already have a volume header, and we encounter
    duplicate special inodes, it is likely that the special inode referenced
    by the volume header is the correct one.
    
    So, instead of erroring out, keep track of which inodes are referenced
    in the volume header, and if there are any duplicates, either ignore or
    delete the unreferenced ones, depending on the -orphans setting. Also be
    a little more verbose when logging errors in this area.
    
    Change-Id: I15e34e8f2bccdc666bb59e3730fdf8bf51624259
    Reviewed-on: http://gerrit.openafs.org/736
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
    Reviewed-by: Steve Simmons <scs@umich.edu>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/vol/vol-salvage.c |  119 ++++++++++++++++++++++++++++++++++++++++++++++---
 1 files changed, 112 insertions(+), 7 deletions(-)

-- 
OpenAFS Master Repository