Re: [PATCH 5/7] ftrace, perf: Add support to use functiontracepoint in perf

[Steven Rostedt]

On Sat, 2012-02-04 at 14:21 +0100, Frederic Weisbecker wrote:
>   struct ftrace_event_call __used event_##call = {			\
> > @@ -170,4 +179,9 @@ struct ftrace_event_call __used event_##call = {			\
> >  struct ftrace_event_call __used						\
> >  __attribute__((section("_ftrace_events"))) *__event_##call = &event_##call;
> >  
> > +#undef FTRACE_ENTRY
> > +#define FTRACE_ENTRY(call, struct_name, etype, tstruct, print)		\
> > +	FTRACE_ENTRY_REG(call, struct_name, etype,			\
> > +			 PARAMS(tstruct), PARAMS(print), NULL)
> > +
> >  #include "trace_entries.h"
> 
> 
> Yeah looks good. I wouldn't mind having only FTRACE_ENTRY() with one
> more parameter but I'm fine with the two macros as well.

I prefer the two macros. It's cleaner.

-- Steve


--
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/