Hi all Since there is not agreement for testpmd to be ownership aware by using the new ownership mechanism, I think we can drop testpmd patch for now(app/testpmd: adjust ethdev port ownership).
Maybe we can add example application to use this API in future. Thanks! From: Matan Azrad > Add ownership mechanism to DPDK Ethernet devices to avoid multiple > management of a device by different DPDK entities. > The port ownership mechanism is a good point to redefine the > synchronization rules in ethdev: > > 1. The port allocation and port release synchronization will be > managed by ethdev. > 2. The port usage synchronization will be managed by the port > owner. > 3. The port ownership synchronization will be managed by ethdev. > 4. DPDK entity which want to use a port safely must take ownership > before. > > > V2: > Synchronize ethdev port creation. > Synchronize port ownership mechanism. > Rename owner remove API to rte_eth_dev_owner_unset. > Remove "ethdev: free a port by a dedicated API" patch - passed to another > series. > Add "ethdev: fix port data reset timing" patch. > Cahnge owner get API to return int value and to pass copy of the owner > structure. > Adjust testpmd to the improved owner get API. > Adjust documentations. > > V3: > Change RTE_ETH_FOREACH_DEV iterator to skip owned ports(Gaetan > suggestion). > Prevent goto in set\unset APIs by adding internal API - this also adds reuse > of > code(Konstantin suggestion). > Group all the shared processes variables in one struct to allow easy > allocation > of it(Konstantin suggestion). > Take owner name truncation as warning and not as error(Konstantin > suggestion). > Mark the new APIs as EXPERIMENTAL. > Rebase on top of master_net_mlx. > Rebase on top of "[PATCH v6 0/6] Fail-safe\ethdev: fix removal handling lack" > series. > Rebase on top of "[PATCH v5 0/8] Introduce virtual driver for Hyper-V/Azure > platforms" . > Add "ethdev: fix used portid allocation" patch suggested y Konstantin. > > v4: > Share => shared in ethdev patches(Thomas suggestion). > Rephase some code comments(Thomas suggestion). > Fix compilation issue caused by wrong rebase with "fix used portid allocation" > patch. > Add assert check for the correct port state to above fix patch. > > V5: > Use defferent print message type as Ferruh suggested. > Fix the name parameter description in set\unset APIs(Ferruh suggestion). > Rebase on top of 18.02-rc1. > Fix issue: ownership API must check that the shared data was allocated > before using the shared ownership lock(relevant when no port was created). > > Matan Azrad (7): > ethdev: fix port data reset timing > ethdev: fix used portid allocation > ethdev: add port ownership > ethdev: synchronize port allocation > net/failsafe: free an eth port by a dedicated API > net/failsafe: use ownership mechanism to own ports > app/testpmd: adjust ethdev port ownership > > app/test-pmd/cmdline.c | 89 +++++------ > app/test-pmd/cmdline_flow.c | 2 +- > app/test-pmd/config.c | 37 ++--- > app/test-pmd/parameters.c | 4 +- > app/test-pmd/testpmd.c | 63 +++++--- > app/test-pmd/testpmd.h | 3 + > doc/guides/prog_guide/poll_mode_drv.rst | 14 +- > drivers/net/failsafe/failsafe.c | 7 + > drivers/net/failsafe/failsafe_eal.c | 16 ++ > drivers/net/failsafe/failsafe_ether.c | 2 +- > drivers/net/failsafe/failsafe_private.h | 2 + > lib/librte_ether/rte_ethdev.c | 267 > +++++++++++++++++++++++++++----- > lib/librte_ether/rte_ethdev.h | 115 +++++++++++++- > lib/librte_ether/rte_ethdev_core.h | 2 + > lib/librte_ether/rte_ethdev_version.map | 6 + > 15 files changed, 486 insertions(+), 143 deletions(-) > > -- > 1.8.3.1
