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.

diff --git a/tools/toollib.c b/tools/toollib.c
index b9db9cb31fdd04e4b37f9a0355d5f492f6e80456..ac2a57a4865760098e33c3e34c904215f655af9d 100644 (file)
--- a/tools/toollib.c
+++ b/tools/toollib.c
@@ -3372,12 +3372,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;