Blame SOURCES/0052-vdo-fix-conversion-of-vdo_slab_size_mb.patch

38b7b2
From 7cb63b05dad453d015bbe462b799fb031dd6952c Mon Sep 17 00:00:00 2001
38b7b2
From: Zdenek Kabelac <zkabelac@redhat.com>
38b7b2
Date: Tue, 31 May 2022 22:48:38 +0200
38b7b2
Subject: [PATCH 52/54] vdo: fix conversion of vdo_slab_size_mb
38b7b2
38b7b2
When converting VDO volume, the parameter vdo_slabSize was
38b7b2
incorrectly copied as vdo_blockMapCacheSize, however this parameter
38b7b2
is then no longer used for any table line creation so the wrong
38b7b2
value was only stored in metadata.
38b7b2
38b7b2
Also use just single get_kb_size_with_unit_ and remove it's duplicate
38b7b2
functionality with get_mb_size_with_unit_.
38b7b2
38b7b2
Use $VERB for vdo remove call.
38b7b2
---
38b7b2
 scripts/lvm_import_vdo.sh | 13 ++-----------
38b7b2
 1 file changed, 2 insertions(+), 11 deletions(-)
38b7b2
38b7b2
diff --git a/scripts/lvm_import_vdo.sh b/scripts/lvm_import_vdo.sh
38b7b2
index 61a82e41e..beb55dbdb 100755
38b7b2
--- a/scripts/lvm_import_vdo.sh
38b7b2
+++ b/scripts/lvm_import_vdo.sh
38b7b2
@@ -125,15 +125,6 @@ get_kb_size_with_unit_() {
38b7b2
 	esac
38b7b2
 }
38b7b2
 
38b7b2
-get_mb_size_with_unit_() {
38b7b2
-	case "$1" in
38b7b2
-	*[mM]) echo $(( ${1%[mM]} )) ;;
38b7b2
-	*[gG]) echo $(( ${1%[gG]} * 1024 )) ;;
38b7b2
-	*[tT]) echo $(( ${1%[tT]} * 1024 * 1024 )) ;;
38b7b2
-	*[pP]) echo $(( ${1%[pP]} * 1024 * 1024 * 1024 )) ;;
38b7b2
-	esac
38b7b2
-}
38b7b2
-
38b7b2
 # Figure out largest possible extent size usable for VG
38b7b2
 # $1   physical size
38b7b2
 # $2   logical size
38b7b2
@@ -328,12 +319,12 @@ allocation {
38b7b2
 	vdo_use_deduplication = $(get_enabled_value_ "$vdo_deduplication")
38b7b2
 	vdo_use_metadata_hints=1
38b7b2
 	vdo_minimum_io_size = $vdo_logicalBlockSize
38b7b2
-	vdo_block_map_cache_size_mb = $(get_mb_size_with_unit_ "$vdo_blockMapCacheSize")
38b7b2
+	vdo_block_map_cache_size_mb = $(( $(get_kb_size_with_unit_ "$vdo_blockMapCacheSize") / 1024 ))
38b7b2
 	vdo_block_map_period = $vdo_blockMapPeriod
38b7b2
 	vdo_check_point_frequency = $vdo_indexCfreq
38b7b2
 	vdo_use_sparse_index = $(get_enabled_value_ "$vdo_indexSparse")
38b7b2
 	vdo_index_memory_size_mb = $(awk "BEGIN {print $vdo_indexMemory * 1024}")
38b7b2
-	vdo_slab_size_mb = $(get_mb_size_with_unit_ "$vdo_blockMapCacheSize")
38b7b2
+	vdo_slab_size_mb = $(( $(get_kb_size_with_unit_ "$vdo_blockMapCacheSize") / 1024 ))
38b7b2
 	vdo_ack_threads = $vdo_ackThreads
38b7b2
 	vdo_bio_threads = $vdo_bioThreads
38b7b2
 	vdo_bio_rotation = $vdo_bioRotationInterval
38b7b2
-- 
38b7b2
2.34.3
38b7b2