[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20121007201854.GA6628@redhat.com>
Date: Sun, 7 Oct 2012 16:18:54 -0400
From: Dave Jones <davej@...hat.com>
To: "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
Cc: Dave Airlie <airlied@...il.com>,
Frederic Weisbecker <fweisbec@...il.com>,
Matthew Garrett <mjg59@...f.ucam.org>,
Kees Cook <keescook@...omium.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
linux-kernel@...r.kernel.org,
"Eric W. Biederman" <ebiederm@...ssion.com>,
Serge Hallyn <serge.hallyn@...onical.com>,
"David S. Miller" <davem@...emloft.net>,
Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] make CONFIG_EXPERIMENTAL invisible and default
On Sun, Oct 07, 2012 at 09:30:29AM -0700, Paul E. McKenney wrote:
> > I think Kconfig is mostly what distro would like to use the thing is
> > the Kconfig text needs to be there upfront when its merged, not two
> > months later, since then it too late for a distro to notice.
> >
> > I'd bet most distros would read the warnings, but in a lot of cases
> > the warning don't exist until its too late.
>
> In the case of CONFIG_RCU_USER_QS you are quite right, the warning
> should have been there from the beginning and was not. I suppose you
> could argue that the warning was not sufficiently harsh in the case of
> CONFIG_RCU_FAST_NO_HZ, but either way it did get ignored:
Maybe if we had a universally agreed upon tag for kconfig, like
"distro recommendation: N" that would make things obvious, and also allow
those of us unfortunate enough to maintain distro kernels to have something
to easily grep for. This would also catch the case when you eventually (hopefully)
flip from an N to a Y.
There will likely still be some distros that will decide they know better
(and I'm pretty sure eventually I'll find reason to do so myself), but it at least
gives the feature maintainer the "I told you so" clause.
Something we do quite often for our in-development kernels is enable something
that's shiny, new and unproven, and then when we branch for a release, we turn
it back off. It would be great if a lot of this kind of thing could be more automated.
Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists