From: Greg KH <greg@kroah.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: ksummit-discuss@lists.linuxfoundation.org
Subject: Re: [Ksummit-discuss] [CORE TOPIC] [nomination] Move Fast and Oops Things
Date: Fri, 23 May 2014 11:13:47 +0900 [thread overview]
Message-ID: <20140523021347.GB26799@kroah.com> (raw)
In-Reply-To: <CAPcyv4jxFM=gqDbEmyaXhot4OehsM6B3hsBVLcuq+ueE6i_z5A@mail.gmail.com>
On Thu, May 22, 2014 at 09:31:44AM -0700, Dan Williams wrote:
> On Thu, May 22, 2014 at 8:48 AM, Theodore Ts'o <tytso@mit.edu> wrote:
> > On Wed, May 21, 2014 at 04:03:49PM -0700, Dan Williams wrote:
> >> Simply, if an end user knows how to override a "gatekeeper" that user
> >> can test features that we are otherwise still debating upstream. They
> >> can of course also apply the patches directly, but I am proposing we
> >> formalize a mechanism to encourage more experimentation in-tree.
> >>
> >> I'm fully aware we do not have the tactical data nor operational
> >> control to run the kernel like a website, that's not my concern. My
> >> concern is with expanding a maintainer's options for mitigating risk.
> >
> > Various maintainers are doing this sort of thing already. For
> > example, file system developers stage new file system features in
> > precisely this way. Both xfs and ext4 have done this sort of thing,
> > and certainly SuSE has used this technique with btrfs to only support
> > those file system features which they are prepared to support.
> >
> > The problem is using this sort of gatekeeper is something that a
> > maintainer has to use in combination with existing techniques, and it
> > doesn't necessarliy accelerate development by all that much. In
> > particular, if it has any kind of kernel ABI or file system format
> > implications, we need to make sure the interfaces are set in stone
> > before we can let it into the mainline kernel, even if it is not
> > enabled by default. (Consider the avidity that userspace application
> > developers can sometimes have for using even debugging interfaces such
> > as ftrace, and the "no userspace breakages" rule. So not only do you
> > have to worry about userspace applicaitons not using a feature which
> > is protected by a gatekeeper, you also have to worry about premature
> > pervasive use of a feature such that you can't change the interface
> > any more.)
>
> I agree that something like this is prickly once it gets entangled
> with ABI concerns. But, I disagree with the speed argument... unless
> you believe -staging has not increased the velocity of kernel
> development?
As the maintainer of drivers/staging/ I don't think it has increased the
speed of the development of other parts of the kernel at all. Do you
have numbers that show otherwise?
> Neil already disabused me of the idea that a "gatekeeper" could be
> used to beneficial effect in the core kernel, and I can see it's
> equally difficult to use this in filesystems that need to be careful
> of ABI changes. However, nothing presented so far has swayed me from
> my top of mind concern which is the ability to ship pre-production
> driver features in the upstream kernel. I'm thinking of it as
> "-staging for otherwise established drivers".
The thing you need to realize is that the large majority of people who
would ever use that new "feature" will not until it ends up in an
"enterprise" kernel release. And that will not be for another few
years, so while you think you got it all right, we really don't know who
is using it, or how well it works, for a few years.
But feel free to try to do this in your subsystem, as Ted points out, it
can be done for somethings, but be careful about thinking things are ok
when you don't have many real users :)
thanks,
greg k-h
next prev parent reply other threads:[~2014-05-23 2:13 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-15 23:13 Dan Williams
2014-05-16 2:56 ` NeilBrown
2014-05-16 15:04 ` Chris Mason
2014-05-16 17:09 ` Andy Grover
2014-05-23 8:11 ` Dan Carpenter
2014-05-16 18:31 ` Randy Dunlap
2014-05-21 7:48 ` Dan Williams
2014-05-21 7:55 ` Greg KH
2014-05-21 9:05 ` Matt Fleming
2014-05-21 12:52 ` Greg KH
2014-05-21 13:23 ` Matt Fleming
2014-05-21 8:25 ` NeilBrown
2014-05-21 8:36 ` Dan Williams
2014-05-21 8:53 ` Matt Fleming
2014-05-21 10:11 ` NeilBrown
2014-05-21 15:35 ` Dan Williams
2014-05-21 23:06 ` Rafael J. Wysocki
2014-05-21 23:03 ` Dan Williams
2014-05-21 23:40 ` Laurent Pinchart
2014-05-22 0:10 ` Rafael J. Wysocki
2014-05-22 15:48 ` Theodore Ts'o
2014-05-22 16:31 ` Dan Williams
2014-05-22 17:38 ` Theodore Ts'o
2014-05-22 18:42 ` Dan Williams
2014-05-22 19:06 ` Chris Mason
2014-05-22 20:31 ` Dan Carpenter
2014-05-22 20:56 ` Geert Uytterhoeven
2014-05-23 6:21 ` James Bottomley
2014-05-23 14:11 ` John W. Linville
2014-05-24 9:14 ` James Bottomley
2014-05-24 19:19 ` Geert Uytterhoeven
2014-05-23 2:13 ` Greg KH [this message]
2014-05-23 3:03 ` Dan Williams
2014-05-23 7:44 ` Greg KH
2014-05-23 14:02 ` Josh Boyer
2014-05-21 23:48 ` NeilBrown
2014-05-22 4:04 ` Dan Williams
2014-05-21 7:22 ` Dan Williams
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140523021347.GB26799@kroah.com \
--to=greg@kroah.com \
--cc=dan.j.williams@intel.com \
--cc=ksummit-discuss@lists.linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox