From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [dpdk=-dev][PATCH v8 0/9] switching devices representation Date: Thu, 26 Apr 2018 17:35:44 +0100 Message-ID: References: <20180416130605.6509-1-declan.doherty@intel.com> <20180426104105.18342-1-declan.doherty@intel.com> <75c7f252-9761-d2be-50ef-644638205703@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Adrien Mazarguil , Thomas Monjalon , Shahaf Shuler , Konstantin Ananyev To: Declan Doherty , dev@dpdk.org Return-path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id EF0CA8E59 for ; Thu, 26 Apr 2018 18:35:47 +0200 (CEST) In-Reply-To: <75c7f252-9761-d2be-50ef-644638205703@intel.com> Content-Language: en-US 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 4/26/2018 5:24 PM, Ferruh Yigit wrote: > On 4/26/2018 11:40 AM, Declan Doherty wrote: >> This patchset follows on from the port rerpesentor patchsets and the >> community discussion that resulted. It outlines the model for >> representing and controlling switching capable devices in a new >> programmer's guide entry based upon the excellent summary by >> Adrien Mazarguil in >> (http://dpdk.org/ml/archives/dev/2018-March/092513.html). >> >> The next patches introduce changes to librte_ether to: >> 1, support the definition of a switch domain and make it public to >> application through the rte_eth_dev_info structure. >> 2, Add generic ethdev create/destroy APIs to facilitate and generalise the >> creation of ethdev's on different bus types. >> 3, Add ethdev attribute to dev_flags to specify that a port is a >> representor port and make public through the rte_eth_dev_info >> structure. >> 4, Add devargs parsing for generic eth_devargs to facilate parsing in >> NET PMDs. This will be refactored to take account of the changes in >> (http://dpdk.org/ml/archives/dev/2018-March/092513.html) >> 5, Add new API to allocate switch domain ids to devices which support >> this feature. >> >> This patchset also includes the enablement of vf port representor for ixgbe >> and i40e PF devices. >> >> V8: >> - add detailed descriptions to switch information structures >> - fix err condition checking ethdev create helper function >> - fix devargs memory leak and error checking + add documentation on >> ethdev args. >> - remove rte_eth_switch_domains structure from export items. >> >> V7: >> >> This patch address the following changes: >> - fixes in documentation patch >> - changes the default value of switch domain id to be INVALID to allow >> applications to easily identify devices which can/cannot support the >> concept. Updates the switch information available through the >> rte_eth_dev_info structure. >> - remove the rte_ethdev_representor.h header and leave representor >> specific initialisation to driver >> - add new APIs for allocating and freeing switch domain identifier to >> enable PMDs to have unique switch domaind ids without the ethdev >> infrastructure placing any restriction on how theses are managed by >> devices. >> - bug fix in ethdev args parsing code. >> >> Declan Doherty (8): >> doc: add switch representation documentation >> ethdev: add switch identifier parameter to port >> ethdev: add generic create/destroy ethdev APIs >> ethdev: Add port representor device flag >> app/testpmd: add port name to device info >> ethdev: add switch domain allocator >> net/i40e: add support for representor ports >> net/ixgbe: add support for representor ports >> >> Remy Horton (1): >> ethdev: add common devargs parser > > > For series, > Reviewed-by: Ferruh Yigit Series applied to dpdk-next-net/master, thanks.