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 D6018A09D9; Wed, 11 Nov 2020 17:01:04 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 017AE31FC; Wed, 11 Nov 2020 17:01:03 +0100 (CET) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id DF2F9F64 for ; Wed, 11 Nov 2020 17:01:01 +0100 (CET) IronPort-SDR: TRpjrkzitvnf4yLXrRgx5OKRjBg56B2vG0nXTv4pAJvZiEkjFwwizYyWq+XocGtcSZh110R37j NAMHXMxDVcqQ== X-IronPort-AV: E=McAfee;i="6000,8403,9801"; a="188148800" X-IronPort-AV: E=Sophos;i="5.77,469,1596524400"; d="scan'208";a="188148800" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Nov 2020 08:00:59 -0800 IronPort-SDR: yCw9mhcc1ey2bBjPavYfmxMyH9zmrllMr7Vb9SMaR4iFdzrXK6TY6o4+9RoEyPDJZ3/HZh8Tag QyONBTgIeJlQ== X-IronPort-AV: E=Sophos;i="5.77,469,1596524400"; d="scan'208";a="473901187" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.241.123]) ([10.213.241.123]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Nov 2020 08:00:57 -0800 To: Jiawen Wu , dev@dpdk.org References: <20201111064936.768604-1-jiawenwu@trustnetic.com> From: Ferruh Yigit Message-ID: <45280e07-d371-875d-feeb-1e26c90ce126@intel.com> Date: Wed, 11 Nov 2020 16:00:54 +0000 MIME-Version: 1.0 In-Reply-To: <20201111064936.768604-1-jiawenwu@trustnetic.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 00/37] net: add txgbe PMD part 2 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/11/2020 6:48 AM, Jiawen Wu wrote: > This patch adds the rest of txgbe PMD for its PF part. > Support include flow API, traffic manager, macsec and ipsec. > > v2: > - Rebase on next-net > > Jiawen Wu (37): > net/txgbe: add ntuple filter init and uninit > net/txgbe: support ntuple filter add and delete > net/txgbe: add ntuple parse rule > net/txgbe: support ntuple filter remove operaion > net/txgbe: support ethertype filter add and delete > net/txgbe: add ethertype parse rule > net/txgbe: support syn filter add and delete > net/txgbe: add syn filter parse rule > net/txgbe: add L2 tunnel filter init and uninit > net/txgbe: config L2 tunnel filter with e-tag > net/txgbe: support L2 tunnel filter add and delete > net/txgbe: add L2 tunnel filter parse rule > net/txgbe: add FDIR filter init and uninit > net/txgbe: configure FDIR filter > net/txgbe: support FDIR add and delete operations > net/txgbe: add FDIR parse normal rule > net/txgbe: add FDIR parse tunnel rule > net/txgbe: add FDIR restore operation > net/txgbe: add RSS filter parse rule > net/txgbe: add RSS filter restore operation > net/txgbe: add filter list init and uninit > net/txgbe: add generic flow API > net/txgbe: add flow API create function > net/txgbe: add flow API destroy function > net/txgbe: add flow API flush function > net/txgbe: support UDP tunnel port add and delete > net/txgbe: add TM configuration init and uninit > net/txgbe: add TM capabilities get operation > net/txgbe: support TM shaper profile add and delete > net/txgbe: support TM node add and delete > net/txgbe: add TM hierarchy commit > net/txgbe: add macsec setting > net/txgbe: add IPsec context creation > net/txgbe: add security session create operation > net/txgbe: support security session destroy > net/txgbe: add security offload in Rx and Tx process > net/txgbe: add security type in flow action Hi Jiawen, There are multiple build errors on patch by patch build, can you please check it, the expectation is after each commit build and functionality shouldn't be broken. I put some comments to the patches, with a change request how to construct the patchset. All these new features are not documented in the driver documentation, it can be documented with more details. And the patchset is loaded, it has many new features, it is up to you but instead of adding all to -rc4, I suggest postponing this set to next release and use longer time for better implementation. Thanks ferruh