On 11/27/20 4:54 PM, Daniel Borkmann wrote: > On 11/27/20 8:27 PM, trix@xxxxxxxxxx wrote: >> From: Tom Rix <trix@xxxxxxxxxx> >> >> The macro use will already have a semicolon. >> >> Signed-off-by: Tom Rix <trix@xxxxxxxxxx> >> --- >> include/trace/events/xdp.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/include/trace/events/xdp.h b/include/trace/events/xdp.h >> index cd24e8a59529..65ffedf8386f 100644 >> --- a/include/trace/events/xdp.h >> +++ b/include/trace/events/xdp.h >> @@ -146,13 +146,13 @@ DEFINE_EVENT(xdp_redirect_template, xdp_redirect_err, >> ); >> #define _trace_xdp_redirect(dev, xdp, to) \ >> - trace_xdp_redirect(dev, xdp, NULL, 0, NULL, to); >> + trace_xdp_redirect(dev, xdp, NULL, 0, NULL, to) >> #define _trace_xdp_redirect_err(dev, xdp, to, err) \ >> trace_xdp_redirect_err(dev, xdp, NULL, err, NULL, to); >> #define _trace_xdp_redirect_map(dev, xdp, to, map, index) \ >> - trace_xdp_redirect(dev, xdp, to, 0, map, index); >> + trace_xdp_redirect(dev, xdp, to, 0, map, index) >> #define _trace_xdp_redirect_map_err(dev, xdp, to, map, index, err) \ >> trace_xdp_redirect_err(dev, xdp, to, err, map, index); >> > > This looks random, why those but not other locations ? Those other macros were never used. Tom > > Thanks, > Daniel >
