linux-kernel - Re: [PATCH] md: suppress spurious superblock update warning for dm-raid

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <420f4ae2-ce71-4a1f-b07d-787ce490bed7@molgen.mpg.de>
Date: Tue, 10 Feb 2026 08:49:52 +0100
From: Paul Menzel <pmenzel@...gen.mpg.de>
To: chencheng@...as.com
Cc: song@...nel.org, yukuai@...as.com, linux-raid@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] md: suppress spurious superblock update warning for
 dm-raid

Dear Chen,


Thank you for your patch.

Am 10.02.26 um 13:41 schrieb chencheng:
> dm-raid have external metadata management (mddev->external = 1) and
> no persistent superblock (mddev->persistent = 0). For these arrays,
> there's no superblock to update, so the warning is spurious.

Please paste the warning.

Please also add a Fixes: tag.

> Signed-off-by: chencheng <chencheng@...as.com>

Would Chen Cheng be the official (transcribed) spelling of your name. If 
you yes, it’d be great, if you updated it:

     git config --global user.name "Chen Cheng"
     git commit --amend -s --author="Chen Cheng <chencheng@...as.com>"

> ---
>   drivers/md/md.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/md/md.c b/drivers/md/md.c
> index 6d73f6e196a..e30b658641e 100644
> --- a/drivers/md/md.c
> +++ b/drivers/md/md.c
> @@ -2790,7 +2790,9 @@ void md_update_sb(struct mddev *mddev, int force_change)
>   	if (!md_is_rdwr(mddev)) {
>   		if (force_change)
>   			set_bit(MD_SB_CHANGE_DEVS, &mddev->sb_flags);
> -		pr_err("%s: can't update sb for read-only array %s\n", __func__, mdname(mddev));
> +		if (!mddev_is_dm(mddev))
> +			pr_err_ratelimited("%s: can't update sb for read-only array %s\n",
> +					   __func__, mdname(mddev));

(It looks like it’s an error level message and not a warning level message.)

>   		return;
>   	}

With the commit message suggestions addressed, feel free to add:

Reviewed-by: Paul Menzel <pmenzel@...gen.mpg.de>


Kind regards,

Paul