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 1AEC4A04DE; Fri, 30 Oct 2020 17:04:11 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id E3863C9D8; Fri, 30 Oct 2020 17:04:09 +0100 (CET) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id 5E4B4C9D3 for ; Fri, 30 Oct 2020 17:04:05 +0100 (CET) IronPort-SDR: z7SeSG/ZmCZUDCKV8cHq6mjrGq1dKDiCnCGf9WtXrlMRxtsyuKBz5OkcPy2yxP8LSPuY7R0Fjw Fog+Uf/eQYIg== X-IronPort-AV: E=McAfee;i="6000,8403,9790"; a="156401470" X-IronPort-AV: E=Sophos;i="5.77,433,1596524400"; d="scan'208";a="156401470" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Oct 2020 09:03:46 -0700 IronPort-SDR: SxxsYgx4A8V4XVVakHnT9ltIO5Wib4ZqPtAPJ4pORiWS7pj10CL8rAo/uJWdt86YD1P+K2UVxT 2FRicJ/c/PUA== X-IronPort-AV: E=Sophos;i="5.77,433,1596524400"; d="scan'208";a="537112178" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.252.23.197]) ([10.252.23.197]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Oct 2020 09:03:43 -0700 To: "Zhang, Qi Z" , "Guo, Jia" , "Wu, Jingjing" , "Xing, Beilei" Cc: "dev@dpdk.org" , "Wang, Haiyue" , "Richardson, Bruce" References: <20200909025415.6185-1-jia.guo@intel.com> <20201030084030.66339-1-jia.guo@intel.com> <39b05c19-794d-4ade-2fa2-34e36382b716@intel.com> <75494659f321446ab285df221467df38@intel.com> From: Ferruh Yigit Message-ID: Date: Fri, 30 Oct 2020 16:03:40 +0000 MIME-Version: 1.0 In-Reply-To: <75494659f321446ab285df221467df38@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v12] net/iavf: support flex desc metadata extraction 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 10/30/2020 11:14 AM, Zhang, Qi Z wrote: > > >> -----Original Message----- >> From: Ferruh Yigit >> Sent: Friday, October 30, 2020 6:52 PM >> To: Guo, Jia ; Wu, Jingjing ; >> Zhang, Qi Z ; Xing, Beilei >> Cc: dev@dpdk.org; Wang, Haiyue ; Richardson, >> Bruce >> Subject: Re: [PATCH v12] net/iavf: support flex desc metadata extraction >> >> On 10/30/2020 8:40 AM, Jeff Guo wrote: >>> Enable metadata extraction for flexible descriptors in AVF, that would >>> allow network function directly get metadata without additional >>> parsing which would reduce the CPU cost for VFs. The enabling metadata >>> extractions involve the metadata of VLAN/IPv4/IPv6/IPv6-FLOW/TCP/MPLS >>> flexible descriptors, and the VF could negotiate the capability of the >>> flexible descriptor with PF and correspondingly configure the specific >>> offload at receiving queues. >>> >>> Signed-off-by: Jeff Guo >>> Acked-by: Haiyue Wang >>> --- >>> v12: >>> refine doc to be briefly >>> >>> v11: >>> update doc in .map and .rst >>> >>> v10: >>> delete the makefile configure and rename the dynamic mbuf name >>> >>> v9: >>> change the undef config >>> >>> v8: >>> rebase patch for apply issue >>> >>> v7: >>> clean some useless and add doc >>> >>> v6: >>> rebase patch >>> >>> v5: >>> remove ovs configure since ovs is not protocol extraction >>> >>> v4: >>> add flex desc type in rx queue for handling vector path handle ovs >>> flex type >>> >>> v3: >>> export these global symbols into .map >>> >>> v2: >>> remove makefile change and modify the rxdid handling >>> --- >>> doc/guides/nics/intel_vf.rst | 4 + >>> doc/guides/rel_notes/release_20_11.rst | 6 + >>> drivers/net/iavf/iavf.h | 24 +- >>> drivers/net/iavf/iavf_ethdev.c | 394 >> ++++++++++++++++++++++++ >>> drivers/net/iavf/iavf_rxtx.c | 252 +++++++++++++-- >>> drivers/net/iavf/iavf_rxtx.h | 169 +++++----- >>> drivers/net/iavf/iavf_rxtx_vec_common.h | 3 + >>> drivers/net/iavf/iavf_vchnl.c | 22 +- >>> drivers/net/iavf/meson.build | 2 + >>> drivers/net/iavf/rte_pmd_iavf.h | 250 +++++++++++++++ >> >> We should add this public header to the API documentation, if that is the only >> change I can do while merging. Something like: > > Yes, it think this is the only change, thanks. > Added following while merging: diff --git a/doc/api/doxy-api-index.md b/doc/api/doxy-api-index.md index c629b5fea9..9c9899c45a 100644 --- a/doc/api/doxy-api-index.md +++ b/doc/api/doxy-api-index.md @@ -44,6 +44,7 @@ The public API headers are grouped by topics: [ixgbe] (@ref rte_pmd_ixgbe.h), [i40e] (@ref rte_pmd_i40e.h), [ice] (@ref rte_pmd_ice.h), + [iavf] (@ref rte_pmd_iavf.h), [ioat] (@ref rte_ioat_rawdev.h), [bnxt] (@ref rte_pmd_bnxt.h), [dpaa] (@ref rte_pmd_dpaa.h), diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in index 567fe62f8f..6eeabba9e1 100644 --- a/doc/api/doxy-api.conf.in +++ b/doc/api/doxy-api.conf.in @@ -13,6 +13,7 @@ INPUT = @TOPDIR@/doc/api/doxy-api-index.md \ @TOPDIR@/drivers/net/dpaa \ @TOPDIR@/drivers/net/dpaa2 \ @TOPDIR@/drivers/net/i40e \ + @TOPDIR@/drivers/net/iavf \ @TOPDIR@/drivers/net/ice \ @TOPDIR@/drivers/net/ixgbe \ @TOPDIR@/drivers/net/mlx5 \