From f37b6dfaf050ec0270979c8310fb471c16b66cd3 Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Fri, 2 Sep 2011 18:32:40 -0400 Subject: [PATCH] Windows: improve handling of volume rename The windows cache manager tracks volumes by volume group. Up to this point all volume location updates have been performed by the volume name. What if the volume name was altered? In this case the volume location information for the in use volume ids will fail until a mount point to the new name is queried. Before marking the volume group as non-existent attempt to perform a lookup using either the volume id for the readwrite or readonly volume. Change-Id: I51b383150b0186867e1c51a28b01e8483ee470f2 Reviewed-on: http://gerrit.openafs.org/5330 Tested-by: BuildBot Tested-by: Jeffrey Altman Reviewed-by: Jeffrey Altman --- src/WINNT/afsd/cm_volume.c | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/src/WINNT/afsd/cm_volume.c b/src/WINNT/afsd/cm_volume.c index 5a4c592..bfec95c 100644 --- a/src/WINNT/afsd/cm_volume.c +++ b/src/WINNT/afsd/cm_volume.c @@ -353,6 +353,27 @@ long cm_UpdateVolumeLocation(struct cm_cell *cellp, cm_user_t *userp, cm_req_t * &method, userp, reqp); } + /* + * What if there was a volume rename? The volume name no longer exists but the + * volume id might. Try to refresh the volume location information based one + * of the readwrite or readonly volume id. + */ + if (code == CM_ERROR_NOSUCHVOLUME) { + if (volp->vol[RWVOL].ID != 0) { + code = cm_GetEntryByID(cellp, volp->vol[RWVOL].ID, &vldbEntry, &nvldbEntry, +#ifdef MULTIHOMED + &uvldbEntry, +#endif + &method, userp, reqp); + } else if (volp->vol[ROVOL].ID != 0) { + code = cm_GetEntryByID(cellp, volp->vol[ROVOL].ID, &vldbEntry, &nvldbEntry, +#ifdef MULTIHOMED + &uvldbEntry, +#endif + &method, userp, reqp); + } + } + lock_ObtainWrite(&volp->rw); if (code == 0) { afs_int32 flags; -- 1.9.4