Re: [dpdk-dev] [PATCH] ppc64le: fix build with Clang

2021-10-14 Thread Piotr Kubaj
The vector -> dpdk_vector renaming was done because there's a namespace pollution. vector is a keyword on ppc64le and that causes an error for clang. On 21-10-14 19:35:37, Piotr Kubaj wrote: > Couple of quite common issues: > 1. In rte_cycles.h, sys/platform/ppc.h and __ppc_get_

[dpdk-dev] [PATCH] ppc64le: fix build with Clang

2021-10-14 Thread Piotr Kubaj
__vector is necessary. 4. In i40e_rxtx_vec_altivec.c, some type changes from vector unsigned long to vector unsigned int is necessary since clang doesn't support vector unsigned long very well and deprecated it. Signed-off-by: Piotr Kubaj --- app/test-bbdev/test_bbdev_perf.c

Re: [dpdk-dev] [PATCH v5] build: optional NUMA and cpu counts detection

2021-09-01 Thread Piotr Kubaj
On 21-08-31 07:54:16, Juraj Linkeš wrote: > > > > -Original Message- > > From: Piotr Kubaj > > Sent: Tuesday, August 31, 2021 2:55 AM > > To: Juraj Linkeš > > Cc: David Christensen ; tho...@monjalon.net; > > david.march...@redhat.com; bruce.r

Re: [dpdk-dev] [PATCH v5] build: optional NUMA and cpu counts detection

2021-08-31 Thread Piotr Kubaj
, August 3, 2021 1:29 AM > > To: Juraj Linkeš ; tho...@monjalon.net; > > david.march...@redhat.com; bruce.richard...@intel.com; > > honnappa.nagaraha...@arm.com; ruifeng.w...@arm.com; > > ferruh.yi...@intel.com; jerinjac...@gmail.com; jer...@marvell.com; > > step...

Re: [dpdk-dev] [PATCH] ppc64le: fix build without glibc and using Clang

2021-05-20 Thread Piotr Kubaj
On 21-05-19 11:59:35, Thomas Monjalon wrote: > 19/05/2021 11:57, Piotr Kubaj: > > Well, isn't FreeBSD already supported? > > > > Even https://www.dpdk.org/ mentions supporting FreeBSD. > > Yes it is supposed to be supported. > But why are you adding all this co

Re: [dpdk-dev] [PATCH] ppc64le: fix build without glibc and using Clang

2021-05-20 Thread Piotr Kubaj
Well, isn't FreeBSD already supported? Even https://www.dpdk.org/ mentions supporting FreeBSD. On 21-05-19 09:53:16, Thomas Monjalon wrote: > 18/05/2021 22:42, David Christensen: > > On 5/16/21 5:46 PM, Piotr Kubaj wrote: > > > __ppc_get_timebase() is only pre

[dpdk-dev] [PATCH] ppc64le: fix build without glibc and using Clang

2021-05-17 Thread Piotr Kubaj
__ppc_get_timebase() is only present when glibc is used. Signed-off-by: Piotr Kubaj --- lib/eal/ppc/include/rte_altivec.h | 3 +++ lib/eal/ppc/include/rte_cycles.h | 12 lib/eal/ppc/rte_cycles.c | 16 3 files changed, 31 insertions(+) diff --git a/lib

Re: [dpdk-dev] [PATCH] ppc64le: fix build with Clang and without glibc

2021-03-23 Thread Piotr Kubaj
> On 3/11/21 8:11 AM, Piotr Kubaj wrote: > > There are couple of issues when building with Clang: > > 1. vector is a keyword and should not be used in code. I undefined it, > > but it would probably be better to just change the variable name. > > 2. vector long is deprecate

Re: [dpdk-dev] [PATCH] ppc64le: fix build with Clang and without glibc

2021-03-23 Thread Piotr Kubaj
Looks like I forgot to commit the change that includes headers for sysctlbyname(). Patch attached. On 21-03-15 12:48:19, David Christensen wrote: > > > On 3/11/21 8:11 AM, Piotr Kubaj wrote: > > There are couple of issues when building with Clang: > > 1. vector is a keyw

Re: [dpdk-dev] [PATCH] net/bnxt: rename a member to avoid conflict

2021-03-14 Thread Piotr Kubaj
and should not be used in code. > Renaming it to avoid conflict. > > Reported-by: Piotr Kubaj > Signed-off-by: Ajit Khaparde > --- > drivers/net/bnxt/bnxt_irq.c | 2 +- > drivers/net/bnxt/bnxt_irq.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > d

[dpdk-dev] [PATCH] ppc64le: fix build with Clang and without glibc

2021-03-14 Thread Piotr Kubaj
, sys/platform/ppc.h is glibc-dependant and is not available in other libc's. Use the portable method of reading TBR when glibc is not used. Taken from https://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/powerpc/sys/platform/ppc.h Signed-off-by: Piotr Kubaj --- app/test-