OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-2496-g21ed79a

Gerrit Code Review
Mon, 7 May 2012 20:22:17 -0700 (PDT)

The following commit has been merged in the master branch:
commit 21ed79aeaee2d3b2b47436db0491943829ac44a6
Author: Andrew Deason <>
Date:   Wed May 2 11:38:57 2012 -0500

    vol: Avoid VBUSY/VRESTARTING trick for offline vop
    Currently, if GetVolume() finds that the volume we're trying to attach
    has a vol op that leaves the volume offline, we do the
    VBUSY/VRESTARTING trick as described in CheckVnode(). This doesn't
    make any sense for a couple of reasons.
    For one, VBUSY/VRESTARTING is not the correct error code to return to
    the client when an offline vol op is in progress and vp->specialStatus
    is not set everywhere else we yield VOFFLINE.
    Additionally, this block of code is only hit once for a particular vol
    op. Once we reach this section, the volume is in UNATTACHED state, and
    so on the next iteration of GetVolume we will immediately return
    VOFFLINE (or specialStatus). So the CheckVnode-like situation is not
    applicable, since we are not returning VBUSY to the same client for 15
    minutes; we would return VBUSY once and then return VOFFLINE.
    Change-Id: I0e8376df7937fd6bd01f9998371b9289c4ad2618
    Reviewed-by: Tom Keiser <>
    Tested-by: BuildBot <>
    Reviewed-by: Derrick Brashear <>

 src/vol/volume.c |    9 +--------
 1 files changed, 1 insertions(+), 8 deletions(-)

OpenAFS Master Repository