Re: [dpdk=-dev][PATCH v8 0/9] switching devices representation

[Ferruh Yigit <ferruh.yigit@intel.com>]

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 <ferruh.yigit@intel.com>