From mboxrd@z Thu Jan  1 00:00:00 1970
From: Milan Broz <mbroz@redhat.com>
Date: Tue, 07 Jul 2009 11:01:38 +0200
Subject: [PATCH] Remove LOCK_KEEP and READ_CHECK_EXISTENCE from vgsplit.
In-Reply-To: <1246940664-14890-2-git-send-email-dwysocha@redhat.com>
References: <1246940664-14890-1-git-send-email-dwysocha@redhat.com>
	<1246940664-14890-2-git-send-email-dwysocha@redhat.com>
Message-ID: <4A530EF2.9040705@redhat.com>
List-Id: <lvm-devel.redhat.com>
To: lvm-devel@redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

Dave Wysochanski wrote:
> diff --git a/tools/vgsplit.c b/tools/vgsplit.c
> index 9cd90b8..0dc5bc4 100644
> --- a/tools/vgsplit.c
> +++ b/tools/vgsplit.c
> @@ -342,11 +342,12 @@ int vgsplit(struct cmd_context *cmd, int argc, char **argv)
>  		existing_vg = 1;
>  		vg_to = vg_read_for_update(cmd, vg_name_to, NULL,
>  					   READ_REQUIRE_RESIZEABLE |
> -					   LOCK_NONBLOCKING | LOCK_KEEP |
> -					   READ_CHECK_EXISTENCE);
> +					   LOCK_NONBLOCKING);
>  
> -		if (vg_read_error(vg_to))
> -			goto_bad;
> +		if (vg_read_error(vg_to)) {

	but vg_name_to is locked by vg_lock_newname() previously?
	so unlock_and_release here?

	also now code missing stack; (note goto_bad before)

> +			vg_release(vg_to);
> +			goto bad2;
> +		}
>  


Milan