On Wed, 2011-07-27 at 15:59 +0300, Avi Kivity wrote:
> On 07/27/2011 03:55 PM, Stefan Hajnoczi wrote:
> > >
> > >  trace-cmd has a plugin API (and a kvm plugin) that you can use for
> > >  this, and I expect perf either does as well or can make use of one.
> > >  I don't think it's useful to complicate the print format description
> > >  further to support our weird use case.
> >
> > The problem with relying on a plugin is that every userspace application
> > (currently two that I am aware of: perf and trace-cmd) has to add
> > special-case kvm code to work around our crappy format.
> 
> Move the plugins to tools/trace-formats and install them in 
> /lib/modules/$(uname -r)/tracepoint-formats/*.so.
> 
> > With these patches we at least get back to an event format that is
> > self-describing - no plugin necessary.
> 
> There will always be special cases.  Look at how trace-cmd formats the 
> kvm_emulate_insn event.
> 

I plan on (hopefully soon) writing libperf.so that will do the plugin
work that trace-cmd has, such that the plugins will work for both
trace-cmd and perf, and any other utility.

-- Steve


--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to