Re: [PATCH 09/15] xen/trace: add tlb flush tracepoints

Jeremy Fitzhardinge
2011-06-24T01:08:22Z

On 06/23/2011 01:23 PM, Steven Rostedt wrote:
> Ouch! I didn't notice this before. Do you really need this empty event?
> Why not just use the function tracer for this? It should work.

Yes, it should, but it would be nice to present something that's
consistent with all the other tlb-related tracepoints.

> If you really need something like this, maybe I'll add a
> DEFINE_EVENT_NOARGS() that will not require arguments and can eliminate
> a lot of code that TRACE_EVENT() adds.

If you like, but only if you think its worth the effort. The empty
string thing is a bit of an ugly hack, but its not like it makes a mess
everywhere.

J