summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZdenek Kabelac <zkabelac@redhat.com>2014-10-26 19:45:17 +0100
committerZdenek Kabelac <zkabelac@redhat.com>2014-10-26 19:45:17 +0100
commite223c801fc3eb12c2a4502aee471f18a0b903de8 (patch)
treee4113cd7b2dbfaf300cf5a47b3c25af5dc3dd31c
parentc28a7706ed00c135e6fb06d22dde1c374f32f847 (diff)
downloadlvm2-e223c801fc3eb12c2a4502aee471f18a0b903de8.tar.gz
pools: workaround hints
Missing code for stacked hint estimation
-rw-r--r--lib/metadata/pool_manip.c29
-rw-r--r--test/shell/lvconvert-cache-raid.sh1
2 files changed, 20 insertions, 10 deletions
diff --git a/lib/metadata/pool_manip.c b/lib/metadata/pool_manip.c
index b84950cb3..d1b449713 100644
--- a/lib/metadata/pool_manip.c
+++ b/lib/metadata/pool_manip.c
@@ -324,16 +324,25 @@ int recalculate_pool_chunk_size_with_dev_hints(struct logical_volume *pool_lv,
pool_data_lv = seg_lv(first_seg(pool_lv), 0);
dm_list_iterate_items(seg, &pool_data_lv->segments) {
- pv = seg_pv(seg, 0);
- if (chunk_size_calc_policy == THIN_CHUNK_SIZE_CALC_METHOD_PERFORMANCE)
- hint = dev_optimal_io_size(cmd->dev_types, pv_dev(pv));
- else
- hint = dev_minimum_io_size(cmd->dev_types, pv_dev(pv));
- if (!hint)
- continue;
- if (previous_hint)
- hint = _lcm(previous_hint, hint);
- previous_hint = hint;
+ switch (seg_type(seg, 0)) {
+ case AREA_PV:
+ pv = seg_pv(seg, 0);
+ if (chunk_size_calc_policy == THIN_CHUNK_SIZE_CALC_METHOD_PERFORMANCE)
+ hint = dev_optimal_io_size(cmd->dev_types, pv_dev(pv));
+ else
+ hint = dev_minimum_io_size(cmd->dev_types, pv_dev(pv));
+ if (!hint)
+ continue;
+
+ if (previous_hint)
+ hint = _lcm(previous_hint, hint);
+ previous_hint = hint;
+ break;
+ case AREA_LV:
+ /* FIXME: hint for stacked (raid) LVs - estimate geometry from LV ?? */
+ default:
+ break;
+ }
}
if (!hint)
diff --git a/test/shell/lvconvert-cache-raid.sh b/test/shell/lvconvert-cache-raid.sh
index 3ac2d0b81..16dd38e28 100644
--- a/test/shell/lvconvert-cache-raid.sh
+++ b/test/shell/lvconvert-cache-raid.sh
@@ -46,6 +46,7 @@ lvremove -f $vg
lvcreate -n cpool_meta -m 1 --type raid1 -l 10 $vg
lvcreate -n cpool -m 1 --type raid1 -l 10 $vg
+lvs -a -o+seg_pe_ranges $vg
lvconvert --yes --type cache-pool --poolmetadata $vg/cpool_meta $vg/cpool
lvcreate -n corigin --type cache --cachepool $vg/cpool -l 10