ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: "Luis R. Rodriguez" <mcgrof@kernel.org>
To: Kees Cook <keescook@chromium.org>
Cc: James Bottomley <James.Bottomley@hansenpartnership.com>,
	ksummit <ksummit-discuss@lists.linuxfoundation.org>
Subject: Re: [Ksummit-discuss] [MAINTAINERS SUMMIT] Developing across multiple areas of the kernel
Date: Thu, 29 Jun 2017 20:20:44 +0200	[thread overview]
Message-ID: <20170629182044.GP21846@wotan.suse.de> (raw)
In-Reply-To: <CAGXu5jLL8_h3AYrpCm4LKV8-rTPf793gAJckS0rQag6iGqk9xw@mail.gmail.com>

On Thu, Jun 29, 2017 at 10:52:51AM -0700, Kees Cook wrote:
> On Thu, Jun 29, 2017 at 10:42 AM, James Bottomley
> <James.Bottomley@hansenpartnership.com> wrote:
> > On Thu, 2017-06-29 at 09:51 -0700, Kees Cook wrote:
> >> On Thu, Jun 29, 2017 at 9:36 AM, James Bottomley
> >> <James.Bottomley@hansenpartnership.com> wrote:
> >> > On Wed, 2017-06-28 at 16:01 -0700, Kees Cook wrote:
> >> Right, I've got no objection to the performance concerns and how that
> >> played out, but it's API-to-conversion steps that seem inefficient.
> >
> > Well don't discount tree merge problems, having seen a few caused by
> > API plus conversion all at once.  However, by putting them through the
> > maintainer trees you got the review that would otherwise have been
> > missing which highlighted the performance concerns.  Even this time
> > around the affected trees have a whole merge window to run performance
> > regressions to verify everything is OK.  Based on this I think the rule
> > should be API in release R - 1 and conversion in release R through the
> > affected trees with the only exception being changes that are trivial
> > enough (for some value of trivial).

Do you mean add API with no users? If so perhaps test drivers can be the
first users for it then. Then they are not naked APIs.

> I'd argue that's what -next is for, but we lack a way to have people
> base their trees on -next sanely.

Linus has said before cross-tree collateral evolutions *could* just be sent to
him as a set of scripts he could run. It might sound easier said than done
though, but we can improve on the process by practicing it daily.

We could just mimic this on linux-next as either a last step as part of its
scripts and a new tag issued, or this could be done on a separate tree.

Its perhaps easier to think about it first as a separate tree, let's call it a
linux-oven [0] -- it would reset itself daily based on linux-next's latest tag,
and as a secondary step just run the tooling to do the API conversions.

An agreed upon set of tools can be supported for the API conversion tasks.  The
rules for what goes in and how would need to be decided. The above rules James
proposed for instance seem to make sense to me, modulo the test driver on R-1
as well.

Just a thought of course.

[0] https://kernelnewbies.org/KernelProjects/linux-oven

  Luis

  reply	other threads:[~2017-06-29 18:20 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-28 23:01 Kees Cook
2017-06-29 13:39 ` Christoph Hellwig
2017-06-30 13:02   ` Daniel Vetter
2017-06-29 16:36 ` James Bottomley
2017-06-29 16:51   ` Kees Cook
2017-06-29 17:42     ` James Bottomley
2017-06-29 17:52       ` Kees Cook
2017-06-29 18:20         ` Luis R. Rodriguez [this message]
2017-06-29 19:07           ` Linus Torvalds
2017-06-29 20:16           ` Kees Cook
2017-06-29 20:27             ` Stephen Rothwell
2017-07-14  4:04               ` Leon Romanovsky
2017-07-14  9:54                 ` Greg KH
2017-07-14 10:29                   ` Leon Romanovsky
2017-07-14 14:10                     ` Andrew Lunn
2017-07-14 15:05                       ` Mark Brown
2017-07-14 15:51                         ` Leon Romanovsky
2017-07-14 16:20                           ` Mark Brown
2017-07-14 15:35                       ` Leon Romanovsky
2017-07-14 15:43                         ` James Bottomley
2017-07-14 16:08                           ` Leon Romanovsky
2017-07-14 16:18                         ` Andrew Lunn
2017-07-14 16:28                           ` Bart Van Assche

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=20170629182044.GP21846@wotan.suse.de \
    --to=mcgrof@kernel.org \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=keescook@chromium.org \
    --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