Re: adding trace-cmd's plugins to perf

[Borislav Petkov]

On Tue, Apr 12, 2011 at 01:39:30PM -0600, David Ahern wrote:
> 
> 
> On 04/12/11 13:01, Borislav Petkov wrote:
> >>> Also, just FYI, I have this RAS daemon patchset which splits perf functionality
> >>> into different sub-libs, which are more or less topic based:
> >>>
> >>> tools/lib/trace/ - all the trace-event* stuff from Steven
> >>> tools/lib/lk/	- generic enough stuff (git headers, etc) which several tools could use
> 
> Is this going to contain the option parsing as well? e.g., something
> tools/kvm is picking up from perf.

Nope, not yet. Acme suggested I export only the stuff I need. But you
can definitely add them there since I have the Makefiles adjusted and
you'd have to move the .[ch] files and adjust the includes.

Here's what's there so far:

lib/lk/
|-- cpumap.c
|-- cpumap.h
|-- ctype.c
|-- debugfs.c
|-- debugfs.h
|-- Makefile
|-- thread_map.c
|-- thread_map.h
|-- types.h
|-- usage.c
|-- util.c
|-- util.h
|-- xyarray.c
`-- xyarray.h

> >>> tools/lib/perf/ - functionality related to perf events
> >>>
> >>> Here's the last submission http://marc.info/?l=linux-edac&m=129562244211501&w=2
> >>> and I'm currently finishing a new rebase against tip/perf/core.
> >>>
> >>> If you're fine with that split I'd suggest I load off my stuff first so
> >>> that David can continue from there and I can finally stop rebasing like
> >>> crazy each time perf moves forward.
> >>>
> >>> Opinions? Suggestions?
> >>
> >> Yeah, David, can you try Borislav's patchset and tell us what you think
> >> wrt servicing your needs?
> > 
> > Yeah, but first let me post the latest version - I'll try to do so in
> > the next couple of days.
> > 
> > Thanks.
> > 
> 
> Sure. This is a background task, and I still need to spend time on
> trace-cmd to understand it. ie. it's not something I expect to do within
> the next week -- more like within the next month if I get the time.

Ok, I'll let you know when I have it done.

-- 
Regards/Gruss,
    Boris.
--
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/