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 7B46DA0583; Fri, 20 Mar 2020 06:37:03 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 192BA2BB9; Fri, 20 Mar 2020 06:37:02 +0100 (CET) Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id DA713F90 for ; Fri, 20 Mar 2020 06:36:59 +0100 (CET) IronPort-SDR: pDw7wTo7T/RCEPQT3x9cvEMbr9TtZXkA7zpMjdu1Lykl9EXjUZD8NOC45x9faq1tt+mLEm+M/l CfLBvpT1H2gA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Mar 2020 22:36:58 -0700 IronPort-SDR: nC4wINVhRiLX6nQdX3AFRbZiWEVB66DqMumaYT4oyRN+/z4ZIGMSwBPAW/WYlDMc1HpNSItqXj BmaqChZOq2GA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,283,1580803200"; d="scan'208";a="263966277" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga002.jf.intel.com with ESMTP; 19 Mar 2020 22:36:58 -0700 Received: from fmsmsx115.amr.corp.intel.com (10.18.116.19) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 19 Mar 2020 22:35:49 -0700 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx115.amr.corp.intel.com (10.18.116.19) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 19 Mar 2020 22:35:48 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.137]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.201]) with mapi id 14.03.0439.000; Fri, 20 Mar 2020 13:35:45 +0800 From: "Zhang, Qi Z" To: Thomas Monjalon , "dev@dpdk.org" CC: "rahul.lakkireddy@chelsio.com" , "Wang, Xiao W" , "xavier.huwei@huawei.com" , "Xing, Beilei" , "Lu, Wenzhuo" , "Yang, Qiming" , "Ananyev, Konstantin" , "Yigit, Ferruh" , "jerinj@marvell.com" , "rmody@marvell.com" , "shshaikh@marvell.com" , "maxime.coquelin@redhat.com" , "Ye, Xiaolong" Thread-Topic: [PATCH 0/3] refresh NIC features matrix Thread-Index: AQHV9/kTpUL0ShYpnECecwcPtDPVJ6hQ/IFQ Date: Fri, 20 Mar 2020 05:35:44 +0000 Message-ID: <039ED4275CED7440929022BC67E70611547E53BE@SHSMSX103.ccr.corp.intel.com> References: <20200311230136.63452-1-thomas@monjalon.net> In-Reply-To: <20200311230136.63452-1-thomas@monjalon.net> 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 0/3] refresh NIC features matrix 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" Hi Thomas: > -----Original Message----- > From: Thomas Monjalon > Sent: Thursday, March 12, 2020 7:02 AM > To: dev@dpdk.org > Cc: rahul.lakkireddy@chelsio.com; Zhang, Qi Z ; Wan= g, > Xiao W ; xavier.huwei@huawei.com; Xing, Beilei > ; Lu, Wenzhuo ; Yang, Qiming > ; Ananyev, Konstantin > ; Yigit, Ferruh ; > jerinj@marvell.com; rmody@marvell.com; shshaikh@marvell.com; > maxime.coquelin@redhat.com; Ye, Xiaolong > Subject: [PATCH 0/3] refresh NIC features matrix >=20 > This series aims to clean-up the big table of ethdev features: > http://doc.dpdk.org/guides/nics/overview.html#id1 >=20 > We could reorganize the information in this table, maybe split it or add/= remove > some rows. > Before going to such reorganization, we should clean it up. >=20 > The first patch is fixing the look & size of the table with recent sphinx= . >=20 > The second and third patches are removing 8 columns which are clearly > unneeded: > - bnx2x_vf > - bonding > - kni > - nfp_vf > - null > - ring > - softnic > - vdev_netvsc >=20 > More columns can be removed by merging PF/VF and vector datapaths. > If a feature cannot be supported in all cases, it should be marked as par= tially > supported (P). > If a feature is PF-specific (like flow control), that's OK to mark it ful= ly supported > because it's obviously impossible for VF. > There are also some features which were probably marked in some columns > and missed in its VF or vector counterpart. > Please work to merge and drop these 16 columns: > - cxgbevf > - fm10k_vf > - hns3_vf > - i40e_vec > - i40e_vf > - i40e_vf_vec > - iavf_vec > - ice_vec > - igb_vf > - ixgbe_vec > - ixgbe_vf > - ixgbe_vf_vec > - octeontx2_vec > - octeontx2_vf > - qede_vf > - virtio_vec >=20 > The total gain is to reduce the table size from 71 to 47 columns. I agree to remove all the column with "vec", since vector PMD can be regard= ed as a feature of the a PMD. But I'm not sure if it is a good idea to merge VF and PF into one column. >From my view, for intel device, VF driver and PF driver just share the code= , but they actually are running at two different context. And likely they will support different feature, merge into one column may c= onfuse our customer if they want to understand what exactly the PMD support= . Thanks Qi >=20 > Please send patches for Chelsio, Huawei, Intel, Marvell and virtio. >=20 >=20 > Thomas Monjalon (3): > doc: fix matrix CSS for recent sphinx > doc: remove empty columns from NIC features matrix > doc: remove similar columns from NIC features matrix >=20 > doc/guides/conf.py | 16 +++++++++---- > doc/guides/nics/features/bnx2x.ini | 1 + > doc/guides/nics/features/bnx2x_vf.ini | 18 --------------- > doc/guides/nics/features/bonding.ini | 6 ----- > doc/guides/nics/features/kni.ini | 7 ------ > doc/guides/nics/features/nfp_vf.ini | 29 ------------------------ > doc/guides/nics/features/null.ini | 6 ----- > doc/guides/nics/features/ring.ini | 6 ----- > doc/guides/nics/features/softnic.ini | 9 -------- > doc/guides/nics/features/vdev_netvsc.ini | 12 ---------- > 10 files changed, 13 insertions(+), 97 deletions(-) delete mode 100644 > doc/guides/nics/features/bnx2x_vf.ini > delete mode 100644 doc/guides/nics/features/bonding.ini > delete mode 100644 doc/guides/nics/features/kni.ini delete mode 100644 > doc/guides/nics/features/nfp_vf.ini > delete mode 100644 doc/guides/nics/features/null.ini delete mode 100644 > doc/guides/nics/features/ring.ini delete mode 100644 > doc/guides/nics/features/softnic.ini > delete mode 100644 doc/guides/nics/features/vdev_netvsc.ini >=20 > -- > 2.25.1