summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Teigland <teigland@redhat.com>2019-05-03 16:12:43 -0500
committerDavid Teigland <teigland@redhat.com>2019-05-03 16:16:29 -0500
commit3405ead1e09aa6916cd5ac84ae629c13991e6ec5 (patch)
treef27b55636ced1ed37684a169881429f6695c65dc
parent6ff1583c1ba58d045fb5c8a198698e0d13f1acee (diff)
downloadlvm2-3405ead1e09aa6916cd5ac84ae629c13991e6ec5.tar.gz
pvs: remove unnecessary label scan
The scanning rework missed removing this instance of label scan. It's no longer needed because of the way that label scan is always run once from the start of the command. This unnecessary scan would be triggered by running 'pvs @tag'.
-rw-r--r--tools/reporter.c15
1 files changed, 1 insertions, 14 deletions
diff --git a/tools/reporter.c b/tools/reporter.c
index 48050ec07..db982b393 100644
--- a/tools/reporter.c
+++ b/tools/reporter.c
@@ -708,7 +708,6 @@ int report_for_selection(struct cmd_context *cmd,
static void _check_pv_list(struct cmd_context *cmd, struct report_args *args, struct single_report_args *single_args)
{
int i;
- int rescan_done = 0;
if (!args->argv)
return;
@@ -719,12 +718,6 @@ static void _check_pv_list(struct cmd_context *cmd, struct report_args *args, st
if (single_args->args_are_pvs && args->argc) {
for (i = 0; i < args->argc; i++) {
- if (!rescan_done && !dev_cache_get(cmd, args->argv[i], cmd->filter)) {
- cmd->filter->wipe(cmd->filter);
- /* FIXME scan only one device */
- lvmcache_label_scan(cmd);
- rescan_done = 1;
- }
if (*args->argv[i] == '@') {
/*
* Tags are metadata related, not label
@@ -732,13 +725,7 @@ static void _check_pv_list(struct cmd_context *cmd, struct report_args *args, st
*/
if (single_args->report_type == LABEL)
single_args->report_type = PVS;
- /*
- * If we changed the report_type and we did rescan,
- * no need to iterate over dev list further - nothing
- * else would change.
- */
- if (rescan_done)
- break;
+ break;
}
}
}