Building OVS tree without DPDK produced the following warning message.
    lib/dpif-netdev.c:1868:5: error: statement with no effect

This error message is complaining the return value of the following
macro, (0),  is not being used.
        #define pmd_thread_setaffinity_cpu(c) (0)

The patch fixed this warnning by redefins the stub functions
as inline funtions.

Signed-off-by: Andy Zhou <az...@nicira.com>
---
 lib/netdev-dpdk.h |   33 ++++++++++++++++++++++++++-------
 1 file changed, 26 insertions(+), 7 deletions(-)

diff --git a/lib/netdev-dpdk.h b/lib/netdev-dpdk.h
index 55984e8..2807310 100644
--- a/lib/netdev-dpdk.h
+++ b/lib/netdev-dpdk.h
@@ -1,8 +1,10 @@
 #ifndef NETDEV_DPDK_H
 #define NETDEV_DPDK_H
 
-#ifdef DPDK_NETDEV
 #include <config.h>
+#include "ofpbuf.h"
+
+#ifdef DPDK_NETDEV
 
 #include <rte_config.h>
 #include <rte_eal.h>
@@ -16,8 +18,6 @@
 #include <rte_launch.h>
 #include <rte_malloc.h>
 
-#include "ofpbuf.h"
-
 int dpdk_init(int argc, char **argv);
 void netdev_dpdk_register(void);
 void free_dpdk_buf(struct ofpbuf *);
@@ -25,10 +25,29 @@ int pmd_thread_setaffinity_cpu(int cpu);
 
 #else
 
-#define dpdk_init(arg1, arg2) (0)
-#define netdev_dpdk_register()
-#define free_dpdk_buf(arg)
-#define pmd_thread_setaffinity_cpu(c) (0)
+static inline int
+dpdk_init(int arg1 OVS_UNUSED, char **arg2 OVS_UNUSED)
+{
+    return 0;
+}
+
+static inline void
+netdev_dpdk_register(void)
+{
+    /* Nothing */
+}
+
+static inline void
+free_dpdk_buf(struct ofpbuf *buf OVS_UNUSED)
+{
+    /* Nothing */
+}
+
+static inline int
+pmd_thread_setaffinity_cpu(int cpu OVS_UNUSED)
+{
+    return 0;
+}
 
 #endif /* DPDK_NETDEV */
 #endif
-- 
1.7.9.5

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to