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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
