From dc02aa435650b1cdf2c2aaa84c143ce19c7707e6 Mon Sep 17 00:00:00 2001 From: Andrew Deason Date: Tue, 26 Mar 2013 17:00:05 -0500 Subject: [PATCH] volser: Indicate busy volume with VBUSY Commit 34fc86bcc749f3bd059831b7e5dae03dc09a9393 changed several uses of VBUSY to VOLSERVOLBUSY in order to detect retriable operations. However, one such change did not change an Rx abort code, but instead was used for the 'status' field for a volintInfo or volintXInfo structure. That is not really a general error code, but a field with a few specific known values (at least, that is how existing clients interpret it). Go back to using VBUSY, so clients indicate the volume as busy, instead of as offline/unattached. Reported by Andy Malato. Reviewed-on: http://gerrit.openafs.org/9678 Tested-by: BuildBot Reviewed-by: Jeffrey Altman Reviewed-by: Derrick Brashear (cherry picked from commit 3929282d2767bd35e54037e5477e917979c84be3) Change-Id: Ic6efeeca7f37d05704b09a3244bcf40641cae5ea Reviewed-on: http://gerrit.openafs.org/9685 Reviewed-by: Andrew Deason Tested-by: BuildBot Reviewed-by: Stephan Wiesand --- src/volser/volprocs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/volser/volprocs.c b/src/volser/volprocs.c index 87e383e45..04a393e78 100644 --- a/src/volser/volprocs.c +++ b/src/volser/volprocs.c @@ -2231,7 +2231,7 @@ GetVolInfo(afs_uint32 partId, ttc = NewTrans(volumeId, partId); if (!ttc) { code = -3; - VOLINT_INFO_STORE(handle, status, VOLSERVOLBUSY); + VOLINT_INFO_STORE(handle, status, VBUSY); VOLINT_INFO_STORE(handle, volid, volumeId); goto drop; } -- 2.39.5