Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-16 Thread Jerin Jacob Kollanukkaran
> -Original Message- > From: Honnappa Nagarahalli > Sent: Monday, June 17, 2019 6:19 AM > To: Jerin Jacob Kollanukkaran ; dev@dpdk.org > Cc: tho...@monjalon.net; Gavin Hu (Arm Technology China) > ; Honnappa Nagarahalli > ; nd ; nd > Subject: [EXT] RE: [dpdk-dev

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-16 Thread Honnappa Nagarahalli
> > > > Reduced the CC list (changing the topic slightly) > > > > > > > > > > My understanding is that the generated code for both your patch > > > > and my changes above is the same. Above suggested changes will > > > > conform to ACLE recommendation. > > > > > > Though instructions are different.

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-11 Thread Jerin Jacob Kollanukkaran
> -Original Message- > From: Honnappa Nagarahalli > Sent: Wednesday, June 12, 2019 1:18 AM > To: Jerin Jacob Kollanukkaran ; dev@dpdk.org > Cc: tho...@monjalon.net; Gavin Hu (Arm Technology China) > ; nd ; nd > Subject: [EXT] RE: [dpdk-dev] [PATCH] acl: fix build

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-11 Thread Honnappa Nagarahalli
Reduced the CC list (changing the topic slightly) > > > > My understanding is that the generated code for both your patch and my > > changes above is the same. Above suggested changes will conform to > > ACLE recommendation. > > Though instructions are different. Effective cycles are same even th

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-11 Thread Jerin Jacob Kollanukkaran
pa Nagarahalli ; > nd ; nd > Subject: [EXT] RE: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 > compiler > > > > > > > > > > > > Since it used in fastpath, a temp variable would be additional > > > > cost for no reason. > >

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-10 Thread Honnappa Nagarahalli
> > > > -- > > > > > Subject: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 > > > > > compiler > > > > > > > > > > From: Jerin Jacob > > > > > > > > > > Some compilers reportin

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-10 Thread Aaron Conole
writes: > From: Jerin Jacob > > Some compilers reporting the following error, though the existing > code doesn't have any uninitialized variable case. > Just to make compiler happy, initialize the int32x4_t variable > one shot in C language. > > ../lib/librte_acl/acl_run_neon.h: In function 'sea

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-10 Thread Jerin Jacob Kollanukkaran
pa Nagarahalli ; > nd ; nd > Subject: [EXT] RE: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 > compiler > > > > -- > > > > Subject: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 > > > > compiler > > > > > > > &g

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-09 Thread Honnappa Nagarahalli
> > > > -- > > > Subject: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 > > > compiler > > > > > > From: Jerin Jacob > > > > > > Some compilers reportin

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-06 Thread Jerin Jacob Kollanukkaran
lanukkaran ; Honnappa Nagarahalli > ; sta...@dpdk.org; nd > Subject: [EXT] RE: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 > compiler > > -------------- > > Subject: [dpdk-dev] [PATCH] acl: fix build issue with s

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-06 Thread Honnappa Nagarahalli
On 6/6/19 10:50 AM, mailto:jer...@marvell.com wrote: From: Jerin Jacob mailto:jer...@marvell.com Some compilers reporting the following error, though the existing code doesn't have any uninitialized variable case. Just to make compiler happy, initialize the int32x4_t variable one shot in C languag

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-06 Thread Honnappa Nagarahalli
> Subject: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler > > From: Jerin Jacob > > Some compilers reporting the following error, though the existing code > doesn't have any uninitialized variable case. > Just to make compiler happy, initialize the i

Re: [dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-06 Thread Michael Santana Francisco
On 6/6/19 10:50 AM, jer...@marvell.com wrote: From: Jerin Jacob Some compilers reporting the following error, though the existing code doesn't have any uninitialized variable case. Just to make compiler happy, initialize the int32x4_t variable one shot in C language. ../lib/librte_acl/acl_run_

[dpdk-dev] [PATCH] acl: fix build issue with some arm64 compiler

2019-06-06 Thread jerinj
From: Jerin Jacob Some compilers reporting the following error, though the existing code doesn't have any uninitialized variable case. Just to make compiler happy, initialize the int32x4_t variable one shot in C language. ../lib/librte_acl/acl_run_neon.h: In function 'search_neon_4' ../lib/librt