Re: [dpdk-dev] [PATCH v7 0/7] replace attach/detach functions

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

On 10/23/2018 9:28 AM, Thomas Monjalon wrote:
> The functions for EAL attach/detach had already some replacements,
> so they are removed.
> The functions for ethdev attach/detach are removed and replaced
> thanks to a new ethdev iterator working with devargs.
> 
>     rte_eth_dev_attach(devargs, &port_id)
> is replaced by:
>     rte_dev_probe(devargs);
>     RTE_ETH_FOREACH_MATCHING_DEV(port_id, devargs, &iterator) {
>         do what you want with the new port_id
>     }
> 
> The biggest benefit is to be able to manage devargs string
> matching several ports to probe.
> 
> 
> Changes in v7:
>   - remove use of eth_dev->data->name after detach in testpmd
> 
> Changes in v6:
>   - improve testpmd logs
>   - update testpmd port_status when detaching multiple ports
> 
> Changes in v5:
>   - check double detach in testpmd
> 
> Changes in v4:
>   - fix compilation (due to stricter checks in v3)
> 
> Changes in v3 - after Andrew's review:
>   - coding rules compliance
>   - stricter checks for snprintf
>   - rte_eth_iterator_free() becomes rte_eth_iterator_cleanup()
>   - add rte_eth_iterator_cleanup in .map
> 
> Changes in v2 - after Andrew's review:
>   - return only 0 or -1 in vdev_dev_match()
>   - fix freeing of strings in the iterator
>   - add rte_eth_iterator_free()
>   - tolerate extra parameters from old syntax
>   - add more comments
>   - separate contributing guide patch
> 
> 
> Thomas Monjalon (7):
>   bus/vdev: add iteration filter on name
>   ethdev: add iterator to match devargs input
>   ethdev: allow iterating with pure class filter
>   doc: replace doxygen example in contribution guide
>   ethdev: remove deprecated attach/detach functions
>   eal: remove deprecated attach/detach functions
>   app/testpmd: check not detaching device twice

Series applied to dpdk-next-net/master, thanks.

(except testpmd patch 7/7)