OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-389-g8ec31f2

Gerrit Code Review gerrit@openafs.org
Mon, 8 Nov 2010 11:40:58 -0800 (PST)


The following commit has been merged in the master branch:
commit 5988923898e15cb50d550e54119869b38cb5bc5b
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Thu Sep 30 11:34:59 2010 -0500

    volser: Do not FSYNC_VOL_DONE temporary volumes
    
    Currently, the volserver just looks at the destroyMe field in a volume
    header to see if it should tell the fileserver a volume has been
    deleted (FSYNC_VOL_DONE opcode). However, temporary volumes created by
    the volserver (such as new volumes or clones) have destroyMe set, but
    are obviously not deleted, and so the fileserver should just be told
    FSYNC_VOL_LEAVE_OFF about these volumes instead.
    
    With commit bb9caf9822a53ad07cd3a5d6cea7f6b7eeeedd59, FSYNC_VOL_DONE
    and FSYNC_VOL_LEAVE_OFF actually do different things, so this does
    make a difference.
    
    Note that this commit slightly changes the meaning of the
    vp->needsPutBack field. Now it must be set to VOL_PUTBACK_DELETE to
    tell the fileserver that the volume has been deleted, and set to
    VOL_PUTBACK if it just needs to be given back to the fileserver.
    
    Change-Id: I020679c853b93593e5c75c4f30ac56312ba0f8d6
    Reviewed-on: http://gerrit.openafs.org/2872
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/vol/volume.c      |   12 +++++++++---
 src/vol/volume.h      |   11 ++++++++---
 src/volser/volprocs.c |    5 ++++-
 3 files changed, 21 insertions(+), 7 deletions(-)

-- 
OpenAFS Master Repository