Le 14/02/2024 à 16:10, Steven Rostedt a écrit :
On Wed, 14 Feb 2024 13:00:16 +0100
Christian König <christian.koenig@xxxxxxx> wrote:
+DEFINE_EVENT(dma_fence_from, dma_fence_sync_to,
For a single event you should probably use TRACE_EVENT() instead of
declaring a class. A class is only used if you have multiple events with
the same parameters.
FYI, TRACE_EVENT() is actually defined as:
#define TRACE_EVENT(name, proto, args, tstruct, assign, print) \
DECLARE_EVENT_CLASS(name, \
PARAMS(proto), \
PARAMS(args), \
PARAMS(tstruct), \
PARAMS(assign), \
PARAMS(print)); \
DEFINE_EVENT(name, name, PARAMS(proto), PARAMS(args));
So basically, you could really just declare one TRACE_EVENT() and add
DEFINE_EVENT()s on top of it ;)
I never recommended that because I thought it would be confusing.
Thanks Steve and Christian for your feedback.
I'm integrating your suggestions in my branch and will re-send the series
after more testing.
Pierre-Eric
-- Steve
