-----Original Message-----
> Date: Mon, 30 Oct 2017 15:36:15 +0800
> From: Jianbo Liu <jianbo....@arm.com>
> To: Guduri Prathyusha <gprathyu...@caviumnetworks.com>
> CC: tomasz.kante...@intel.com, guduriprathyu...@gmail.com, dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] examples/l3fwd: fix NEON instructions
> User-Agent: Mutt/1.5.21 (2010-09-15)
> 
> The 10/30/2017 13:01, 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: <a,b,c,d> and
> > dp2 should contain: <b,c,d,e> in the first iteration. dp1 should
> > be <e,f,g,h> and dp2 should be <f,g,h,i> in the next iteration.
> >
> > Whereas the existing code incorrectly calculates dp1 as <d,e,f,g> 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 <gprathyu...@caviumnetworks.com>
> 
> Acked-by: Jianbo Liu <jianbo....@arm.com>

Acked-by: Jerin Jacob <jerin.ja...@caviumnetworks.com>

Cc: Thomas Monjalon <tho...@monjalon.net> to add this patch in RC3

> 
> > ---
> >
> > v2:
> >
> > * fix as suggested by Jianbo Liu
> > ---
> >  examples/l3fwd/l3fwd_neon.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/examples/l3fwd/l3fwd_neon.h b/examples/l3fwd/l3fwd_neon.h
> > index 42d50d3c2..4bc161394 100644
> > --- a/examples/l3fwd/l3fwd_neon.h
> > +++ b/examples/l3fwd/l3fwd_neon.h
> > @@ -192,7 +192,7 @@ send_packets_multi(struct lcore_conf *qconf, struct 
> > rte_mbuf **pkts_burst,
> >                        * dp1:
> >                        * <d[j], d[j+1], d[j+2], d[j+3], ... >
> >                        */
> > -                     dp1 = vextq_u16(dp1, dp1, FWDSTEP - 1);
> > +                     dp1 = vextq_u16(dp2, dp1, FWDSTEP - 1);
> >               }
> >
> >               /*
> > --
> > 2.14.1
> >
> 
> --
> IMPORTANT NOTICE: The contents of this email and any attachments are 
> confidential and may also be privileged. If you are not the intended 
> recipient, please notify the sender immediately and do not disclose the 
> contents to any other person, use it for any purpose, or store or copy the 
> information in any medium. Thank you.

Reply via email to