feature/mdadm: workaround for the error in determining the RAID device in use

Sometimes on Fedora, bash may end up with the following error:

  /srv/features/mdadm/guess/device: line 46: /sys/devices/virtual/block/md127/uevent: Success

In this line, the file is sourced from sysfs:

  MAJOR=
  MINOR=
  . "$SYSFS_PATH$1"/uevent

As a workaround, we try to read the file in other ways. If that doesn't
work, we make the rules for all raid devices.

Signed-off-by: Alexey Gladkov <[email protected]>

Author: legionus

features/mdadm/bin/generate-udev-rules

@@ -1,4 +1,4 @@
-#!/bin/bash -efu
+#!/bin/bash -eu
 # SPDX-License-Identifier: GPL-3.0-or-later
 
 . sh-functions
@@ -9,6 +9,20 @@ if [ "${1-}" = dirs ]; then
 	mkdir ${verbose2-} -p -- "$DIR/etc/initrd/md"
 fi
 
+if [ -z "${GENERATE_UDEV_RULES_FOR_MD_DEVICE-}" ]; then
+	GENERATE_UDEV_RULES_FOR_MD_DEVICE=
+
+	# If the variable is not specified, we generate rules for all
+	# raid devices.
+	for path in /dev/dev/block/*/md; do
+		if [ -d "$path" ]; then
+			majmin="${path%/md}"
+			majmin="${majmin##*/}"
+			GENERATE_UDEV_RULES_FOR_MD_DEVICE="$GENERATE_UDEV_RULES_FOR_MD_DEVICE $majmin"
+		fi
+	done
+fi
+
 for majmin in ${GENERATE_UDEV_RULES_FOR_MD_DEVICE-}; do
 	verbose2 "Processing MD device $majmin..."
 

features/mdadm/guess/device

@@ -41,8 +41,18 @@ fi
 
 guess_feature mdadm
 
-MAJOR=
-MINOR=
-. "$SYSFS_PATH$1"/uevent
-
-guess_variable GENERATE_UDEV_RULES_FOR_MD_DEVICE "$MAJOR:$MINOR"
+if [ -f "$SYSFS_PATH$1"/uevent ]; then
+	MAJOR=
+	MINOR=
+
+	# Sometimes on Fedora, bash may end the following command with an error:
+	# /srv/features/mdadm/guess/device: line 46: /sys/devices/virtual/block/md127/uevent: Success
+	#
+	. "$SYSFS_PATH$1"/uevent || {
+		MAJOR="$(sed -n -e 's,MAJOR=,,p' "$SYSFS_PATH$1"/uevent)"
+		MINOR="$(sed -n -e 's,MINOR=,,p' "$SYSFS_PATH$1"/uevent)"
+	} ||:
+
+	[ -z "$MAJOR" ] || [ -z "$MINOR" ] ||
+		guess_variable GENERATE_UDEV_RULES_FOR_MD_DEVICE "$MAJOR:$MINOR"
+fi