On Tue, 9 Apr 2024 18:09:34 +0800 Jason Xing <kerneljasonx...@gmail.com> wrote:
> /* > * tcp event with arguments sk and skb > @@ -74,20 +75,38 @@ DEFINE_EVENT(tcp_event_sk_skb, tcp_retransmit_skb, > TP_ARGS(sk, skb) > ); > > +#undef FN1 > +#define FN1(reason) TRACE_DEFINE_ENUM(SK_RST_REASON_##reason); > +#undef FN2 > +#define FN2(reason) TRACE_DEFINE_ENUM(SKB_DROP_REASON_##reason); > +DEFINE_RST_REASON(FN1, FN1) Interesting. I've never seen the passing of the internal macros to the main macro before. I see that you are using it for handling both the SK_RST_REASON and the SK_DROP_REASON. > + > +#undef FN1 > +#undef FNe1 > +#define FN1(reason) { SK_RST_REASON_##reason, #reason }, > +#define FNe1(reason) { SK_RST_REASON_##reason, #reason } > + > +#undef FN2 > +#undef FNe2 > +#define FN2(reason) { SKB_DROP_REASON_##reason, #reason }, > +#define FNe2(reason) { SKB_DROP_REASON_##reason, #reason } Anyway, from a tracing point of view, as it looks like it would work (I haven't tested it). Reviewed-by: Steven Rostedt (Google) <rost...@goodmis.org> -- Steve