diff options
author | David Teigland <teigland@redhat.com> | 2015-10-15 10:50:27 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2015-10-20 17:02:11 -0500 |
commit | d59e43782480dcd99a37e17e62d4c9725f609cae (patch) | |
tree | 8d33c5e6120d4b7481eb1319af87d4575723a5d9 | |
parent | 445baa9967e3f525f25d91b4412e42a73ca58d00 (diff) | |
download | lvm2-d59e43782480dcd99a37e17e62d4c9725f609cae.tar.gz |
lvmetad: warn about all duplicate PVs in command
When the command gets a list of alternate devices
from lvmetad, warn about them all directly. This
is not the same as the warnings when adding lvmcache,
which are related to which duplicate is preferred.
-rw-r--r-- | lib/cache/lvmetad.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/lib/cache/lvmetad.c b/lib/cache/lvmetad.c index c7f6ecb03..e315a73b3 100644 --- a/lib/cache/lvmetad.c +++ b/lib/cache/lvmetad.c @@ -439,6 +439,14 @@ static int _pv_populate_lvmcache(struct cmd_context *cmd, while (alt_device) { dev_alternate = dev_cache_get_by_devt(alt_device->v.i, cmd->filter); + + log_warn("PV on device %s (%d:%d %d) is also on device %s (%d:%d %d) %s", + dev_name(dev), + (int)MAJOR(devt), (int)MINOR(devt), (int)devt, + dev_alternate ? dev_name(dev_alternate) : "unknown", + (int)MAJOR(alt_device->v.i), (int)MINOR(alt_device->v.i), (int)alt_device->v.i, + pvid_txt); + if (dev_alternate) { if ((info_alternate = lvmcache_add(fmt->labeller, (const char *)&pvid, dev_alternate, vgname, (const char *)&vgid, 0))) { @@ -446,9 +454,6 @@ static int _pv_populate_lvmcache(struct cmd_context *cmd, info = info_alternate; lvmcache_get_label(info)->dev = dev_alternate; } - } else { - log_warn("Duplicate of PV %s dev %s exists on unknown device %"PRId64 ":%" PRId64, - pvid_txt, dev_name(dev), MAJOR(alt_device->v.i), MINOR(alt_device->v.i)); } alt_device = alt_device->next; } |