On Mon, Feb 10, 2025 at 2:37 PM Jerin Jacob <jer...@marvell.com> wrote:
> > +
> > +#define __RTE_TRACE_POINT(_mode, _tp, _args, ...) \ extern
> > +rte_trace_point_t __##_tp; \ static __rte_always_inline void _tp _args
> > +{ } \ static __rte_always_inline void \ _tp ## _register (void) \ { \
> > + __rte_trace_point_emit_header_##_mode(&__##_tp); \
> > + __VA_ARGS__ \
>
> I have not tested this code, Will this calling __rte_trace_point_emit*
> version of register?
> If not, CTF spec file not generated properly.
> https://github.com/DPDK/dpdk/blob/main/lib/eal/include/rte_trace_point_register.h#L35
Yes, there are issues with this series that I did not catch until recently.
This will be fixed in a new revision (that will also add babeltrace
calls in CI..).
--
David Marchand
