From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 41EE71B3D4 for ; Tue, 7 Nov 2017 09:05:02 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga105.fm.intel.com with ESMTP; 07 Nov 2017 00:05:01 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,357,1505804400"; d="scan'208";a="918402352" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.255.231.67]) ([10.255.231.67]) by FMSMGA003.fm.intel.com with ESMTP; 07 Nov 2017 00:05:01 -0800 To: Jerin Jacob , Jianbo Liu Cc: Guduri Prathyusha , tomasz.kantecki@intel.com, guduriprathyusha@gmail.com, dev@dpdk.org, thomas@monjalon.net References: <20171030073126.1662-1-gprathyusha@caviumnetworks.com> <20171030073614.GA27106@arm.com> <20171102103342.GA20462@jerin> From: Ferruh Yigit Message-ID: <79e54de9-f542-a624-e55f-e43ba7d8a019@intel.com> Date: Tue, 7 Nov 2017 00:05:00 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20171102103342.GA20462@jerin> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2] examples/l3fwd: fix NEON instructions X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 07 Nov 2017 08:05:05 -0000 On 11/2/2017 3:33 AM, Jerin Jacob wrote: > -----Original Message----- >> Date: Mon, 30 Oct 2017 15:36:15 +0800 >> From: Jianbo Liu >> To: Guduri Prathyusha >> CC: tomasz.kantecki@intel.com, guduriprathyusha@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: and >>> dp2 should contain: in the first iteration. dp1 should >>> be and dp2 should be in the next iteration. >>> >>> 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 >> >> Acked-by: Jianbo Liu > > Acked-by: Jerin Jacob Applied to dpdk/master, thanks.