Thread (49 messages) 49 messages, 7 authors, 2020-10-26

Re: [dpdk-dev] [RFC 1/3] net/ixgbe: avoid multpile definitions of 'bool'

From: Stephen Hemminger <stephen@networkplumber.org>
Date: 2019-09-05 15:26:53
Subsystem: networking drivers, the rest · Maintainers: Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds

On Sun,  1 Sep 2019 06:58:08 +0000
Dharmik Thakkar [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Compilation issue arises due to multiple definitions of 'bool'
in 'ixgbe_ethdev.h'.
(Please note that this issue showed up when 'rte_rcu_qsbr.h' got included
within 'rte_hash.h', because 'rte_rcu_qsbr.h' includes 'stdbool.h'. This
is a temporary fix. Recommend suggesting better solution.)

Signed-off-by: Dharmik Thakkar <redacted>
---
 drivers/net/ixgbe/ixgbe_ethdev.h | 1 +
 1 file changed, 1 insertion(+)
diff --git a/drivers/net/ixgbe/ixgbe_ethdev.h b/drivers/net/ixgbe/ixgbe_ethdev.h
index 6e9ed2e10f3c..dd4d6c022545 100644
--- a/drivers/net/ixgbe/ixgbe_ethdev.h
+++ b/drivers/net/ixgbe/ixgbe_ethdev.h
@@ -18,6 +18,7 @@
 #include <rte_flow.h>
 #include <rte_time.h>
 #include <rte_hash.h>
+#undef bool
 #include <rte_pci.h>
 #include <rte_bus_pci.h>
 #include <rte_tm_driver.h>
Just fix ixgbe_ethdev.h to include stdbool itself?
diff --git a/drivers/net/ixgbe/ixgbe_ethdev.h b/drivers/net/ixgbe/ixgbe_ethdev.h
index 6e9ed2e10f3c..cf32b1761acb 100644
--- a/drivers/net/ixgbe/ixgbe_ethdev.h
+++ b/drivers/net/ixgbe/ixgbe_ethdev.h
@@ -6,6 +6,7 @@
 #define _IXGBE_ETHDEV_H_
 
 #include <stdint.h>
+#include <stdbool.h>
 
 #include "base/ixgbe_type.h"
 #include "base/ixgbe_dcb.h"
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help