From: Jacob Thebault-Spieker <summatusmentis@gmail.com>
Date: Tue, 19 Jul 2011 16:02:49 +0000 (-0700)
Subject: remove check for disallowing clones of backup or ro volumes
X-Git-Tag: upstream/1.8.0_pre1^2~3136
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=4e1a65ba7edec519acc67f0dcf2237a0f26e92aa;p=packages%2Fo%2Fopenafs.git

remove check for disallowing clones of backup or ro volumes

removes the if-statement ensuring that the volume being cloned is
not a backup volume, nor a read-only volume. This allows clones
from any type of volume to a given volume. Parent volume meta-data
is maintained, only the cloneId value changes.

Change-Id: If8444fcabc9b9106db7db7df881ae07e8d287915
Reviewed-on: http://gerrit.openafs.org/5046
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
---

diff --git a/src/volser/volprocs.c b/src/volser/volprocs.c
index 227a632f3..8083cc2e3 100644
--- a/src/volser/volprocs.c
+++ b/src/volser/volprocs.c
@@ -930,12 +930,6 @@ VolReClone(struct rx_call *acid, afs_int32 atrans, afs_int32 cloneId)
     TSetRxCall(tt, acid, "ReClone");
 
     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));