From 5967c6e11287180615cc4df4dbb3835c2aa4e5ec Mon Sep 17 00:00:00 2001 From: Josep Puigdemont Date: Tue, 16 Jan 2018 13:07:22 +0100 Subject: [PATCH] pktio: honor config.h settings 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 to match the description in config.h Signed-off-by: Josep Puigdemont --- .../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/mdev.c | 2 +- platform/linux-generic/pktio/mdev/cxgb4.c | 2 +- platform/linux-generic/pktio/mdev/i40e.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 | 32 +++++++++---------- platform/linux-generic/pktio/tap.c | 2 +- 12 files changed, 27 insertions(+), 27 deletions(-) diff --git a/platform/linux-generic/include/odp_packet_internal.h b/platform/linux-generic/include/odp_packet_internal.h index fed562aa34..72ba6ac188 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 7d39bb1bc4..3f716ab101 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 diff --git a/platform/linux-generic/pktio/dpdk.h b/platform/linux-generic/pktio/dpdk.h index 7495b59653..d4e8cf064d 100644 --- a/platform/linux-generic/pktio/dpdk.h +++ b/platform/linux-generic/pktio/dpdk.h @@ -13,7 +13,7 @@ #include -#ifdef ODP_PKTIO_DPDK +#if defined(ODP_PKTIO_DPDK) && ODP_PKTIO_DPDK == 1 #include #include diff --git a/platform/linux-generic/pktio/ipc.c b/platform/linux-generic/pktio/ipc.c index 9d928c72a9..4ca7d64ec9 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 #include diff --git a/platform/linux-generic/pktio/mdev.c b/platform/linux-generic/pktio/mdev.c index 45ea2d1138..e3917146f4 100644 --- a/platform/linux-generic/pktio/mdev.c +++ b/platform/linux-generic/pktio/mdev.c @@ -6,7 +6,7 @@ #include "config.h" -#ifdef ODP_MDEV +#if defined(ODP_MDEV) && ODP_MDEV == 1 #include #include diff --git a/platform/linux-generic/pktio/mdev/cxgb4.c b/platform/linux-generic/pktio/mdev/cxgb4.c index 31c3a9f585..f09835d9ff 100644 --- a/platform/linux-generic/pktio/mdev/cxgb4.c +++ b/platform/linux-generic/pktio/mdev/cxgb4.c @@ -6,7 +6,7 @@ #include "config.h" -#ifdef ODP_MDEV +#if defined(ODP_MDEV) && ODP_MDEV == 1 #include #include diff --git a/platform/linux-generic/pktio/mdev/i40e.c b/platform/linux-generic/pktio/mdev/i40e.c index c69da09cc2..6f3b71712a 100644 --- a/platform/linux-generic/pktio/mdev/i40e.c +++ b/platform/linux-generic/pktio/mdev/i40e.c @@ -6,7 +6,7 @@ #include "config.h" -#ifdef ODP_MDEV +#if defined(ODP_MDEV) && ODP_MDEV == 1 #include #include diff --git a/platform/linux-generic/pktio/netmap.c b/platform/linux-generic/pktio/netmap.c index 2631c816d9..fc17da076c 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 diff --git a/platform/linux-generic/pktio/socket.c b/platform/linux-generic/pktio/socket.c index 320de973c5..6c98bc60ca 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 diff --git a/platform/linux-generic/pktio/socket_mmap.c b/platform/linux-generic/pktio/socket_mmap.c index 26d0a2957a..1e49d5b28b 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 diff --git a/platform/linux-generic/pktio/subsystem.c b/platform/linux-generic/pktio/subsystem.c index 9ba3f877d1..199ed83fb4 100644 --- a/platform/linux-generic/pktio/subsystem.c +++ b/platform/linux-generic/pktio/subsystem.c @@ -22,29 +22,29 @@ 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 -#ifdef ODP_MDEV +#if defined(ODP_MDEV) && ODP_MDEV == 1 extern int enable_link_cxgb4_pktio_ops; extern int enable_link_i40e_pktio_ops; #endif @@ -55,29 +55,29 @@ 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 -#ifdef ODP_MDEV +#if defined(ODP_MDEV) && ODP_MDEV == 1 enable_link_cxgb4_pktio_ops = 1; enable_link_i40e_pktio_ops = 1; #endif diff --git a/platform/linux-generic/pktio/tap.c b/platform/linux-generic/pktio/tap.c index bb404d9143..81eaffef4d 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