diff options
author | David Teigland <teigland@redhat.com> | 2023-02-15 10:21:13 -0600 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2023-02-15 10:21:13 -0600 |
commit | 08265641f3a8590f7bb41c92af65e572cdc8d099 (patch) | |
tree | 5e4b8336ee4e1effcc46ca92cd675bf8bc62b34f /tools/pvscan.c | |
parent | bd05318ba2fc588be6339f5dc61f09195996b0e9 (diff) | |
download | lvm2-08265641f3a8590f7bb41c92af65e572cdc8d099.tar.gz |
pvscan: remove warning about excluded root dev
This was trying to warn about a misconfig where the root PV
was excluded by the devices file, but it wasn't covering all
cases. Also, it's not very useful because the root LV is
usually activated directly in the initrd.
Diffstat (limited to 'tools/pvscan.c')
-rw-r--r-- | tools/pvscan.c | 29 |
1 files changed, 0 insertions, 29 deletions
diff --git a/tools/pvscan.c b/tools/pvscan.c index b834dfd4c..8f51a0b42 100644 --- a/tools/pvscan.c +++ b/tools/pvscan.c @@ -1355,29 +1355,6 @@ static int _pvscan_cache_all(struct cmd_context *cmd, int argc, char **argv, return 1; } -/* - * If /dev/sda* of /dev/vda* is excluded by the devices file - * it's usually a misconfiguration that prevents proper booting, - * so make it a special case to give extra info to help debugging. - */ -static void _warn_excluded_root(struct cmd_context *cmd, struct device *dev) -{ - struct dev_use *du; - const char *cur_idname; - - if (!(du = get_du_for_devname(cmd, dev_name(dev)))) { - log_warn("WARNING: no autoactivation for %s: not found in system.devices.", dev_name(dev)); - return; - } - - cur_idname = device_id_system_read(cmd, dev, du->idtype); - - log_warn("WARNING: no autoactivation for %s: system.devices %s current %s.", - dev_name(dev), du->idname, cur_idname ?: "missing device id"); - - free((void*) cur_idname); -} - static int _pvscan_cache_args(struct cmd_context *cmd, int argc, char **argv, struct dm_list *complete_vgnames) { @@ -1523,12 +1500,6 @@ static int _pvscan_cache_args(struct cmd_context *cmd, int argc, char **argv, log_print_pvscan(cmd, "%s excluded: %s.", dev_name(devl->dev), dev_filtered_reason(devl->dev)); dm_list_del(&devl->list); - - /* Special case warning when probable root dev is missing from system.devices */ - if ((devl->dev->filtered_flags & DEV_FILTERED_DEVICES_FILE) && - (!strncmp(dev_name(devl->dev), "/dev/sda", 8) || - !strncmp(dev_name(devl->dev), "/dev/vda", 8))) - _warn_excluded_root(cmd, devl->dev); } } |