diff mbox

[dpdk-dev] examples/performance-thread: fix compilation on FreeBSD 10.0

Message ID 20170428101815.24090-1-bruce.richardson@intel.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Bruce Richardson April 28, 2017, 10:18 a.m. UTC 
  While later releases in the FreeBSD 10 series have a CPU_COUNT macro
defined, FreeBSD 10.0 and 10.1 do not have this macro. Therefore we provide
a basic fallback implementation of the macro for platforms where it is not
defined.

Fixes: 433ba6228f9a ("examples/performance-thread: add pthread_shim app")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 examples/performance-thread/pthread_shim/pthread_shim.c | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

Comments

De Lara Guarch, Pablo April 28, 2017, 12:38 p.m. UTC | #1 
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Bruce Richardson
> Sent: Friday, April 28, 2017 11:18 AM
> To: dev@dpdk.org
> Cc: Richardson, Bruce
> Subject: [dpdk-dev] [PATCH] examples/performance-thread: fix compilation
> on FreeBSD 10.0
> 
> While later releases in the FreeBSD 10 series have a CPU_COUNT macro
> defined, FreeBSD 10.0 and 10.1 do not have this macro. Therefore we
> provide
> a basic fallback implementation of the macro for platforms where it is not
> defined.
> 
> Fixes: 433ba6228f9a ("examples/performance-thread: add pthread_shim
> app")
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

Missing "Cc: stable@dpdk.org"

Apart from that:

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Thomas Monjalon May 1, 2017, 2:12 p.m. UTC | #2 
28/04/2017 14:38, De Lara Guarch, Pablo:
> From: Bruce Richardson
> > 
> > While later releases in the FreeBSD 10 series have a CPU_COUNT macro
> > defined, FreeBSD 10.0 and 10.1 do not have this macro. Therefore we
> > provide
> > a basic fallback implementation of the macro for platforms where it is not
> > defined.
> > 
> > Fixes: 433ba6228f9a ("examples/performance-thread: add pthread_shim
> > app")
> > 
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> 
> Missing "Cc: stable@dpdk.org"
> 
> Apart from that:
> 
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Applied, thanks
diff mbox

Patch

diff --git a/examples/performance-thread/pthread_shim/pthread_shim.c b/examples/performance-thread/pthread_shim/pthread_shim.c
index 7167311..113bafa 100644
--- a/examples/performance-thread/pthread_shim/pthread_shim.c
+++ b/examples/performance-thread/pthread_shim/pthread_shim.c
@@ -48,6 +48,21 @@ 
 
 #define POSIX_ERRNO(x)  (x)
 
+/* some releases of FreeBSD 10, e.g. 10.0, don't have CPU_COUNT macro */
+#ifndef CPU_COUNT
+#define CPU_COUNT(x) __cpu_count(x)
+
+static inline unsigned int
+__cpu_count(const rte_cpuset_t *cpuset)
+{
+	unsigned int i, count = 0;
+	for (i = 0; i < RTE_MAX_LCORE; i++)
+		if (CPU_ISSET(i, cpuset))
+			count++;
+	return count;
+}
+#endif
+
 /*
  * this flag determines at run time if we override pthread
  * calls and map then to equivalent lthread calls