linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@kernel.org>
To: Martin Steigerwald <martin@lichtvoll.de>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Andy Lutomirski <luto@amacapital.net>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	David Herrmann <dh.herrmann@gmail.com>,
	Djalal Harouni <tixxdz@opendz.org>,
	Greg KH <gregkh@linuxfoundation.org>,
	Havoc Pennington <havoc.pennington@gmail.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>,
	Tom Gundersen <teg@jklm.no>, Daniel Mack <daniel@zonque.org>
Subject: Re: kdbus: to merge or not to merge?
Date: Wed, 24 Jun 2015 15:18:50 +0200	[thread overview]
Message-ID: <20150624131850.GA7346@gmail.com> (raw)
In-Reply-To: <6295881.qFxhe0B0Ak@merkaba>


* Martin Steigerwald <martin@lichtvoll.de> wrote:

> Am Mittwoch, 24. Juni 2015, 10:05:02 schrieb Ingo Molnar:
>
> >  - Once one (or two) major distros go with kdbus, it becomes a de-facto ABI. 
> > If the ABI is bad then that distro will hurt from it regardless of whether we 
> > merge it upstream or not - so technical pressure is there to improve it. But 
> > if the kernel refuses to merge it, Linux users will get hurt 
> > disproportionately badly. The kernel not being the first mover with a new ABI 
> > is absolutely sensible. But once Linux distros have taken the initial 
> > (non-trivial) plunge, not merging a zero-cost ABI upstream becomes more like 
> > revenge and obstruction, which is not productive. The kernel has very little 
> > value without full user-space, after all, so within reason the kernel project 
> > has to own up to distro ABI mistakes as well.
> 
> So, in order to merge something that is not accepted upstream yet, is it an 
> accepted way to encourage distros to use it nonetheless, to get it upstream then 
> anyway as in "as, look, now this and this distro uses it"?
> 
> When I read
> 
> > Not because I like it so much, but because I think the merge process should be 
> > stripped of politics and emotion as much as possible: if an initial submission 
> > is good and addresses all technical review properly, and if the cost to the 
> > core kernel is low, then barring alternative, fully equivalent and superior 
> > patch submissions, rejecting it does more harm than good.
> 
> I think you didn´t mean it that way, as you state proper technical review as a 
> requirement.
> 
> Can you clarify?

There's no conflict: when merging something upstream, technical feedback has to be 
addressed. AFAICS that is what happened when we merged controversial bits in the 
past where Linux distros jumped the gun: such as AppArmor or Binder.

The main question that gets eliminated by a major distro using something is the 
(important) question of: 'does the Linux kernel need an ABI like that?'.

Distros still run a considerable risk when forking new ABIs, obviously - as 'pre 
release' ABIs rarely survive upstreaming, and there's no guarantee that it will be 
accepted upstream.

> Still as far as I got it, Andy raised technical concerns which Greg outrightly 
> rejected as invalid without any further explaination. That does not seem like 
> technical concerns have been properly addressed to me.

I haven't seen such responses but maybe I haven't managed to dig deep enough into 
the rather sizable discussion. Not addressing valid technical feedback would be a 
first for Greg in my book, so he definitely deserves the benefit of doubt from me.

And the thing is, in hindsight, after such huge flamewars, years down the line, 
almost never do I see the following question asked: 'what were we thinking merging 
that crap??'. If any question arises it's usually along the lines of: 'what was 
the big fuss about?'. So I think by and large the process works.

