Messages in this thread |  | | | Date | Wed, 15 Apr 2009 09:07:55 -0700 | | From | Jeremy Fitzhardinge <> | | Subject | Re: [PATCH 2/8] tracing: create automated trace defines |
| |
Mathieu Desnoyers wrote: > * Jeremy Fitzhardinge (jeremy@goop.org) wrote: >> -#define __DO_TRACE(tp, proto, args) \ >> - do { \ >> +#define DEFINE_DO_TRACE(name, proto, args) \ >> + void __do_trace_##name(struct tracepoint *tp, TP_PROTO(proto)) \ >> > > I fear that won't work with "void" prototype. If we need this kind of > flexibility, we will need to create a special case for empty prototype. >
Yes, that has been a bit awkward. I couldn't find a way to create a no-param tracepoint, and so ended up passing a dummy arg. Stupid C syntax.
On the other hand, I can get something that actually compiles this way...
J
|  |