Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758281Ab3GOWWV (ORCPT ); Mon, 15 Jul 2013 18:22:21 -0400 Received: from cantor2.suse.de ([195.135.220.15]:33169 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754924Ab3GOWWT (ORCPT ); Mon, 15 Jul 2013 18:22:19 -0400 Date: Tue, 16 Jul 2013 00:22:16 +0200 (CEST) From: Jiri Kosina To: Greg KH Cc: James Bottomley , ksummit-2013-discuss@lists.linuxfoundation.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [Ksummit-2013-discuss] KS Topic request: Handling the Stable kernel, let's dump the cc: stable tag In-Reply-To: <20130715214422.GA2478@kroah.com> Message-ID: References: <1373916476.2748.69.camel@dabdike> <20130715214422.GA2478@kroah.com> User-Agent: Alpine 2.00 (LNX 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2842 Lines: 60 On Mon, 15 Jul 2013, Greg KH wrote: > > The solution, to me, looks simple: Let's co-opt a process we already > > know how to do: mailing list review and tree handling. So the proposal > > is simple: > > > > 1. Drop the cc: stable@ tag: it makes it way too easy to add an ill > > reviewed patch to stable > > 2. All patches to stable should follow current review rules: They > > should go to the mailing list the original patch was sent to > > once the original is upstream as a request for stable. > > 3. Following debate on the list, the original maintainer would be > > responsible for collecting the patches (including the upstream > > commit) adjudicating on them and passing them on to stable after > > list review (either by git tree pull or email to stable@). > > > > I contend this raises the bar for adding patches to stable much higher, > > which seems to be needed, and adds a review stage which involves all the > > original reviewers. > > I don't like this at all, just for the simple reason that it will push > the majority of the work of stable kernel development on to the > subsystem maintainers, who have enough work to do as it is. Sorry Greg, but I disagree. If the point of the stable tree really is about rock-solid stability, the "it applies without fuzz and there was no explicit NACK" just isn't enough. Someone who actually understands the code (maintainer) should absolutely give his Acked-for-stable-by: (*), otherwise the result is much less trustworthy. I think 991f76f83 might serve as a good example. It has been marked "Cc: stable", it applied without cleanly, so it has been applied to all the existing stable trees, including 3.0. The problem is that one has to actually perform a review of the patch with respect to 3.0.x codebase to notice that the pre-requisity for this patch (ef3d0fd27e) is only present in 3.2 and later, and hasn't been marked for stable (which is correct, it has no business there). (*) For me personally, the best mode of operation would actually be to have for-stable/3.x branches in my git tree, cherry-pick from other topic branches once the patches are in Linus' tree, and send you pull request for stable regularly (for each stable branch separately of course) This model would make maintainers clearly responsible for the contents of stable tree, wouldn't cause any extra work for you (quite the contrary, I'd say), and it'd follow the development model we have for Linus' tree. -- Jiri Kosina SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/