Gitweb:
https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=19f2105b87773047854...
Commit: 19f2105b87773047854ef172de8fc48da110b2fa
Parent: 420af27f088a3808816ba3ab5b47dfa400fef308
Author: Bryn M. Reeves <bmr(a)redhat.com>
AuthorDate: Thu Nov 1 16:47:56 2018 +0000
Committer: Bryn M. Reeves <bmr(a)redhat.com>
CommitterDate: Thu Nov 1 16:59:56 2018 +0000
libdm-stats: move no regions warning after dm_stats_list()
It doesn't make sense to test or warn about the region count until
the stats handle has been listed: at this point it may or may not
contain valid information (but is guaranteed to be correct after
the list).
---
libdm/libdm-stats.c | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/libdm/libdm-stats.c b/libdm/libdm-stats.c
index 0143957..ca46d9a 100644
--- a/libdm/libdm-stats.c
+++ b/libdm/libdm-stats.c
@@ -2338,11 +2338,6 @@ int dm_stats_populate(struct dm_stats *dms, const char
*program_id,
return 0;
}
- if (!dms->nr_regions) {
- log_error("No regions registered.");
- return 0;
- }
-
/* allow zero-length program_id for populate */
if (!program_id)
program_id = dms->program_id;
@@ -2354,6 +2349,11 @@ int dm_stats_populate(struct dm_stats *dms, const char
*program_id,
goto_bad;
}
+ if (!dms->nr_regions) {
+ log_verbose("No stats regions registered: %s", dms->name);
+ return 0;
+ }
+
dms->walk_flags = DM_STATS_WALK_REGION;
dm_stats_walk_start(dms);
do {