From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linuxfoundation.org ([140.211.169.12]:48876 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751134AbcLEMw3 (ORCPT ); Mon, 5 Dec 2016 07:52:29 -0500 Date: Mon, 5 Dec 2016 13:52:36 +0100 From: Greg KH To: Michal Hocko Cc: LKML , Stable tree , Andy Lutomirski , Willy Tarreau , Jiri Kosina , Michal Hocko Subject: Re: [RFC PATCH] doc: change the way how the stable backport is requested Message-ID: <20161205125236.GA19696@kroah.com> References: <20161205072154.8177-1-mhocko@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161205072154.8177-1-mhocko@kernel.org> Sender: stable-owner@vger.kernel.org List-ID: On Mon, Dec 05, 2016 at 08:21:54AM +0100, Michal Hocko wrote: > From: Michal Hocko > > Currently if a patch should aim a stable tree backport one should add > > Cc: stable@vger.kernel.org # $version > > to the s-o-b block. This has two major disadvantages a) it spams the > stable mailing list with patches which are just discussed and not merged > yet That's not a problem in that I know I like to see them to give me a "heads up" that something is coming down the pipeline soon. I don't think anyone has ever complained of this before, do you? > and b) it is easy to make a mistake and disclose a patch via > git-send-email while it is still discussed under security embargo. Having this happen only once (maybe twice) in a over a decade really isn't that bad of odds. We have loads of embargoed security patches that properly include the cc: stable tag, yet don't leak the patch to the public mailing list. So this really is a rare thing to have happen. (also when it did happen, no one except me seemed to notice it, which was pretty funny in itself...) > In fact it is not necessary to have the stable mailing list address in > the Cc until it hits the Linus tree and all we need is to have a > grepable marker for automatic identification of such a patch. Let's > use > > stable-request: $version[s] > > instead. Where $version would tell which stable trees might be > interested in the backport. This will make the process much less error > prone without any actual downsides. We still have whole subsystems that have yet to learn about how to put proper "cc: stable@..." in their patches, why do we want to change the muscle memory of those that are doing the right thing to now have to do something else? So I don't think we need this change, let's just keep things as they are. If more and more people get sloppy and mess up, we can revisit it then. thanks, greg k-h