From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 9D519A09E0; Fri, 11 Dec 2020 13:06:12 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 81D17AC9E; Fri, 11 Dec 2020 13:06:11 +0100 (CET) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id D8097AC9A for ; Fri, 11 Dec 2020 13:06:08 +0100 (CET) IronPort-SDR: Ua7/FiEnQDmlrZ3op2aavYWJV2yVT87KbfYbCGWPZo1tgZzEhzCYZBMHacFphwm5x6doQ/FcXq Ea7aNu03hMfw== X-IronPort-AV: E=McAfee;i="6000,8403,9831"; a="162177747" X-IronPort-AV: E=Sophos;i="5.78,411,1599548400"; d="scan'208";a="162177747" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Dec 2020 04:06:06 -0800 IronPort-SDR: CIDK1JbBZKSFrC0LLqyu8+5Khcd4e2jmZ5UbqymlT7kzBhqa3COxDzxoxi6ZM6GwZhMikFtYCj e1CsBp6yVnoA== X-IronPort-AV: E=Sophos;i="5.78,411,1599548400"; d="scan'208";a="365384536" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.200.9]) ([10.213.200.9]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Dec 2020 04:06:05 -0800 To: Feifei Wang Cc: dev@dpdk.org, nd@arm.com References: <20201118073524.25646-1-feifei.wang2@arm.com> <20201118104859.29047-1-feifei.wang2@arm.com> From: Ferruh Yigit Message-ID: Date: Fri, 11 Dec 2020 12:06:04 +0000 MIME-Version: 1.0 In-Reply-To: <20201118104859.29047-1-feifei.wang2@arm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 0/4] Enable Checksum Offloading for NEON vector 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 11/18/2020 10:48 AM, Feifei Wang wrote: > This patch series are mainly to enable checksum offloading for IXGBE NEON > vector PMD, including good and bad checksum flags. In the meanwhile, the > first patch enable VLAN stripping flag for Arm. > Following are the test results for the patches: > > NICs: 82599(igb) > Driver: ixgbe(vector) > $:./app/dpdk-testpmd -c 0x3 -w 0002:f9:00.0 -- -i --port-topology=chained > test-pmd> set fwd rxonly > test-pmd> set verbose 1 > test-pmd> start > > With Patch a: > enable vlan stripping: > src=00:00:00:00:00:02 - dst=00:00:00:00:00:01 - type=0x0800 - length=70 - nb_segs=1 - VLAN tci=0x1 > ol_flags: PKT_RX_VLAN PKT_RX_VLAN_STRIPPED > > With Patch b: > Packet: IPv4_checksum = 0xee && UDP_checksum = 0xee > src=00:00:00:00:00:02 - dst=00:00:00:00:00:01 - type=0x0800 - length=70 - nb_segs=1 > ol_flags: PKT_RX_L4_CKSUM_BAD PKT_RX_IP_CKSUM_BAD > > With Patch c: > Packet: IPv4_checksum = correct value && UDP_checksum = correct value > ol_flags: PKT_RX_L4_CKSUM_GOOD PKT_RX_IP_CKSUM_GOOD > > v2: > 1. update the commit message title to align the design (Wang, Haiyue) > 2. make the commit message more descriptive and move the test results > to the cover letter (Wang, Haiyue) > > Feifei Wang (4): > a). net/ixgbe: add new flag of stripped VLAN for NEON vector > b). net/ixgbe: support bad checksum flag for NEON vector > c). net/ixgbe: support good checksum flag for NEON vector > d). net/ixgbe: enable IXGBE NEON vector when need to checksum > > drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 94 ++++++++++++++++++------- > 1 file changed, 68 insertions(+), 26 deletions(-) > Merge note seems sent to different version, moving here: Applied to dpdk-next-net-intel. Pulling it to next-net.