From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id B2995ADA7 for ; Tue, 16 Feb 2016 14:27:05 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga104.fm.intel.com with ESMTP; 16 Feb 2016 05:27:04 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,455,1449561600"; d="scan'208";a="747241033" Received: from irsmsx152.ger.corp.intel.com ([163.33.192.66]) by orsmga003.jf.intel.com with ESMTP; 16 Feb 2016 05:27:03 -0800 Received: from irsmsx101.ger.corp.intel.com ([169.254.1.113]) by IRSMSX152.ger.corp.intel.com ([169.254.6.200]) with mapi id 14.03.0248.002; Tue, 16 Feb 2016 13:27:02 +0000 From: "Kobylinski, MichalX" To: Jerin Jacob , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v4 0/3] add lpm support for NEON Thread-Index: AQHRZZEO9cgxYwRQAE6mS3SmSKofy58ur3NQ Date: Tue, 16 Feb 2016 13:27:02 +0000 Message-ID: <987EE72691933347B9F0B5C19E71B5BB1F02C6E3@IRSMSX101.ger.corp.intel.com> References: <1454040645-23864-1-git-send-email-jerin.jacob@caviumnetworks.com> <1455280123-9311-1-git-send-email-jerin.jacob@caviumnetworks.com> In-Reply-To: <1455280123-9311-1-git-send-email-jerin.jacob@caviumnetworks.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "viktorin@rehivetech.com" Subject: Re: [dpdk-dev] [PATCH v4 0/3] add lpm support for NEON X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Feb 2016 13:27:06 -0000 > -----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 >=20 > - This patch enables lpm for ARM > - Used architecture agnostic xmm_t to represent 128 bit SIMD variable in > rte_lpm_lookupx4 API definition > - Tested on Juno and Thunderx boards > - Tested and verified the changes with following DPDK unit test cases > --lpm_autotest > --lpm6_autotest > v1..v2 > - make rte_lpm_lookupx4 API definition architecture agnostic > - vect_* abstraction scope reduce to only app/test as this abstraction us= ed only > to load/store and set vectors in test application which is the consumer o= f > rte_lpm_lookupx4 like API > - support for armv7 apart from armv8 > - taken changes from Jianbo's lpm patches >=20 > v2..v3 > - add Acked-by for 0001-lpm-make-rte_lpm_lookupx4-API-definition- > architectur.patch > - re-based to DPDK 2.2 > -- fixed the conflict in config/defconfig_arm-armv7a-linuxapp-gcc and > MAINTAINERS file >=20 > v3..v4 > -Instead of defaulting the lpm implementation to SSE, SSE implementation = kept > under RTE_ARCH_X86 conditional compilation check as suggested by Thomas >=20 > 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 >=20 > 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 >=20 > -- > 2.1.0 Hi Jerin, 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. Please look at my patchset with proposal increase next_hop field and struct= ure to configure. http://patchwork.dpdk.org/dev/patchwork/patch/10249/ http://patchwork.dpdk.org/dev/patchwork/patch/10250/ Best Regards, Michal