diff options
author | David Teigland <teigland@redhat.com> | 2016-04-07 13:45:26 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2016-04-15 13:04:40 -0500 |
commit | 8388434d8644322baa0a99aa4c9938af906ec180 (patch) | |
tree | 69c7d9cd8e8c73f63846fba3befc80643af61e37 | |
parent | 850b7fef1009e9a174b48256ff47bcfd0c45bcd4 (diff) | |
download | lvm2-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.c | 6 |
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; |