From 98f2e3d974bcceaf6dbac9f80f42a0eaf6512059 Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Wed, 9 May 2012 12:12:21 +0000 Subject: [PATCH] Fix regression in for_each_sub_lv pool_lv is not a sub lv in terms for this function. It has caused problem with renaming thin_volume, where it has tried to rename pool LV as well. --- WHATS_NEW | 1 + lib/metadata/lv_manip.c | 7 ------- 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 6e59f72da..5fef28e3e 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.02.96 - ================================ + Fix lvrename for thin volumes (regression in for_each_sub_lv() 2.02.89). Fix up-convert when mirror activation is controled by volume_list and tags. Disallow snapshots of mirror segment type. Fix bug in cmirror that caused incorrect status info to print on some nodes. diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index 7150fa443..f8765a67a 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -2814,13 +2814,6 @@ int for_each_sub_lv(struct cmd_context *cmd, struct logical_volume *lv, return_0; } - if (seg->pool_lv) { - if (!fn(cmd, seg->pool_lv, data)) - return_0; - if (!for_each_sub_lv(cmd, seg->pool_lv, fn, data)) - return_0; - } - if (seg->metadata_lv) { if (!fn(cmd, seg->metadata_lv, data)) return_0; -- 2.43.5