OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-556-gf88c91f

Gerrit Code Review gerrit@openafs.org
Fri, 10 Dec 2010 13:02:38 -0800 (PST)


The following commit has been merged in the master branch:
commit 95fc9ece507d3c5bd7914afadaaae9456eea9b7a
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Wed Dec 8 12:41:44 2010 -0600

    DAFS: Fix VOL_QUERY_VOP error codes
    
    Volumes in the VOL_STATE_DELETED state effectively do not exist. So,
    when receiving an FSYNC_VOL_QUERY_VOP request for a volume, report
    FSYNC_UNKNOWN_VOLID for a volume in the VOL_STATE_DELETED state.
    Similarly, if the volume exists but on a different partition, we
    should return the FSYNC_WRONG_PART reason code.
    
    This allows volumes to be usable by the volume server in some corner
    cases. For example, when a volume X exists on partitions A and B, and
    the volserver deletes X on B. The fileserver then puts volume X in the
    DELETED state, allowing checkouts over fsync, but FSYNC_VOL_QUERY_VOP
    reports that no vop exists.
    
    With this change, FSYNC_VOL_QUERY_VOP will instead report that the
    volume does not exist, and so FSYNC_VerifyCheckout can succeed for
    such volumes.
    
    Change-Id: I6c4f7deead745ddba44963a9f66a9f2ef25686a5
    Reviewed-on: http://gerrit.openafs.org/3493
    Reviewed-by: Tom Keiser <tkeiser@sinenomine.net>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/vol/fssync-client.c |    8 +++++---
 src/vol/fssync-server.c |    8 +++++---
 2 files changed, 10 insertions(+), 6 deletions(-)

-- 
OpenAFS Master Repository