Gitweb:
https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=3d9fc7d6f32db7fb605...
Commit: 3d9fc7d6f32db7fb605e0551afacb0fc3ccac3ea
Parent: c21440536d2a129dc4cb3194a59eb62f72e002c6
Author: Zdenek Kabelac <zkabelac(a)redhat.com>
AuthorDate: Thu Oct 31 12:38:56 2019 +0100
Committer: Zdenek Kabelac <zkabelac(a)redhat.com>
CommitterDate: Thu Oct 31 15:31:30 2019 +0100
manip: optimize lvs_using_lv
Instead of checking all LVs in a VG - do just a direct copy of LVs
from the existing list ->segs_using_thin_lv.
TODO: maybe it could be better to expose seg_list to /tools...
---
lib/metadata/mirror.c | 33 ++++++++-------------------------
1 files changed, 8 insertions(+), 25 deletions(-)
diff --git a/lib/metadata/mirror.c b/lib/metadata/mirror.c
index 73fed56..01f0246 100644
--- a/lib/metadata/mirror.c
+++ b/lib/metadata/mirror.c
@@ -1323,10 +1323,8 @@ struct dm_list *lvs_using_lv(struct cmd_context *cmd, struct
volume_group *vg,
struct logical_volume *lv)
{
struct dm_list *lvs;
- struct logical_volume *lv1;
- struct lv_list *lvl, *lvl1;
- struct lv_segment *seg;
- uint32_t s;
+ struct lv_list *lvl;
+ struct seg_list *sl;
if (!(lvs = dm_pool_alloc(cmd->mem, sizeof(*lvs)))) {
log_error("lvs list alloc failed.");
@@ -1335,29 +1333,14 @@ struct dm_list *lvs_using_lv(struct cmd_context *cmd, struct
volume_group *vg,
dm_list_init(lvs);
- /* Loop through all LVs except the one supplied */
- dm_list_iterate_items(lvl1, &vg->lvs) {
- lv1 = lvl1->lv;
- if (lv1 == lv)
- continue;
-
+ dm_list_iterate_items(sl, &lv->segs_using_this_lv) {
/* Find whether any segment points at the supplied LV */
- dm_list_iterate_items(seg, &lv1->segments) {
- for (s = 0; s < seg->area_count; s++) {
- if (seg_type(seg, s) != AREA_LV ||
- seg_lv(seg, s) != lv)
- continue;
- if (!(lvl = dm_pool_alloc(cmd->mem, sizeof(*lvl)))) {
- log_error("lv_list alloc failed.");
- return NULL;
- }
- lvl->lv = lv1;
- dm_list_add(lvs, &lvl->list);
- goto next_lv;
- }
+ if (!(lvl = dm_pool_alloc(cmd->mem, sizeof(*lvl)))) {
+ log_error("lv_list alloc failed.");
+ return NULL;
}
- next_lv:
- ;
+ lvl->lv = sl->seg->lv;
+ dm_list_add(lvs, &lvl->list);
}
return lvs;