From 69ea2e3f94815dc94b5fdd8284d30a667690f5df Mon Sep 17 00:00:00 2001 From: David Teigland Date: Wed, 8 Feb 2023 13:34:35 -0600 Subject: [PATCH] vg_read: keep MISSING_PV when device with no mda reappears Remove old code that became incorrect at some point. It's probably a fragment of an old condition that was left behind because it wasn't understood. We don't want to drop the MISSING_PV flag just because the PV has no mda in use. The device that was missing may have stale data, so the user needs to decide if the device should be removed or restored. --- lib/metadata/metadata.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/lib/metadata/metadata.c b/lib/metadata/metadata.c index 91a23f931..10af75665 100644 --- a/lib/metadata/metadata.c +++ b/lib/metadata/metadata.c @@ -3571,12 +3571,6 @@ static void _set_pv_device(struct format_instance *fid, if (!pv->dev) pv->status |= MISSING_PV; - /* is this correct? */ - if ((pv->status & MISSING_PV) && pv->dev && (pv_mda_used_count(pv) == 0)) { - pv->status &= ~MISSING_PV; - log_info("Found a previously MISSING PV %s with no MDAs.", pv_dev_name(pv)); - } - /* Fix up pv size if missing or impossibly large */ if ((!pv->size || pv->size > (1ULL << 62)) && pv->dev) { if (!dev_get_size(pv->dev, &pv->size)) {