* [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules [not found] ` <alpine.LRH.2.02.1309191359300.12162@file01.intranet.prod.int.rdu2.redhat.com> @ 2013-09-19 18:31 ` Joe Perches 2013-09-19 21:32 ` Francois Romieu ` (2 more replies) 0 siblings, 3 replies; 9+ messages in thread From: Joe Perches @ 2013-09-19 18:31 UTC (permalink / raw) To: David Miller Cc: stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML Networking is once again "special", so at least document how it's working today in the hope that doing so makes less work for all that actually read the documentation. Signed-off-by: Joe Perches <joe@perches.com> --- On Thu, 19 Sep 2013, David Miller wrote: > Secondly, CC:'ing stable is not the correct way to submit networking > patches for -stable inclusion. You simply ask me to queue them up > for -stable explicitly instead. Documentation/stable_kernel_rules.txt | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/Documentation/stable_kernel_rules.txt b/Documentation/stable_kernel_rules.txt index b0714d8..a2d6da0 100644 --- a/Documentation/stable_kernel_rules.txt +++ b/Documentation/stable_kernel_rules.txt @@ -29,6 +29,11 @@ Rules on what kind of patches are accepted, and which ones are not, into the Procedure for submitting patches to the -stable tree: + - The networking tree (net/ and drivers/net/) is 'special' and doesn't + follow the rules below. Don't send or cc: patches for the -stable tree to + stable@vger.kernel.org. Don't mark them stable. Just send the patches to + netdev@vger.kernel.org and let the networking maintainer decide what to do + with them. - Send the patch, after verifying that it follows the above rules, to stable@vger.kernel.org. You must note the upstream commit ID in the changelog of your submission, as well as the kernel version you wish ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-19 18:31 ` [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules Joe Perches @ 2013-09-19 21:32 ` Francois Romieu 2013-09-19 21:45 ` Joe Perches 2013-09-20 14:54 ` Joe Perches 2013-09-22 18:51 ` Christoph Hellwig 2 siblings, 1 reply; 9+ messages in thread From: Francois Romieu @ 2013-09-19 21:32 UTC (permalink / raw) To: Joe Perches Cc: David Miller, stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML Joe Perches <joe@perches.com> : [...] > diff --git a/Documentation/stable_kernel_rules.txt b/Documentation/stable_kernel_rules.txt > index b0714d8..a2d6da0 100644 > --- a/Documentation/stable_kernel_rules.txt > +++ b/Documentation/stable_kernel_rules.txt > @@ -29,6 +29,11 @@ Rules on what kind of patches are accepted, and which ones are not, into the > > Procedure for submitting patches to the -stable tree: > > + - The networking tree (net/ and drivers/net/) is 'special' and doesn't > + follow the rules below. Don't send or cc: patches for the -stable tree to > + stable@vger.kernel.org. Don't mark them stable. Just send the patches to > + netdev@vger.kernel.org and let the networking maintainer decide what to do > + with them. David said "simply ask me to queue them up for -stable explicitly". He did not say "send the patches and let me decide what to do with them". -- Ueimor ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-19 21:32 ` Francois Romieu @ 2013-09-19 21:45 ` Joe Perches 2013-09-19 22:37 ` Francois Romieu 0 siblings, 1 reply; 9+ messages in thread From: Joe Perches @ 2013-09-19 21:45 UTC (permalink / raw) To: Francois Romieu Cc: David Miller, stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML On Thu, 2013-09-19 at 23:32 +0200, Francois Romieu wrote: > Joe Perches <joe@perches.com> : > [...] > > diff --git a/Documentation/stable_kernel_rules.txt b/Documentation/stable_kernel_rules.txt > > index b0714d8..a2d6da0 100644 > > --- a/Documentation/stable_kernel_rules.txt > > +++ b/Documentation/stable_kernel_rules.txt > > @@ -29,6 +29,11 @@ Rules on what kind of patches are accepted, and which ones are not, into the > > > > Procedure for submitting patches to the -stable tree: > > > > + - The networking tree (net/ and drivers/net/) is 'special' and doesn't > > + follow the rules below. Don't send or cc: patches for the -stable tree to > > + stable@vger.kernel.org. Don't mark them stable. Just send the patches to > > + netdev@vger.kernel.org and let the networking maintainer decide what to do > > + with them. > > David said "simply ask me to queue them up for -stable explicitly". > > He did not say "send the patches and let me decide what to do with them". > David selects them regardless. from Documentation/networking/netdev-FAQ.txt: Q: How can I tell what patches are queued up for backporting to the various stable releases? A: Normally Greg Kroah-Hartman collects stable commits himself, but for networking, Dave collects up patches he deems critical for the networking subsystem, and then hands them off to Greg. There is a patchworks queue that you can see here: http://patchwork.ozlabs.org/bundle/davem/stable/?state=* It contains the patches which Dave has selected, but not yet handed off to Greg. If Greg already has the patch, then it will be here: http://git.kernel.org/cgit/linux/kernel/git/stable/stable-queue.git A quick way to find whether the patch is in this stable-queue is to simply clone the repo, and then git grep the mainline commit ID, e.g. stable-queue$ git grep -l 284041ef21fdf2e releases/3.0.84/ipv6-fix-possible-crashes-in-ip6_cork_release.patch releases/3.4.51/ipv6-fix-possible-crashes-in-ip6_cork_release.patch releases/3.9.8/ipv6-fix-possible-crashes-in-ip6_cork_release.patch stable/stable-queue$ Q: I see a network patch and I think it should be backported to stable. Should I request it via "stable@vger.kernel.org" like the references in the kernel's Documentation/stable_kernel_rules.txt file say? A: No, not for networking. Check the stable queues as per above 1st to see if it is already queued. If not, then send a mail to netdev, listing the upstream commit ID and why you think it should be a stable candidate. Before you jump to go do the above, do note that the normal stable rules in Documentation/stable_kernel_rules.txt still apply. So you need to explicitly indicate why it is a critical fix and exactly what users are impacted. In addition, you need to convince yourself that you _really_ think it has been overlooked, vs. having been considered and rejected. Generally speaking, the longer it has had a chance to "soak" in mainline, the better the odds that it is an OK candidate for stable. So scrambling to request a commit be added the day after it appears should be avoided. Q: I have created a network patch and I think it should be backported to stable. Should I add a "Cc: stable@vger.kernel.org" like the references in the kernel's Documentation/ directory say? A: No. See above answer. In short, if you think it really belongs in stable, then ensure you write a decent commit log that describes who gets impacted by the bugfix and how it manifests itself, and when the bug was introduced. If you do that properly, then the commit will get handled appropriately and most likely get put in the patchworks stable queue if it really warrants it. If you think there is some valid information relating to it being in stable that does _not_ belong in the commit log, then use the three dash marker line as described in Documentation/SubmittingPatches to temporarily embed that information into the patch that you send. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-19 21:45 ` Joe Perches @ 2013-09-19 22:37 ` Francois Romieu 0 siblings, 0 replies; 9+ messages in thread From: Francois Romieu @ 2013-09-19 22:37 UTC (permalink / raw) To: Joe Perches Cc: David Miller, stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML Joe Perches <joe@perches.com> : [...] > David selects them regardless. > > from Documentation/networking/netdev-FAQ.txt: I don't believe that those who read Documentation/stable_kernel_rules.txt will magically read networking/netdev-FAQ.txt as well nor figure that while they should not mark the patches stables (skr.txt), they are expected to apply some extra rules as well (nF.txt). It's fine if you disagree. I won't argue. -- Ueimor ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-19 18:31 ` [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules Joe Perches 2013-09-19 21:32 ` Francois Romieu @ 2013-09-20 14:54 ` Joe Perches 2013-09-20 15:59 ` David Miller 2013-09-22 18:51 ` Christoph Hellwig 2 siblings, 1 reply; 9+ messages in thread From: Joe Perches @ 2013-09-20 14:54 UTC (permalink / raw) To: David Miller Cc: stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML On Thu, 2013-09-19 at 11:31 -0700, Joe Perches wrote: > Networking is once again "special", so at least document > how it's working today in the hope that doing so makes > less work for all that actually read the documentation. David, why did you mark this N/A is patchwork? It's your rule, why not document it? ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-20 14:54 ` Joe Perches @ 2013-09-20 15:59 ` David Miller 0 siblings, 0 replies; 9+ messages in thread From: David Miller @ 2013-09-20 15:59 UTC (permalink / raw) To: joe; +Cc: stephen, netdev, mpatocka, gregkh, rob, linux-doc, linux-kernel From: Joe Perches <joe@perches.com> Date: Fri, 20 Sep 2013 07:54:40 -0700 > On Thu, 2013-09-19 at 11:31 -0700, Joe Perches wrote: >> Networking is once again "special", so at least document >> how it's working today in the hope that doing so makes >> less work for all that actually read the documentation. > > David, why did you mark this N/A is patchwork? > It's your rule, why not document it? Because it should go through whoever maintains that document, and that isn't me. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-19 18:31 ` [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules Joe Perches 2013-09-19 21:32 ` Francois Romieu 2013-09-20 14:54 ` Joe Perches @ 2013-09-22 18:51 ` Christoph Hellwig 2013-09-23 20:34 ` Joe Perches 2 siblings, 1 reply; 9+ messages in thread From: Christoph Hellwig @ 2013-09-22 18:51 UTC (permalink / raw) To: Joe Perches Cc: David Miller, stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML, xfs This is also the preferred way to do it for XFS. Maybe word it in a way that we can easily add subsystems. To me it generally seems to be the best way to do it - having random Ccs and lots of stable trees doesn't seem like a very good way of handling it. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-22 18:51 ` Christoph Hellwig @ 2013-09-23 20:34 ` Joe Perches 2013-09-24 8:48 ` Christoph Hellwig 0 siblings, 1 reply; 9+ messages in thread From: Joe Perches @ 2013-09-23 20:34 UTC (permalink / raw) To: Christoph Hellwig Cc: David Miller, stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML, xfs On Sun, 2013-09-22 at 11:51 -0700, Christoph Hellwig wrote: > This is also the preferred way to do it for XFS. Maybe word it in a way > that we can easily add subsystems. > > To me it generally seems to be the best way to do it - having random Ccs > and lots of stable trees doesn't seem like a very good way of handling > it. Maybe adding a mechanism to MAINTAINERS would be better. Maybe a default B: (backport?) of stable@vger.kernel.org with a per-subsystem override? SUBSYSTEM TYPE M: maintainer@email.address L: list@email.address S: Supported F: file/pattern/ B: stable@email.address ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules 2013-09-23 20:34 ` Joe Perches @ 2013-09-24 8:48 ` Christoph Hellwig 0 siblings, 0 replies; 9+ messages in thread From: Christoph Hellwig @ 2013-09-24 8:48 UTC (permalink / raw) To: Joe Perches Cc: Christoph Hellwig, David Miller, stephen, netdev, Mikulas Patocka, Greg Kroah-Hartman, Rob Landley, linux-doc, LKML, xfs On Mon, Sep 23, 2013 at 01:34:05PM -0700, Joe Perches wrote: > Maybe adding a mechanism to MAINTAINERS would be better. > Maybe a default B: (backport?) of stable@vger.kernel.org > with a per-subsystem override? Sounds fine to me. ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2013-09-24 8:48 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <alpine.LRH.2.02.1309191229480.32035@file01.intranet.prod.int.rdu2.redhat.com>
[not found] ` <20130919.135628.1201613770803318193.davem@davemloft.net>
[not found] ` <alpine.LRH.2.02.1309191359300.12162@file01.intranet.prod.int.rdu2.redhat.com>
2013-09-19 18:31 ` [PATCH] stable_kernel_rules.txt: Exclude networking from stable rules Joe Perches
2013-09-19 21:32 ` Francois Romieu
2013-09-19 21:45 ` Joe Perches
2013-09-19 22:37 ` Francois Romieu
2013-09-20 14:54 ` Joe Perches
2013-09-20 15:59 ` David Miller
2013-09-22 18:51 ` Christoph Hellwig
2013-09-23 20:34 ` Joe Perches
2013-09-24 8:48 ` Christoph Hellwig
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox