From 90597a4b6e4fc471a8f14cf5ca30f54387a80f3d Mon Sep 17 00:00:00 2001 From: Wu Guanghao Date: Thu, 19 Aug 2021 09:30:08 -0500 Subject: pvscan: use orphans lock to fix parallel pvscan and vgcreate start pvscan --cache /dev/sdb (read metadata from disk, no vg info) pvcreate /dev/sdb vgcreate vg /dev/sdb (write vg information to metadata, and update lvmetad) finish pvscan --cache /dev/sdb (No vg information in metadata from disk, overwrite lvmetad) vgs couldn't find vg Signed-off-by: Wu Guanghao --- tools/pvscan.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tools/pvscan.c b/tools/pvscan.c index 2e7a864de..51b5daa9a 100644 --- a/tools/pvscan.c +++ b/tools/pvscan.c @@ -491,6 +491,11 @@ static int _pvscan_cache(struct cmd_context *cmd, int argc, char **argv) return ECMD_FAILED; } + if (!lock_vol(cmd, VG_ORPHANS, LCK_VG_READ, NULL)) { + log_error("Can't get lock for orphan PVs."); + return ECMD_FAILED; + } + /* * This a special case where use_lvmetad=1 in lvm.conf but pvscan * cannot use lvmetad for some reason. In this case pvscan should @@ -780,6 +785,7 @@ out: if (!sync_local_dev_names(cmd)) stack; + unlock_vg(cmd, NULL, VG_ORPHANS); unlock_vg(cmd, NULL, VG_GLOBAL); return ret; } -- cgit v1.2.1