summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Teigland <teigland@redhat.com>2016-04-07 13:45:26 -0500
committerDavid Teigland <teigland@redhat.com>2016-04-15 13:04:40 -0500
commit8388434d8644322baa0a99aa4c9938af906ec180 (patch)
tree69c7d9cd8e8c73f63846fba3befc80643af61e37
parent850b7fef1009e9a174b48256ff47bcfd0c45bcd4 (diff)
downloadlvm2-8388434d8644322baa0a99aa4c9938af906ec180.tar.gz
toollib: remove unneeded call in process_each_pv
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.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/tools/toollib.c b/tools/toollib.c
index b1225cd67..dce00268d 100644
--- a/tools/toollib.c
+++ b/tools/toollib.c
@@ -3360,12 +3360,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;