[lvm-devel] master - lvmlockd: VG lock can be used when changing lock type

David Teigland teigland at fedoraproject.org
Thu Aug 27 21:37:16 UTC 2015


Gitweb:        http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=e5d99cb9e685ba167f10413694e5d9d34fe90bb0
Commit:        e5d99cb9e685ba167f10413694e5d9d34fe90bb0
Parent:        3c1924c9c0d45cbce67aa3c1dad70737a6956cc3
Author:        David Teigland <teigland at redhat.com>
AuthorDate:    Thu Aug 27 16:34:51 2015 -0500
Committer:     David Teigland <teigland at redhat.com>
CommitterDate: Thu Aug 27 16:34:51 2015 -0500

lvmlockd: VG lock can be used when changing lock type

This bit was missed from commit de4db6a that added
changing lock_type.
---
 tools/vgchange.c |    9 ---------
 1 files changed, 0 insertions(+), 9 deletions(-)

diff --git a/tools/vgchange.c b/tools/vgchange.c
index f57fa15..6d86051 100644
--- a/tools/vgchange.c
+++ b/tools/vgchange.c
@@ -1048,15 +1048,6 @@ static int _lockd_vgchange(struct cmd_context *cmd, int argc, char **argv)
 		cmd->lockd_vg_disable = 1;
 
 	/*
-	 * In most cases, lockd_vg does not apply when changing lock type.
-	 * (We don't generally allow changing *from* lockd type yet.)
-	 * lockd_vg could be called within _vgchange_locktype as needed.
-	 */
-
-	if (arg_is_set(cmd, locktype_ARG))
-		cmd->lockd_vg_disable = 1;
-
-	/*
 	 * Changing system_id or lock_type must only be done on explicitly
 	 * named vgs.
 	 */




More information about the lvm-devel mailing list