diff options
author | David Teigland <teigland@redhat.com> | 2016-01-14 13:47:44 -0600 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2016-01-18 09:36:55 -0600 |
commit | 5cd4d46f303ecf5212b5de0eb96c0a356453d289 (patch) | |
tree | c23302b0e74df833e5ab6ba1ac040061c1baed5f | |
parent | 06346eab849b272bcd99196a9b594da29abdb19a (diff) | |
download | lvm2-5cd4d46f303ecf5212b5de0eb96c0a356453d289.tar.gz |
Revert "Revert "process_each_pv: remove unnecessary workaround""
This reverts commit 6d09c8c2c45ea1dea243134b8badc841a87cc979.
Try again to remove the workaround.
-rw-r--r-- | tools/toollib.c | 17 |
1 files changed, 1 insertions, 16 deletions
diff --git a/tools/toollib.c b/tools/toollib.c index 728e61d4d..97cc1bf6e 100644 --- a/tools/toollib.c +++ b/tools/toollib.c @@ -3033,7 +3033,6 @@ static int _process_pvs_in_vg(struct cmd_context *cmd, const char *pv_name; int selected; int process_pv; - int dev_found; int ret_max = ECMD_PROCESSED; int ret = 0; @@ -3087,21 +3086,7 @@ static int _process_pvs_in_vg(struct cmd_context *cmd, else log_very_verbose("Processing PV %s in VG %s.", pv_name, vg->name); - dev_found = _device_list_remove(all_devices, pv->dev); - - /* - * FIXME PVs with no mdas may turn up in an orphan VG when - * not using lvmetad as well as their correct VG. They - * will be missing from all_devices the second time - * around but must not be processed twice or trigger a message. - * - * Missing PVs will also need processing even though they are - * not present in all_devices. - */ - if (!dev_found && !is_missing_pv(pv)) { - log_verbose("Skipping PV %s in VG %s: not in device list.", pv_name, vg->name); - continue; - } + _device_list_remove(all_devices, pv->dev); if (!skip) { ret = process_single_pv(cmd, vg, pv, handle); |