Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package mdadm for openSUSE:Factory checked 
in at 2025-05-08 20:39:48
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/mdadm (Old)
 and      /work/SRC/openSUSE:Factory/.mdadm.new.30101 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "mdadm"

Thu May  8 20:39:48 2025 rev:155 rq:1275591 version:4.4

Changes:
--------
--- /work/SRC/openSUSE:Factory/mdadm/mdadm.changes      2025-04-14 
12:56:12.394767138 +0200
+++ /work/SRC/openSUSE:Factory/.mdadm.new.30101/mdadm.changes   2025-05-08 
20:39:56.168368013 +0200
@@ -1,0 +2,11 @@
+Wed May  7 15:59:57 UTC 2025 - Martin Wilck <mwi...@suse.com>
+
+- Allow any valid minor name in md device name (bsc#1240789)
+  * add 1007-mdadm-allow-any-valid-minor-number-in-md-device-name.patch
+
+-------------------------------------------------------------------
+Tue May  6 15:54:25 UTC 2025 - Martin Wilck <mwi...@suse.com>
+
+- Add dependency on suse-module-tools for SLE15 (bsc#1242696)
+
+-------------------------------------------------------------------

New:
----
  1007-mdadm-allow-any-valid-minor-number-in-md-device-name.patch

BETA DEBUG BEGIN:
  New:- Allow any valid minor name in md device name (bsc#1240789)
  * add 1007-mdadm-allow-any-valid-minor-number-in-md-device-name.patch
BETA DEBUG END:

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ mdadm.spec ++++++
--- /var/tmp/diff_new_pack.jFUska/_old  2025-05-08 20:39:56.844396230 +0200
+++ /var/tmp/diff_new_pack.jFUska/_new  2025-05-08 20:39:56.844396230 +0200
@@ -31,6 +31,9 @@
 BuildRequires:  pkgconfig(libudev)
 BuildRequires:  pkgconfig(systemd)
 BuildRequires:  pkgconfig(udev)
+%if 0%{?suse_version} < 1550
+BuildRequires:  suse-module-tools
+%endif
 PreReq:         %fillup_prereq
 PreReq:         coreutils
 URL:            http://www.kernel.org/pub/linux/utils/raid/mdadm/
@@ -49,6 +52,8 @@
 Patch1004:      1004-call-mdadm_env.sh-from-usr-libexec-mdadm.patch
 Patch1005:      1005-mdadm-enable-Intel-Alderlake-RSTe-configuration.patch
 Patch1006:      1006-imsm-Fix-RAID0-to-RAID10-migration.patch
+Patch1007:      1007-mdadm-allow-any-valid-minor-number-in-md-device-name.patch
+
 %define _udevdir %(pkg-config --variable=udevdir udev)
 %define _systemdshutdowndir %{_unitdir}/../system-shutdown
 

++++++ 1007-mdadm-allow-any-valid-minor-number-in-md-device-name.patch ++++++
>From 48319768f534e4655ef66176a95d2355a431d735 Mon Sep 17 00:00:00 2001
From: Martin Wilck <mwi...@suse.com>
Date: Wed, 30 Apr 2025 21:18:36 +0200
Subject: [PATCH 1007/1008] mdadm: allow any valid minor number in md device
 name

Since 25aa732 ("mdadm: numbered names verification"), it is not possible
any more to create arrays /dev/md${N} with N >= 127. The limit has later
been increased to 1024, which is also artificial. The error message printed
by mdadm is misleading, as the problem is not POSIX compatibility here.

  # mdadm -C -v /dev/md9999 --name=foo -l1 -n2 /dev/loop0 /dev/loop1
  mdadm: Value "/dev/md9999" cannot be set as devname. Reason: Not POSIX 
compatible.

Given that mdadm creates an array with minor number ${N} if the argument is
/dev/md${N}, the natural limit for the number is the highest minor number
available, which is (1 << MINORBITS) with MINORBITS=20 on Linux.

Fixes: 25aa732 ("mdadm: numbered names verification")
Fixes: f786072 ("mdadm: Increase number limit in md device name to 1024.")
Signed-off-by: Martin Wilck <mwi...@suse.com>
---
 util.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/util.c b/util.c
index 9fe2d22..0f77521 100644
--- a/util.c
+++ b/util.c
@@ -972,7 +972,8 @@ static bool is_devname_numbered(const char *devname, const 
char *pref, const int
        if (parse_num(&val, devname + pref_len) != 0)
                return false;
 
-       if (val > 1024)
+       /* Allow any number that represents a valid minor number */
+       if (val >= (1 << 20))
                return false;
 
        return true;
-- 
2.49.0

Reply via email to