[PATCH] bpf: remove trailing semicolon in macro definition

From: Tom Rix <trix@redhat.com>

The macro use will already have a semicolon.

Signed-off-by: Tom Rix <trix@redhat.com>
---
 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);

-- 
2.18.4

On 11/27/20 8:27 PM, trix@redhat.com wrote:

This looks random, why those but not other locations ?

Thanks,
Daniel

On 11/27/20 4:54 PM, Daniel Borkmann wrote:

On 11/27/20 8:27 PM, trix@redhat.com wrote:

quoted

From: Tom Rix <trix@redhat.com>

The macro use will already have a semicolon.

Signed-off-by: Tom Rix <trix@redhat.com>
---
  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