Thanks,

	Ingo

  reply	other threads:[~2015-06-24 13:19 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-23  6:06 kdbus: to merge or not to merge? Andy Lutomirski
2015-06-23  6:31 ` Andy Lutomirski
2015-06-23  6:41 ` Greg KH
2015-06-23  7:22   ` Richard Weinberger
2015-06-23  9:25     ` Martin Steigerwald
2015-06-23  9:38       ` Martin Steigerwald
2015-06-23 15:07     ` Andy Lutomirski
2015-06-25  2:14       ` Steven Rostedt
2015-06-25  2:20         ` Linus Torvalds
2015-06-25  6:01           ` Martin Steigerwald
2015-06-25  6:05             ` Martin Steigerwald
2015-06-25 13:34               ` Theodore Ts'o
2015-06-25 14:03                 ` Martin Steigerwald
2015-06-23  9:12   ` Borislav Petkov
2015-07-08 13:54   ` Pavel Machek
2015-07-09  8:39     ` Geert Uytterhoeven
2015-07-09 10:29       ` Joe Perches
2015-07-09 10:57         ` Geert Uytterhoeven
2015-07-09 11:36       ` Pavel Machek
2015-06-23 23:19 ` Linus Torvalds
2015-06-24  0:52   ` Andy Lutomirski
2015-06-24  8:05   ` Ingo Molnar
2015-06-24 10:41     ` Eric W. Biederman
2015-06-24 10:46     ` Martin Steigerwald
2015-06-24 13:18       ` Ingo Molnar [this message]
2015-06-24 17:39         ` David Lang
2015-06-24 18:41           ` Eric W. Biederman
2015-06-24 18:50           ` Martin Steigerwald
2015-06-24 19:12             ` David Lang
2015-06-25  7:57               ` Geert Uytterhoeven
2015-06-25 15:26                 ` Steven Rostedt
2015-06-25  6:31           ` Greg KH
2015-06-25  6:48             ` David Lang
2015-06-25  7:47           ` Ingo Molnar
2015-06-25  7:51             ` Ingo Molnar
2015-06-24 11:43     ` Martin Steigerwald
2015-06-24 13:27       ` Ingo Molnar
2015-06-24  9:55 ` Alexander Larsson
2015-06-24 14:38   ` Andy Lutomirski
     [not found]     ` <CAHr-LrYWNwv6_YLoP-B3duQ1QsjPiTiaEnjBQ7j2brPMeTgA3A@mail.gmail.com>
     [not found]       ` <CALCETrW3F6YP_H1oRJa47f1DT7B35OubhJYSnq0U-_GmFQHNOA@mail.gmail.com>
2015-06-24 17:11         ` Alexander Larsson
2015-06-24 19:43           ` Andy Lutomirski
2015-06-24 20:45             ` Alexander Larsson
2015-08-03 23:02 ` Andy Lutomirski
2015-08-04  8:58   ` David Herrmann
2015-08-04 13:46     ` Linus Torvalds
2015-08-04 14:09       ` David Herrmann
2015-08-04 14:47         ` Andy Lutomirski
2015-08-05  0:18           ` Andy Lutomirski
2015-08-06  7:06             ` Daniel Mack
2015-08-06 15:27               ` Andy Lutomirski
2015-08-06 17:24                 ` Daniel Mack
2015-08-05  7:10           ` David Herrmann
2015-08-05 20:11             ` Andy Lutomirski
2015-08-06  8:04               ` David Herrmann
2015-08-06  8:25                 ` Martin Steigerwald
2015-08-06 15:21                 ` Andy Lutomirski
2015-08-06 18:14                   ` Daniel Mack
2015-08-06 18:43                     ` Andy Lutomirski
2015-08-07 14:40                       ` Daniel Mack
2015-08-07 15:09                         ` Andy Lutomirski
     [not found]                         ` <CA+55aFxDLt-5+=xXeYG4nJKMb8L_iD9FmwTZ2VuughBku-mW3g@mail.gmail.com>
2015-08-09 19:00                           ` Greg Kroah-Hartman
2015-08-09 22:11                             ` Daniel Mack
2015-08-10  2:10                               ` Andy Lutomirski
2015-08-10 17:04                               ` Linus Torvalds
2015-08-10  2:48                             ` David Lang
2015-08-07 15:37                       ` cee1
  -- strict thread matches above, loose matches on Subject: below --
2015-07-01  0:03 Kalle A. Sandstrom
2015-07-01 16:51 ` David Herrmann
2015-07-06 21:18   ` Kalle A. Sandstrom

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=20150624131850.GA7346@gmail.com \
    --to=mingo@kernel.org \
    --cc=daniel@zonque.org \
    --cc=dh.herrmann@gmail.com \
    --cc=ebiederm@xmission.com \
    --cc=gnomes@lxorguk.ukuu.org.uk \
    --cc=gregkh@linuxfoundation.org \
    --cc=havoc.pennington@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luto@amacapital.net \
    --cc=martin@lichtvoll.de \
    --cc=teg@jklm.no \
    --cc=tixxdz@opendz.org \
    --cc=torvalds@linux-foundation.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;
as well as URLs for NNTP newsgroup(s).