From mboxrd@z Thu Jan  1 00:00:00 1970
From: Milan Broz <mbroz@redhat.com>
Date: Mon, 15 Mar 2010 14:18:17 +0100
Subject: [PATCH] Fix pvcreate device check.
In-Reply-To: <1268658848-24437-1-git-send-email-mbroz@redhat.com>
References: <1268658848-24437-1-git-send-email-mbroz@redhat.com>
Message-ID: <4B9E3399.2020800@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


On 03/15/2010 02:14 PM, Milan Broz wrote:
> @@ -1294,17 +1294,9 @@ static int pvcreate_check(struct cmd_context *cmd, const char *name,
>  
>  	/* Is there an md superblock here? */
>  	if (!dev && md_filtering()) {
> -		unlock_vg(cmd, VG_ORPHANS);
> -
> -		persistent_filter_wipe(cmd->filter);
> -		lvmcache_destroy(cmd, 1);
> -
> +		refresh_filters(cmd);
>  		init_md_filtering(0);
> -		if (!lock_vol(cmd, VG_ORPHANS, LCK_VG_WRITE)) {
> -			log_error("Can't get lock for orphan PVs");
> -			init_md_filtering(1);
> -			return 0;
> -		}


> +		lvmcache_label_scan(cmd, 0);

even this rescan line is not needed, device must be in cache already,
just it is filtered out by MD filter.

Milan