stable.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Yu Kuai <yukuai1@huaweicloud.com>
Cc: stable@vger.kernel.org, stable-commits@vger.kernel.org,
	xni@redhat.com, Song Liu <song@kernel.org>,
	"yukuai (C)" <yukuai3@huawei.com>
Subject: Re: Patch "md: call del_gendisk in control path" has been added to the 6.6-stable tree
Date: Mon, 18 Aug 2025 11:39:32 +0200	[thread overview]
Message-ID: <2025081812-stalemate-hug-a179@gregkh> (raw)
In-Reply-To: <e0fef2d7-9c72-495e-4c62-7c4fd766c84d@huaweicloud.com>

On Mon, Aug 18, 2025 at 03:13:11PM +0800, Yu Kuai wrote:
> Hi,
> 
> 在 2025/08/18 14:38, Greg KH 写道:
> > On Mon, Aug 18, 2025 at 02:26:23PM +0800, Yu Kuai wrote:
> > > Hi,
> > > 
> > > 在 2025/08/18 13:55, Greg KH 写道:
> > > > On Mon, Aug 18, 2025 at 09:03:39AM +0800, Yu Kuai wrote:
> > > > > Hi,
> > > > > 
> > > > > 在 2025/08/17 22:18, Sasha Levin 写道:
> > > > > > This is a note to let you know that I've just added the patch titled
> > > > > > 
> > > > > >        md: call del_gendisk in control path
> > > > > > 
> > > > > > to the 6.6-stable tree which can be found at:
> > > > > >        http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > > > > > 
> > > > > > The filename of the patch is:
> > > > > >         md-call-del_gendisk-in-control-path.patch
> > > > > > and it can be found in the queue-6.6 subdirectory.
> > > > > > 
> > > > > > If you, or anyone else, feels it should not be added to the stable tree,
> > > > > > please let <stable@vger.kernel.org> know about it.
> > > > > > 
> > > > > > 
> > > > > This patch should be be backported to any stable kernel, this change
> > > > > will break user tools mdadm:
> > > > > 
> > > > > https://lore.kernel.org/all/f654db67-a5a5-114b-09b8-00db303daab7@redhat.com/
> > > > 
> > > > Is it reverted in Linus's tree?
> > > > 
> > > 
> > > No, we'll not revert it, this is an improvement. In order to keep user
> > > tools compatibility, we added a switch in the kernel. As discussed in
> > > the thread, for old tools + new kernel, functionality is the same,
> > > however, there will be kernel warning about deprecated behaviour to
> > > inform user upgrading user tools.
> > > 
> > > However, I feel this new warning messages is not acceptable for
> > > stable kernels.
> > 
> > Why?  What is so special about stable kernels that taking the same
> > functionality in newer kernels is not ok?
> > 
> > Why not just "warn" the same here if you want to fix an issue where
> > userspace should be also updating some tools.  As long as you aren't
> > breaking anything, it should be fine, right?
> 
> Yes, it's fine, just in downstream kernels, people won't be happy about
> new warnings.

People are NEVER happy about new warnings.  So why are you warning them
at all in newer kernels?

> > Or are you breaking existing workflows?  You should be able to take a
> > new kernel without any userspace changes and all should work the same.
> > Why make new kernel users change userspace tools at all?
> 
> There is a pending fix in mdraid that will be pushed soon, with this
> nothing will be broken.

Great, what is the git id?

> And because user tools have problems in this case as well, both kernel
> and user tools have to be fixed to make things better.

So Linus's tree right now doesn't work without the pending fix as well?

thanks,

greg k-h

  reply	other threads:[~2025-08-18  9:39 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20250817141818.2370452-1-sashal@kernel.org>
2025-08-18  1:03 ` Patch "md: call del_gendisk in control path" has been added to the 6.6-stable tree Yu Kuai
2025-08-18  5:55   ` Greg KH
2025-08-18  6:26     ` Yu Kuai
2025-08-18  6:38       ` Greg KH
2025-08-18  7:13         ` Yu Kuai
2025-08-18  9:39           ` Greg KH [this message]
2025-08-19  1:06             ` Yu Kuai
2025-08-19  6:12               ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2025081812-stalemate-hug-a179@gregkh \
    --to=gregkh@linuxfoundation.org \
    --cc=song@kernel.org \
    --cc=stable-commits@vger.kernel.org \
    --cc=stable@vger.kernel.org \
    --cc=xni@redhat.com \
    --cc=yukuai1@huaweicloud.com \
    --cc=yukuai3@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).