Re: [PATCH 09/15] xen/trace: add tlb flush tracepoints
From: Jeremy Fitzhardinge
Date: Thu Jun 23 2011 - 21:08:25 EST
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
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/