[dpdk-dev] [PATCH v2] examples/l3fwd: fix aliasing in port grouping

2017-11-03 Thread Guduri Prathyusha
y and compiler reorders instructions that depend on pnum and pn. This breaks port grouping algorithm. This patch eliminates the above problem by introducing a compiler barrier between the instructions that depend on pnum, pn and lp. Fixes: 569b290cdb36 ("examples/l3fwd: add NEON implementat

Re: [dpdk-dev] [PATCH ] examples/l3fwd: fix aliasing in port grouping

2017-11-02 Thread Guduri Prathyusha
On Fri, Nov 03, 2017 at 11:21:43AM +0800, jianbo@arm.com wrote: > The 11/02/2017 15:52, Ananyev, Konstantin wrote: > > > > > > > -Original Message- > > > From: Guduri Prathyusha [mailto:gprathyu...@caviumnetworks.com] > > > Sent: Thursday,

Re: [dpdk-dev] [PATCH ] examples/l3fwd: fix aliasing in port grouping

2017-11-02 Thread Guduri Prathyusha
On Thu, Nov 02, 2017 at 02:46:43PM +, Ananyev, Konstantin wrote: > Hi, Hi > > > -Original Message- > > From: Guduri Prathyusha [mailto:gprathyu...@caviumnetworks.com] > > Sent: Thursday, November 2, 2017 2:31 PM > > To: Kantecki, Tomasz > > C

[dpdk-dev] [PATCH ] examples/l3fwd: fix aliasing in port grouping

2017-11-02 Thread Guduri Prathyusha
LD and ST instructions directly and hence no performance overhead. Fixes: 569b290cdb36 ("examples/l3fwd: add NEON implementation") Fixes: af1694d94bf1 ("examples/l3fwd: fix crash with gcc 5") Signed-off-by: Guduri Prathyusha --- examples/l3fwd/l3fwd_neon.h | 11 +++ exam

[dpdk-dev] [PATCH v2] examples/l3fwd: fix NEON instructions

2017-10-30 Thread Guduri Prathyusha
. Whereas the existing code incorrectly calculates dp1 as from second iteration. This patch fixes the incorrect ARM NEON instructions on dp1. Fixes: 569b290cdb36 ("examples/l3fwd: add NEON implementation") Signed-off-by: Guduri Prathyusha --- v2: * fix as suggested by Jianbo Liu ---

Re: [dpdk-dev] [PATCH] examples/l3fwd: fix NEON instructions

2017-10-30 Thread Guduri Prathyusha
On Mon, Oct 30, 2017 at 02:27:09PM +0800, Jianbo Liu wrote: > The 10/29/2017 13:18, Guduri Prathyusha wrote: > > To group consecutive packets with same destination port in bursts of 4 > > neon intrinsic data types dp1 and dp2 are calculated such that if > > dst_port[]={a,b,

Re: [dpdk-dev] [PATCH] examples/l3fwd: fix NEON instructions

2017-10-29 Thread Guduri Prathyusha
+ jianbo@arm.com On Sun, Oct 29, 2017 at 01:18:07PM +0530, Guduri Prathyusha wrote: > To group consecutive packets with same destination port in bursts of 4 > neon intrinsic data types dp1 and dp2 are calculated such that if > dst_port[]={a,b,c,d,e,f,g,h,i...} dp1 should contain:

[dpdk-dev] [PATCH] examples/l3fwd: fix NEON instructions

2017-10-29 Thread Guduri Prathyusha
NEON implementation") Signed-off-by: Guduri Prathyusha --- examples/l3fwd/l3fwd_neon.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/l3fwd/l3fwd_neon.h b/examples/l3fwd/l3fwd_neon.h index 42d50d3c2..1eace4e03 100644 --- a/examples/l3fwd/l3fwd_neon.h +++

Re: [dpdk-dev] [PATCH v2] service: fix service lcore stop function

2017-09-06 Thread Guduri Prathyusha
On Wed, Sep 06, 2017 at 08:53:30AM +, Van Haaren, Harry wrote: > > From: Guduri Prathyusha [mailto:gprathyu...@caviumnetworks.com] > > Sent: Wednesday, September 6, 2017 8:16 AM > > To: Van Haaren, Harry > > Cc: dev@dpdk.org; Guduri Prathyusha > > Subject: [dpd

[dpdk-dev] [PATCH v2] service: fix service lcore stop function

2017-09-06 Thread Guduri Prathyusha
of stopping the service lcore before and after stopping the service respectively Fixes: 21698354c832 ("service: introduce service cores concept") Fixes: f038a81e1c56 ("service: add unit tests") Signed-off-by: Guduri Prathyusha --- This patch is based on [1] v2 changes: - Co

Re: [dpdk-dev] [PATCH 1/2] service: fix service lcore stop function

2017-09-05 Thread Guduri Prathyusha
On Tue, Sep 05, 2017 at 02:34:44PM +, Van Haaren, Harry wrote: > > From: Guduri Prathyusha [mailto:gprathyu...@caviumnetworks.com] > > Sent: Tuesday, September 5, 2017 3:11 PM > > To: Van Haaren, Harry > > Cc: dev@dpdk.org; Guduri Prathyusha > > Subject: [dpdk

[dpdk-dev] [PATCH 2/2] service: fix service lcore start stop unit test

2017-09-05 Thread Guduri Prathyusha
: add unit tests") Signed-off-by: Guduri Prathyusha --- test/test/test_service_cores.c | 4 1 file changed, 4 insertions(+) diff --git a/test/test/test_service_cores.c b/test/test/test_service_cores.c index 88fac8f44..b043397ef 100644 --- a/test/test/test_service_cores.c +++ b

[dpdk-dev] [PATCH 1/2] service: fix service lcore stop function

2017-09-05 Thread Guduri Prathyusha
lcore_states store the state of the lcore. Fixing the invalid dereference of lcore_states with service number Fixes: 21698354c832 ("service: introduce service cores concept") Signed-off-by: Guduri Prathyusha --- lib/librte_eal/common/rte_service.c | 2 +- 1 file changed, 1 inser

[dpdk-dev] [PATCH] service: fix service lcore stop function

2017-09-05 Thread Guduri Prathyusha
lcore_states store the state of the lcore. Fixing the invalid dereference of lcore_states with service number Fixes: 21698354c832 ("service: introduce service cores concept") Signed-off-by: Guduri Prathyusha --- lib/librte_eal/common/rte_service.c | 2 +- 1 file changed, 1 inser

Re: [dpdk-dev] [PATCH] librte_cfgfile: add section num entries by index

2017-09-04 Thread Guduri Prathyusha
On Mon, Sep 04, 2017 at 11:58:44AM +0100, Bruce Richardson wrote: > On Thu, Apr 27, 2017 at 12:21:40PM +0530, Guduri Prathyusha wrote: > > rte_cfgfile_section_num_entries_by_index() is added to get the number of > > entries of a section when multiple sections of the same name

[dpdk-dev] [PATCH] librte_cfgfile: add section num entries by index

2017-04-26 Thread Guduri Prathyusha
rte_cfgfile_section_num_entries_by_index() is added to get the number of entries of a section when multiple sections of the same name are present. Signed-off-by: Guduri Prathyusha --- lib/librte_cfgfile/rte_cfgfile.c | 13 + lib/librte_cfgfile/rte_cfgfile.h | 20

[dpdk-dev] [PATCH] usertools: fix device binding when using with kernel tools

2017-04-26 Thread Guduri Prathyusha
y: Tu LijuanX A Signed-off-by: Guduri Prathyusha --- usertools/dpdk-devbind.py | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/usertools/dpdk-devbind.py b/usertools/dpdk-devbind.py index bb4d536..1dc1065 100755 --- a/usertools/dpdk-d