From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Subject: Re: [PATCH v4 0/3] add lpm support for NEON
Date: Tue, 16 Feb 2016 22:14:10 +0530
Message-ID: <20160216164408.GA10571@localhost.localdomain>
References: <1454040645-23864-1-git-send-email-jerin.jacob@caviumnetworks.com>
 <1455280123-9311-1-git-send-email-jerin.jacob@caviumnetworks.com>
 <987EE72691933347B9F0B5C19E71B5BB1F02C6E3@IRSMSX101.ger.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Cc: "dev@dpdk.org" <dev@dpdk.org>,
 "viktorin@rehivetech.com" <viktorin@rehivetech.com>
To: "Kobylinski, MichalX" <michalx.kobylinski@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from na01-bn1-obe.outbound.protection.outlook.com
 (mail-bn1bon0082.outbound.protection.outlook.com [157.56.111.82])
 by dpdk.org (Postfix) with ESMTP id 5D0972A5E
 for <dev@dpdk.org>; Tue, 16 Feb 2016 17:44:22 +0100 (CET)
Content-Disposition: inline
In-Reply-To: <987EE72691933347B9F0B5C19E71B5BB1F02C6E3@IRSMSX101.ger.corp.intel.com>
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On Tue, Feb 16, 2016 at 01:27:02PM +0000, Kobylinski, MichalX wrote:
> 
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jerin Jacob
> > Sent: Friday, February 12, 2016 1:29 PM
> > To: dev@dpdk.org
> > Cc: viktorin@rehivetech.com
> > Subject: [dpdk-dev] [PATCH v4 0/3] add lpm support for NEON
> > 
[snip]
> > 
> > Jerin Jacob (3):
> >   lpm: make rte_lpm_lookupx4 API definition architecture agnostic
> >   lpm: add support for NEON
> >   maintainers: claim responsibility for arm64 specific files of hash and
> >     lpm
> > 
> >  MAINTAINERS                                |   3 +
> >  app/test/test_lpm.c                        |  21 ++--
> >  app/test/test_xmmt_ops.h                   |  67 +++++++++++++
> >  config/defconfig_arm-armv7a-linuxapp-gcc   |   3 -
> >  config/defconfig_arm64-armv8a-linuxapp-gcc |   3 -
> >  lib/librte_lpm/Makefile                    |   6 ++
> >  lib/librte_lpm/rte_lpm.h                   |  99 ++-----------------
> >  lib/librte_lpm/rte_lpm_neon.h              | 148 +++++++++++++++++++++++++++++
> >  lib/librte_lpm/rte_lpm_sse.h               | 143 ++++++++++++++++++++++++++++
> >  9 files changed, 386 insertions(+), 107 deletions(-)  create mode 100644
> > app/test/test_xmmt_ops.h  create mode 100644 lib/librte_lpm/rte_lpm_neon.h
> > create mode 100644 lib/librte_lpm/rte_lpm_sse.h
> > 
> > --
> > 2.1.0
> 
> Hi Jerin,

Hi Michal,

> Are you planning increase next_hop field for ARM? I extended next_hop field from 8 bits to 24 bits and created structure to configure LPM for x86.

Yes, I am planning to increase next_hop field for ARM as a separate
patch.  Let this base patchset get merges.

I will make  ARM specific changes for your new feature in
'rte_lpm_lookupx4' as a separate patch on top of your series.
So that in case if I want to go back to 8 bit then I can do it

Jerin

> Please look at my patchset with proposal increase next_hop field and structure to configure.
> 
> http://patchwork.dpdk.org/dev/patchwork/patch/10249/
> http://patchwork.dpdk.org/dev/patchwork/patch/10250/
> 
> Best Regards,
> Michal
>