[lkml]   [2012]   [Apr]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: Our failure on tracing tools unification (Was: Re: [RFC][PATCH 00/15] tools: Unify perf and trace-cmd trace event format parsing v2)

* 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.
> 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.

As long as perf builds it not as a DSO - or has it in
tools/perf/libparseevent/ and co-installs the .so together with
perf (and it's packaged together) I'm fine with it.

One thing that would suck is perf being bound by various
versions of this library floating out there.

One reason why perf has become so popular is that it's
self-sufficient to a large degree and is very easy to
build/install. Lets not kill that aspect by the death of a
thousand cuts.

We had nothing but pain from external libraries so far, and
there's a reason why the Git project (3+ times larger than perf)
avoids library dependencies like the plague...

Anyway, Frederic, mind posting a version that does the above, so
I can have a look at how it all works in practice?



 \ /
  Last update: 2012-04-25 10:09    [W:0.093 / U:6.596 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site