Re: [dpdk-dev] [PATCH v3 00/10] devargs cleanup

[Thomas Monjalon <thomas@monjalon.net>]

21/03/2018 00:20, Gaetan Rivet:
> Cleanup the rte_devargs API.
> 
> This is a continuous process.
> The previous version of this patchset was dependent on changes
> on the PCI bus and the rte_bus library. Not all these changes were
> integrated. This patchset has thus been limited to elements that are
> available right now, without dependencies.
> 
> The devargs list is made private. All devargs users are meant to use them
> using the proper API. Devargs will allocate dynamic memory. With device hotplug,
> this memory is usually mishandled and lost by libraries not following the evolutions
> of rte_devargs.
> 
> Gaetan Rivet (10):
>   devargs: introduce iterator
>   devargs: introduce foreach macro
>   bus/vdev: do not reference devargs list
>   bus/pci: do not reference devargs list
>   net/vdev_netvsc: do not reference devargs list
>   test: remove devargs unit tests
>   devargs: make devargs list private
>   devargs: make parsing variadic
>   devargs: use proper namespace prefix
>   devargs: rename legacy API

Sorry, I missed this series sent one month ago.

I did few small comments.
When then they are addressed, you can rebase and add my
Acked-by: Thomas Monjalon <thomas@monjalon.net>

Thanks