All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Greg KH <greg@kroah.com>
Cc: Peter Zijlstra <peterz@infradead.org>,
	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
	devel@driverdev.osuosl.org, lttng-dev@lists.lttng.org,
	linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 09/11] sched: export task_prio to GPL modules
Date: Mon, 5 Dec 2011 15:17:49 +0100	[thread overview]
Message-ID: <20111205141749.GC28866@elte.hu> (raw)
In-Reply-To: <20111201231751.GA4961@kroah.com>


* Greg KH <greg@kroah.com> wrote:

> On Fri, Dec 02, 2011 at 12:07:10AM +0100, Peter Zijlstra wrote:
> > On Thu, 2011-12-01 at 14:14 -0800, Greg KH wrote:
> > > greg k-h
> > 
> > Greg, why are you merging this crap anyway? Aren't there enough tracer
> > thingies around already?
> 
> I don't know, is there?
> 
> There's some reason the distros, and users, still use lttng, 
> so I'm guessing that it fits the needs of quite a few people.

Same goes for a whole lot of other crap that distros are 
carrying. Would we want to merge a different CPU scheduler or 
the 4g:4g patch or a completely new networking stack into 
drivers/staging/? I don't think so.

I.e. putting LTTNG into drivers/staging/ will not really solve 
anything - and in may in fact delay any sane technical 
resolution:

There's a difference between a driver that has to go into 
drivers/staging/ because nobody cares enough [and the driver 
isnt high quality enough yet], and a core kernel feature that we 
DO care about and which HAS BEEN REJECTED IN ITS FORM.

> That's why I'm merging it, if that the in-kernel stuff 
> obsoletes lttng, great, let me, and the distros know.

I'm NAK-ing the LTTNG driver really, as it's a workaround for a 
core kernel NAK.

Mathieu, please work with the tracing folks who DO care about 
this stuff. It's not like there's a lack of interest in this 
area, nor is there a lack of willingness to take patches. What 
there is a lack of is your willingness to actually work on 
getting something unified, integrated to users...

LTTNG has been going on for how many years? I havent seen many 
steps towards actually *merging* its functionality - you insist 
on doing your own random thing, which is different in random 
ways. Yes, some of those random ways may in fact be better than 
what we have upstream - would you be interested in filtering 
those out and pushing them upstream? I certainly would like to 
see that happen.

We want to pick the best features, and throw away current 
upstream code in favor of superior out of tree code - this 
concept of letting crap sit alongside each other when people do 
care i cannot agree with.

Thanks,

	Ingo

  reply	other threads:[~2011-12-05 14:19 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1322775683-8741-1-git-send-email-mathieu.desnoyers@efficios.com>
2011-12-01 21:41 ` [PATCH 01/11] mm: export vmalloc_sync_all symbol to GPL modules Mathieu Desnoyers
2011-12-01 21:41   ` Mathieu Desnoyers
2011-12-01 21:57   ` Christoph Hellwig
2011-12-01 21:57   ` Christoph Hellwig
2011-12-01 21:57   ` Christoph Hellwig
2011-12-01 21:57   ` Christoph Hellwig
2011-12-01 21:57     ` Christoph Hellwig
2011-12-01 22:13     ` Greg KH
2011-12-01 22:13       ` Greg KH
2011-12-01 22:19       ` Mathieu Desnoyers
2011-12-01 22:19         ` Mathieu Desnoyers
2011-12-01 22:41         ` Greg KH
2011-12-01 22:41           ` Greg KH
2011-12-01 22:28       ` Christoph Hellwig
2011-12-01 22:28         ` Christoph Hellwig
2011-12-01 23:00         ` Greg KH
2011-12-01 23:00           ` Greg KH
2011-12-01 21:41 ` [PATCH 03/11] fs/splice: export splice_to_pipe " Mathieu Desnoyers
2011-12-02  7:19   ` Jens Axboe
2011-12-02 12:32     ` Mathieu Desnoyers
2011-12-01 21:41 ` [PATCH 09/11] sched: export task_prio " Mathieu Desnoyers
2011-12-01 21:56   ` Peter Zijlstra
2011-12-01 22:04     ` Mathieu Desnoyers
2011-12-01 22:10       ` Peter Zijlstra
2011-12-01 22:15         ` Mathieu Desnoyers
2011-12-01 22:36           ` Mathieu Desnoyers
2011-12-01 23:05             ` Peter Zijlstra
2011-12-02 13:51               ` Mathieu Desnoyers
2011-12-01 23:06           ` Peter Zijlstra
2011-12-01 23:18             ` Greg KH
2011-12-01 23:47               ` Mathieu Desnoyers
2011-12-01 22:14     ` Greg KH
2011-12-01 22:20       ` Mathieu Desnoyers
2011-12-01 23:07       ` Peter Zijlstra
2011-12-01 23:17         ` Greg KH
2011-12-05 14:17           ` Ingo Molnar [this message]
2011-12-06 21:44             ` Greg KH
2011-12-08  5:23               ` Ingo Molnar
2011-12-08 23:27                 ` Greg KH
2011-12-19 10:49                   ` Ingo Molnar
2011-12-19 15:30                     ` [lttng-dev] " Mathieu Desnoyers
2011-12-20 11:08                       ` Ingo Molnar
2011-12-20 21:46                         ` Frank Rowand
2011-12-23 10:51                           ` Ingo Molnar
2011-12-21 18:47                         ` Aaron Spear
2011-12-21 18:58                           ` Christoph Hellwig
2011-12-23 16:46                         ` Perf ABI (was: Re: [lttng-dev] [PATCH 09/11] sched: export task_prio to GPL modules) Mathieu Desnoyers
2011-12-23 17:21                           ` Ted Ts'o
2011-12-23 18:16                             ` Mathieu Desnoyers
2011-12-25 17:46                               ` Ted Ts'o
2012-01-12 14:09                                 ` Mathieu Desnoyers
2012-01-12 14:54                                   ` Steven Rostedt
2012-01-12 15:39                                     ` [lttng-dev] Perf ABI (was: " Mathieu Desnoyers
2012-01-12 15:53                                       ` Steven Rostedt
2012-01-12 15:59                                         ` Steven Rostedt
2012-01-12 16:27                                         ` Mathieu Desnoyers
2012-01-12 16:34                                           ` Steven Rostedt
2012-01-12 20:00                                       ` Greg KH
2012-01-16  8:55                                         ` Ingo Molnar
2011-12-07 22:57             ` [PATCH 09/11] sched: export task_prio to GPL modules Mathieu Desnoyers
2011-12-08  5:40               ` Ingo Molnar

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=20111205141749.GC28866@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@linux-foundation.org \
    --cc=devel@driverdev.osuosl.org \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lttng-dev@lists.lttng.org \
    --cc=mathieu.desnoyers@efficios.com \
    --cc=peterz@infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.