diff options
author | David Teigland <teigland@redhat.com> | 2016-04-07 13:45:26 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2016-04-13 14:19:06 -0500 |
commit | 58aa3692c7ac593d641d5436429923bd6ea2e701 (patch) | |
tree | 7d5522d24e65c7533b06423e1879785633ba6678 | |
parent | 1c8881004532cc54ab01e5b7c1ccb7327ee96b87 (diff) | |
download | lvm2-dev-dct-lvmetad-disabled-dups-13.tar.gz |
toollib: remove unneeded call in process_each_pvdev-dct-lvmetad-disabled-dups-13
process_each_pv was doing:
1. lvmcache_seed_infos_from_lvmetad()
sends pv_list request to lvmetad.
2. get_vgnameids()
sends vg_list request to lvmetad.
3. _get_all_devices()
first calls lvmcache_seed_infos_from_lvmetad(),
which is a no-op if it's already been called.
Because get_vgnameids() does not use the information
from lvmcache_seed_infos_from_lvmetad(), it does not
need to be called prior to get_all_devices where
it is actually needed.
-rw-r--r-- | tools/toollib.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/tools/toollib.c b/tools/toollib.c index ea6f4bc39..af6054751 100644 --- a/tools/toollib.c +++ b/tools/toollib.c @@ -3356,12 +3356,6 @@ int process_each_pv(struct cmd_context *cmd, dev_cache_full_scan(cmd->full_filter); } - /* - * Need pvid's set on all PVs before processing so that pvid's - * can be compared to find duplicates while processing. - */ - lvmcache_seed_infos_from_lvmetad(cmd); - if (!get_vgnameids(cmd, &all_vgnameids, only_this_vgname, 1)) { stack; return ret; |