Blame SOURCES/0538-dracut-functions.sh-check_vol_slaves-speedup-LV-VG-n.patch

0903d0
From 15384bcedb8002440e1327e97cd9c139af8a32dd Mon Sep 17 00:00:00 2001
0903d0
From: Harald Hoyer <harald@redhat.com>
0903d0
Date: Mon, 15 Jan 2018 15:44:46 +0100
0903d0
Subject: [PATCH] dracut-functions.sh:check_vol_slaves() speedup LV -> VG name
0903d0
0903d0
Instead of trying all /dev/mapper/* devices to match the maj:min, and
0903d0
get the VG name with "lvm lvs", use the dm/name from /sys and dmsetup
0903d0
splitname.
0903d0
0903d0
This should speedup execution with lots of LVs.
0903d0
0903d0
Cherry-picked from: 9ed6eb741
0903d0
Resolves: #1531503
0903d0
---
0903d0
 dracut-functions.sh | 62 +++++++++++++++++++++++++----------------------------
0903d0
 1 file changed, 29 insertions(+), 33 deletions(-)
0903d0
0903d0
diff --git a/dracut-functions.sh b/dracut-functions.sh
0903d0
index ca54bd7f..53289ca0 100755
0903d0
--- a/dracut-functions.sh
0903d0
+++ b/dracut-functions.sh
0903d0
@@ -687,43 +687,39 @@ for_each_host_dev_and_slaves()
0903d0
 # but you cannot create the logical volume without the volume group.
0903d0
 # And the volume group might be bigger than the devices the LV needs.
0903d0
 check_vol_slaves() {
0903d0
-    local _lv _vg _pv
0903d0
-    for i in /dev/mapper/*; do
0903d0
-        [[ $i == /dev/mapper/control ]] && continue
0903d0
-        _lv=$(get_maj_min $i)
0903d0
-        if [[ $_lv = $2 ]]; then
0903d0
-            _vg=$(lvm lvs --noheadings -o vg_name $i 2>/dev/null)
0903d0
-            # strip space
0903d0
-            _vg="${_vg//[[:space:]]/}"
0903d0
-            if [[ $_vg ]]; then
0903d0
-                for _pv in $(lvm vgs --noheadings -o pv_name "$_vg" 2>/dev/null)
0903d0
-                do
0903d0
-                    check_block_and_slaves $1 $(get_maj_min $_pv) && return 0
0903d0
-                done
0903d0
-            fi
0903d0
-        fi
0903d0
-    done
0903d0
+    local _lv _vg _pv _dm _majmin
0903d0
+    _majmin="$2"
0903d0
+    _lv="/dev/block/$_majmin"
0903d0
+    _dm=/sys/dev/block/$_majmin/dm
0903d0
+    [[ -f $_dm/uuid  && $(<$_dm/uuid) =~ LVM-* ]] || return 1
0903d0
+    _vg=$(dmsetup splitname --noheadings -o vg_name $(<"$_dm/name") )
0903d0
+    # strip space
0903d0
+    _vg="${_vg//[[:space:]]/}"
0903d0
+    if [[ $_vg ]]; then
0903d0
+        for _pv in $(lvm vgs --noheadings -o pv_name "$_vg" 2>/dev/null)
0903d0
+        do
0903d0
+            check_block_and_slaves $1 $(get_maj_min $_pv) && return 0
0903d0
+        done
0903d0
+    fi
0903d0
     return 1
0903d0
 }
0903d0
 
0903d0
 check_vol_slaves_all() {
0903d0
-    local _lv _vg _pv
0903d0
-    for i in /dev/mapper/*; do
0903d0
-        [[ $i == /dev/mapper/control ]] && continue
0903d0
-        _lv=$(get_maj_min $i)
0903d0
-        if [[ $_lv = $2 ]]; then
0903d0
-            _vg=$(lvm lvs --noheadings -o vg_name $i 2>/dev/null)
0903d0
-            # strip space
0903d0
-            _vg="${_vg//[[:space:]]/}"
0903d0
-            if [[ $_vg ]]; then
0903d0
-                for _pv in $(lvm vgs --noheadings -o pv_name "$_vg" 2>/dev/null)
0903d0
-                do
0903d0
-                    check_block_and_slaves_all $1 $(get_maj_min $_pv)
0903d0
-                done
0903d0
-                return 0
0903d0
-            fi
0903d0
-        fi
0903d0
-    done
0903d0
+    local _lv _vg _pv _majmin
0903d0
+    _majmin="$2"
0903d0
+    _lv="/dev/block/$_majmin"
0903d0
+    _dm="/sys/dev/block/$_majmin/dm"
0903d0
+    [[ -f $_dm/uuid  && $(<$_dm/uuid) =~ LVM-* ]] || return 1
0903d0
+    _vg=$(dmsetup splitname --noheadings -o vg_name $(<"$_dm/name") )
0903d0
+    # strip space
0903d0
+    _vg="${_vg//[[:space:]]/}"
0903d0
+    if [[ $_vg ]]; then
0903d0
+        for _pv in $(lvm vgs --noheadings -o pv_name "$_vg" 2>/dev/null)
0903d0
+        do
0903d0
+            check_block_and_slaves_all $1 $(get_maj_min $_pv)
0903d0
+        done
0903d0
+        return 0
0903d0
+    fi
0903d0
     return 1
0903d0
 }
0903d0