On Wed, Apr 10, 2013 at 10:08:56PM +0300, Abel Gordon wrote: > > > Gleb Natapov <gleb@xxxxxxxxxx> wrote on 08/04/2013 01:27:28 PM: > > > > stat counters are deprecated in favor of trace points. Adding kvmnested > > trace system is very welcome though. > > So, should I keep or remove this patch ? Remove. > If a kvmnested trace system is added then I'll add shadow-vmcs related > events Adding it is as simple as adding arch/x86/kvm/nestedtrace.h file similar to arch/x86/kvm/mmutrace.h. Doing #define TRACE_SYSTEM kvmnested there and adding trace events that you want to trace. The advantage is that you can provide additional information with each trace point. For example name and value of a field that was vmwread. -- Gleb. -- 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