2022-08-24 17:55:23

by Steve French

[permalink] [raw]
Subject: "Fixes:" vs. "Cc: stable ..."

Do changesets that already included the "Fixes:" tag in the commit
description also need to include the "Cc: [email protected]" in
order to be included in stable?

--
Thanks,

Steve


2022-08-24 18:01:19

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: "Fixes:" vs. "Cc: stable ..."

On Wed, Aug 24, 2022 at 12:14:12PM -0500, Steve French wrote:
> Do changesets that already included the "Fixes:" tag in the commit
> description also need to include the "Cc: [email protected]" in
> order to be included in stable?

As per the documentation:
https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html

You should be putting cc: stable@... on the patch.

But as not all maintainers do, we have to dig through those with Fixes:
in order to actually catch all bugfixes :(

So please, use cc: stable.

thanks,

greg k-h

2022-08-24 18:36:33

by Steve French

[permalink] [raw]
Subject: Re: "Fixes:" vs. "Cc: stable ..."

On Wed, Aug 24, 2022 at 12:58 PM Greg KH <[email protected]> wrote:
>
> On Wed, Aug 24, 2022 at 12:14:12PM -0500, Steve French wrote:
> > Do changesets that already included the "Fixes:" tag in the commit
> > description also need to include the "Cc: [email protected]" in
> > order to be included in stable?
>
> As per the documentation:
> https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
>
> You should be putting cc: stable@... on the patch.
>
> But as not all maintainers do, we have to dig through those with Fixes:
> in order to actually catch all bugfixes :(
>
> So please, use cc: stable.

Makes sense - I had noticed a patch which I was going to include in next P/R
which was missing this (fixed now).


--
Thanks,

Steve