| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 18 Jun 2012 17:35:56 +0900 From: Namhyung Kim <namhyung@...nel.org> To: David Ahern <dsahern@...il.com>, rostedt@...dmis.org, acme@...stprotocols.net Cc: linux-kernel@...r.kernel.org, weisbec@...il.com, namhyung.kim@....com, mingo@...nel.org, peterz@...radead.org Subject: Re: [RFC PATCH 0/2] libtraceevent/perf: Add support for trace-cmd plugins Hi, David On Thu, 14 Jun 2012 11:35:31 -0600, David Ahern wrote: > Now that perf is using libtraceevent and libtraceevent is based > on trace-cmd both can be extended to leverage the plugins written > for trace-cmd to improve pretty printing of the events. > > Given that it is based on code from trace-cmd I am not sure what the > right approach is, so wanted to throw this out for comments/suggestions. > Yeah, it can be useful to reuse existing code for extending the functionality. But I'm not so sure including the plugin APIs into libtraceevent is the right thing (at least in its current form). And for this particular case in patch 2/2, it seems that format of the kvm_emulate_insn event is broken already and should be fixed anyway. Further improvement in this area can be addressed in perf kvm or other users if needed. So I'd like to hear from others. Arnaldo and Steven, what do you think? Thanks, Namhyung -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists