diff mbox series

[CATERPILLAR,v1,1/1] pktio: honor config.h settings

Message ID 1516107608-12646-2-git-send-email-odpbot@yandex.ru
State New
Headers show
Series [CATERPILLAR,v1,1/1] pktio: honor config.h settings | expand

Commit Message

Github ODP bot Jan. 16, 2018, 1 p.m. UTC 
From: Josep Puigdemont <josep.puigdemont@linaro.org>


It's not enough to chech if a macro is defined to enable or disable a
pktio, we also need to check if it is set to 1 as per its description.

Signed-off-by: Josep Puigdemont <josep.puigdemont@linaro.org>

---
/** Email created from pull request 394 (joseppc:check-macro-value)
 ** https://github.com/Linaro/odp/pull/394
 ** Patch: https://github.com/Linaro/odp/pull/394.patch
 ** Base sha: 4d17f8ae64aba0e6f24877be30f86ae5880cef7e
 ** Merge commit sha: 33b93a8a6df7810422c61b2c5f0eb58d1388b980
 **/
 .../linux-generic/include/odp_packet_internal.h    |  2 +-
 platform/linux-generic/pktio/dpdk.c                |  2 +-
 platform/linux-generic/pktio/dpdk.h                |  2 +-
 platform/linux-generic/pktio/ipc.c                 |  2 +-
 platform/linux-generic/pktio/netmap.c              |  2 +-
 platform/linux-generic/pktio/socket.c              |  2 +-
 platform/linux-generic/pktio/socket_mmap.c         |  2 +-
 platform/linux-generic/pktio/subsystem.c           | 28 +++++++++++-----------
 platform/linux-generic/pktio/tap.c                 |  2 +-
 9 files changed, 22 insertions(+), 22 deletions(-)
diff mbox series

Patch

diff --git a/platform/linux-generic/include/odp_packet_internal.h b/platform/linux-generic/include/odp_packet_internal.h
index fed562aa3..72ba6ac18 100644
--- a/platform/linux-generic/include/odp_packet_internal.h
+++ b/platform/linux-generic/include/odp_packet_internal.h
@@ -145,7 +145,7 @@  typedef struct {
 	/* Result for crypto packet op */
 	odp_crypto_packet_result_t crypto_op_result;
 
-#ifdef ODP_PKTIO_DPDK
+#if defined(ODP_PKTIO_DPDK) && ODP_PKTIO_DPDK == 1
 	/* Type of extra data */
 	uint8_t extra_type;
 	/* Extra space for packet descriptors. E.g. DPDK mbuf  */
diff --git a/platform/linux-generic/pktio/dpdk.c b/platform/linux-generic/pktio/dpdk.c
index 7d39bb1bc..3f716ab10 100644
--- a/platform/linux-generic/pktio/dpdk.c
+++ b/platform/linux-generic/pktio/dpdk.c
@@ -6,7 +6,7 @@ 
 
 #include "config.h"
 
-#ifdef ODP_PKTIO_DPDK
+#if defined(ODP_PKTIO_DPDK) && ODP_PKTIO_DPDK == 1
 
 #include <odp_posix_extensions.h>
 
diff --git a/platform/linux-generic/pktio/dpdk.h b/platform/linux-generic/pktio/dpdk.h
index 7495b5965..d4e8cf064 100644
--- a/platform/linux-generic/pktio/dpdk.h
+++ b/platform/linux-generic/pktio/dpdk.h
@@ -13,7 +13,7 @@ 
 
 #include <net/if.h>
 
-#ifdef ODP_PKTIO_DPDK
+#if defined(ODP_PKTIO_DPDK) && ODP_PKTIO_DPDK == 1
 #include <rte_config.h>
 #include <rte_mbuf.h>
 
diff --git a/platform/linux-generic/pktio/ipc.c b/platform/linux-generic/pktio/ipc.c
index 9d928c72a..4ca7d64ec 100644
--- a/platform/linux-generic/pktio/ipc.c
+++ b/platform/linux-generic/pktio/ipc.c
@@ -6,7 +6,7 @@ 
 
 #include "config.h"
 
-#ifdef ODP_PKTIO_IPC
+#if defined(ODP_PKTIO_IPC) && ODP_PKTIO_IPC == 1
 
 #include <odp_debug_internal.h>
 #include <odp_packet_io_internal.h>
diff --git a/platform/linux-generic/pktio/netmap.c b/platform/linux-generic/pktio/netmap.c
index 2631c816d..fc17da076 100644
--- a/platform/linux-generic/pktio/netmap.c
+++ b/platform/linux-generic/pktio/netmap.c
@@ -6,7 +6,7 @@ 
 
 #include "config.h"
 
-#ifdef ODP_NETMAP
+#if defined(ODP_NETMAP) && ODP_NETMAP == 1
 
 #include <odp_posix_extensions.h>
 
diff --git a/platform/linux-generic/pktio/socket.c b/platform/linux-generic/pktio/socket.c
index 320de973c..6c98bc60c 100644
--- a/platform/linux-generic/pktio/socket.c
+++ b/platform/linux-generic/pktio/socket.c
@@ -7,7 +7,7 @@ 
 
 #include "config.h"
 
-#ifdef ODP_PKTIO_SOCKET
+#if defined(ODP_PKTIO_SOCKET) && ODP_PKTIO_SOCKET == 1
 
 #include <odp_posix_extensions.h>
 
diff --git a/platform/linux-generic/pktio/socket_mmap.c b/platform/linux-generic/pktio/socket_mmap.c
index 26d0a2957..1e49d5b28 100644
--- a/platform/linux-generic/pktio/socket_mmap.c
+++ b/platform/linux-generic/pktio/socket_mmap.c
@@ -7,7 +7,7 @@ 
 
 #include "config.h"
 
-#ifdef ODP_PKTIO_SOCKET_MMAP
+#if defined(ODP_PKTIO_SOCKET_MMAP) && ODP_PKTIO_SOCKET_MMAP == 1
 
 #include <odp_posix_extensions.h>
 
diff --git a/platform/linux-generic/pktio/subsystem.c b/platform/linux-generic/pktio/subsystem.c
index a3b36c144..e00206efc 100644
--- a/platform/linux-generic/pktio/subsystem.c
+++ b/platform/linux-generic/pktio/subsystem.c
@@ -22,26 +22,26 @@  ODP_SUBSYSTEM_FOREACH_TEMPLATE(pktio_ops, term_global, ODP_ABORT)
 /* Temporary variable to enable link modules,
  * will remove in Makefile scheme changes.
  */
-#ifdef ODP_PKTIO_DPDK
+#if defined(ODP_PKTIO_DPDK) && ODP_PKTIO_DPDK == 1
 extern int enable_link_dpdk_pktio_ops;
 #endif
-#ifdef ODP_PKTIO_IPC
+#if defined(ODP_PKTIO_IPC) && ODP_PKTIO_IPC == 1
 extern int enable_link_ipc_pktio_ops;
 #endif
 extern int enable_link_loopback_pktio_ops;
-#ifdef ODP_NETMAP
+#if defined(ODP_NETMAP) && ODP_NETMAP == 1
 extern int enable_link_netmap_pktio_ops;
 #endif
-#ifdef HAVE_PCAP
+#if defined(HAVE_PCAP) && HAVE_PCAP == 1
 extern int enable_link_pcap_pktio_ops;
 #endif
-#ifdef ODP_PKTIO_SOCKET
+#if defined(ODP_PKTIO_SOCKET) && ODP_PKTIO_SOCKET == 1
 extern int enable_link_socket_pktio_ops;
 #endif
-#ifdef ODP_PKTIO_SOCKET_MMAP
+#if defined(ODP_PKTIO_SOCKET_MMAP) && ODP_PKTIO_SOCKET_MMAP == 1
 extern int enable_link_socket_mmap_pktio_ops;
 #endif
-#ifdef ODP_PKTIO_TAP
+#if defined(ODP_PKTIO_TAP) && ODP_PKTIO_TAP == 1
 extern int enable_link_tap_pktio_ops;
 #endif
 
@@ -51,26 +51,26 @@  ODP_SUBSYSTEM_CONSTRUCTOR(pktio_ops)
 
 	/* Further initialization per subsystem */
 
-#ifdef ODP_PKTIO_DPDK
+#if defined(ODP_PKTIO_DPDK) && ODP_PKTIO_DPDK == 1
 	enable_link_dpdk_pktio_ops = 1;
 #endif
-#ifdef ODP_PKTIO_IPC
+#if defined(ODP_PKTIO_IPC) && ODP_PKTIO_IPC == 1
 	enable_link_ipc_pktio_ops = 1;
 #endif
 	enable_link_loopback_pktio_ops = 1;
-#ifdef ODP_NETMAP
+#if defined(ODP_NETMAP) && ODP_NETMAP == 1
 	enable_link_netmap_pktio_ops = 1;
 #endif
-#ifdef HAVE_PCAP
+#if defined(HAVE_PCAP) && HAVE_PCAP == 1
 	enable_link_pcap_pktio_ops = 1;
 #endif
-#ifdef ODP_PKTIO_SOCKET
+#if defined(ODP_PKTIO_SOCKET) && ODP_PKTIO_SOCKET == 1
 	enable_link_socket_pktio_ops = 1;
 #endif
-#ifdef ODP_PKTIO_SOCKET_MMAP
+#if defined(ODP_PKTIO_SOCKET_MMAP) && ODP_PKTIO_SOCKET_MMAP == 1
 	enable_link_socket_mmap_pktio_ops = 1;
 #endif
-#ifdef ODP_PKTIO_TAP
+#if defined(ODP_PKTIO_TAP) && ODP_PKTIO_TAP == 1
 	enable_link_tap_pktio_ops = 1;
 #endif
 }
diff --git a/platform/linux-generic/pktio/tap.c b/platform/linux-generic/pktio/tap.c
index bb404d914..81eaffef4 100644
--- a/platform/linux-generic/pktio/tap.c
+++ b/platform/linux-generic/pktio/tap.c
@@ -6,7 +6,7 @@ 
 
 #include "config.h"
 
-#ifdef ODP_PKTIO_TAP
+#if defined(ODP_PKTIO_TAP) && ODP_PKTIO_TAP == 1
 
 /**
  * @file