From: Derrick Brashear <shadow@dementix.org>
Date: Tue, 13 Dec 2011 16:22:38 +0000 (-0500)
Subject: volser: allow cloning non-rw volumes
X-Git-Tag: upstream/1.6.1.pre4^2~63
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=6b66b3b705b78c959b3ae321a8cb06e2fe166086;p=packages%2Fo%2Fopenafs.git

volser: allow cloning non-rw volumes

remove EROFS error which is the only thing preventing a working clone
on a non-RW.

Reviewed-on: http://gerrit.openafs.org/6249
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Tested-by: Derrick Brashear <shadow@dementix.org>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
(cherry picked from commit f1de04f3b35e91923efddca57e744b2138619223)

Change-Id: Ieb02a2d2c4d59681f5d6f372c7cd77a181d214dd
Reviewed-on: http://gerrit.openafs.org/6760
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
---

diff --git a/src/volser/volprocs.c b/src/volser/volprocs.c
index 95339ad57..89196911b 100644
--- a/src/volser/volprocs.c
+++ b/src/volser/volprocs.c
@@ -743,12 +743,6 @@ VolClone(struct rx_call *acid, afs_int32 atrans, afs_uint32 purgeId,
 	purgevp = NULL;
     }
     originalvp = tt->volume;
-    if ((V_type(originalvp) == backupVolume)
-	|| (V_type(originalvp) == readonlyVolume)) {
-	Log("1 Volser: Clone: The volume to be cloned must be a read/write; aborted\n");
-	error = EROFS;
-	goto fail;
-    }
     if ((V_destroyMe(originalvp) == DESTROY_ME) || !V_inService(originalvp)) {
 	Log("1 Volser: Clone: Volume %d is offline and cannot be cloned\n",
 	    V_id(originalvp));