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 6B3AEA0588; Thu, 16 Apr 2020 14:21:20 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 456731DC37; Thu, 16 Apr 2020 14:21:20 +0200 (CEST) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 0907B1DC35 for ; Thu, 16 Apr 2020 14:21:18 +0200 (CEST) IronPort-SDR: 4TwDY1XRSKsQTNrU0moK7O39ss9NWKFVguqV7MpqrPtE/uHVo1Z8USADS8XjktcVTQxdxisUAS 6/ldWxZn3nWg== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Apr 2020 05:21:18 -0700 IronPort-SDR: 5YEW9DphKyNXeAamT8EWAZ1Cg3HwQAXpAmiMWOJ43WQA/eP0qQvE/MhUCx2hcCEOMmw0pwcxUF 2dhIJ+KU2RxA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,390,1580803200"; d="scan'208";a="400652370" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by orsmga004.jf.intel.com with ESMTP; 16 Apr 2020 05:21:17 -0700 Received: from fmsmsx158.amr.corp.intel.com (10.18.116.75) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 16 Apr 2020 05:21:17 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx158.amr.corp.intel.com (10.18.116.75) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 16 Apr 2020 05:21:17 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.146]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.209]) with mapi id 14.03.0439.000; Thu, 16 Apr 2020 20:21:14 +0800 From: "Zhang, Qi Z" To: "Rong, Leyi" , "Wu, Jingjing" , "Xing, Beilei" , "Ye, Xiaolong" CC: "dev@dpdk.org" Thread-Topic: [PATCH v5 00/11] framework for advanced iAVF PMD Thread-Index: AQHWE8e/OwoBh46ve0Kmn5gjLpqJw6h7q1wg Date: Thu, 16 Apr 2020 12:21:13 +0000 Message-ID: <039ED4275CED7440929022BC67E70611547FBE9A@SHSMSX103.ccr.corp.intel.com> References: <20200316074603.10998-1-leyi.rong@intel.com> <20200416080947.55005-1-leyi.rong@intel.com> In-Reply-To: <20200416080947.55005-1-leyi.rong@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v5 00/11] framework for advanced iAVF PMD 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" > -----Original Message----- > From: Rong, Leyi > Sent: Thursday, April 16, 2020 4:10 PM > To: Wu, Jingjing ; Zhang, Qi Z > ; Xing, Beilei ; Ye, Xiaolon= g > > Cc: dev@dpdk.org; Rong, Leyi > Subject: [PATCH v5 00/11] framework for advanced iAVF PMD >=20 > This patchset enable framework for advanced iAVF, includes flexible > descriptor support, FDIR mark id and RSS hash support. >=20 > --- > v5: > - Set rxdid =3D 0 in iavf_configure_queues() when it's supported and > RTE_LIBRTE_IAVF_16BYTE_RX_DESC is y. So the error can be captured > explicitly as kernel PF does not support it yet. > - Fix some build errors when RTE_LIBRTE_IAVF_16BYTE_RX_DESC is y. >=20 > v4: > - Avoid initial declaration in 'for' loop to backward-compatible > with older gcc versions. >=20 > v3: > - Remove patch to query DDP package info as it's specific to DCF. > - Specify fdir_ref_cnt as per VF value for enable/disable FDIR ID parse. > - Move fdir_enabled flag to per Queue value for cache benefit when > access in vector routines. > - Store extracted flow_id value to mbuf without judgement to avoid > branch speculation fail, which is benefit to performance. >=20 > v2: > - Revert macro RTE_LIBRTE_IAVF_16BYTE_RX_DESC deletion as > it's defined in AVF spec. > - Typo RTE_LIBRTE_ICE_16BYTE_RX_DESC fix in iavf_rxtx.c. > - Move flex desc definitions into iavf_rxtx.h. > - Up to date to match with the latest version of virtchnl.h. > - Extract a new internal func iavf_update_rx_tail to call. > - Remove > iavf_dev_rxq_count_flex_rxd()/iavf_dev_rx_desc_status_flex_rxd(), > as the accompanying legacy ones can deal with the flex desc cases. > - Move rxq->rxdid assignment from iavf_configure_queues() > to iavf_dev_rx_queue_setup(). > - Unfold _mm_extract_epi32(fdir_id0_3, i) to fix build error > when using GCC compile option -O0. >=20 > Leyi Rong (11): > net/iavf: flexible Rx descriptor definitions > net/iavf: return error if opcode is mismatched > net/iavf: flexible Rx descriptor support in normal path > net/iavf: flexible Rx descriptor support in AVX path > net/iavf: flexible Rx descriptor support in SSE path > net/iavf: add flow director enabled switch value > net/iavf: support flow mark in normal data path > net/iavf: support flow mark in AVX path > net/iavf: support flow mark in SSE path > net/iavf: add RSS hash parsing in AVX path > net/iavf: add RSS hash parsing in SSE path >=20 > drivers/net/iavf/iavf.h | 6 + > drivers/net/iavf/iavf_ethdev.c | 8 + > drivers/net/iavf/iavf_rxtx.c | 546 ++++++++++++++++++-- > drivers/net/iavf/iavf_rxtx.h | 250 +++++++++ > drivers/net/iavf/iavf_rxtx_vec_avx2.c | 702 > +++++++++++++++++++++++++- drivers/net/iavf/iavf_rxtx_vec_sse.c | > 520 +++++++++++++++++++ > drivers/net/iavf/iavf_vchnl.c | 59 ++- > 7 files changed, 2042 insertions(+), 49 deletions(-) >=20 > -- > 2.17.1 Reviewed-by: Qi Zhang