Re: Our failure on tracing tools unification (Was: Re: [RFC][PATCH00/15] tools: Unify perf and trace-cmd trace event format parsing v2)

From: Steven Rostedt
Date: Mon Apr 23 2012 - 11:31:40 EST


On Mon, 2012-04-23 at 17:08 +0200, Peter Zijlstra wrote:
> On Mon, 2012-04-23 at 16:47 +0200, Frederic Weisbecker wrote:
> > Ingo doesn't seem to want this library outside perf in order to avoid
> > the fragmentation of the efforts on tracing tools.
>
> I really don't see the point.. parsing /debug/tracing/events/*/format
> gunk really is separate, furthermore things like powertop et al already
> are separate projects and really need this lib.

Note, powertop has already pulled in the lib into itself, thus we now
have 3 dis-joint locations. I can imagine it getting worse soon.

-- Steve

>
> I mean, who gives a bother where all that crap lives, if its in the
> kernel tree its all close enough to keep an eye on.. its not like
> kernel/events/ and kernel/trace/ are the same directory.
>
> Ingo, please could you lighten up and let people get stuff done? Merging
> the two implementations, wherever the result lives, is a better
> situation that two dis-joint implementations. So either let Frederic
> work or do it yourself but don't hand-wave and road-block stuff.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/