> On Apr 12, 2019, at 8:48 AM, Viacheslav Ovsiienko <viachesl...@mellanox.com>
> wrote:
>
> At port detach routine after calling the rte_dev_remove()
> testpmd performs some cleanup, checking the statuses of
> remaining ports and closes the appropriate ones. The port
> scanning is based on the new sibling iterator which uses the
> invalid (already closed) port for reference. This patch
> replaces the foreach iterator and uses the device object for
> reference. It is worth to note this cleanup is not needed
> for some drivers (at least for mlx5 and probably for others)
> and is preserved due to possible compatibility issues.
>
> Fixes: 13302cd5bd63 ("app/testpmd: use port sibling iterator in device
> cleanup")
>
> Signed-off-by: Viacheslav Ovsiienko <viachesl...@mellanox.com>
> ---
Acked-by: Yongseok Koh <ys...@mellanox.com>
> app/test-pmd/testpmd.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
> index aeaa74c..739f96e 100644
> --- a/app/test-pmd/testpmd.c
> +++ b/app/test-pmd/testpmd.c
> @@ -2369,8 +2369,7 @@ struct extmem_param {
> TESTPMD_LOG(ERR, "Failed to detach device %s\n", dev->name);
> return;
> }
> -
> - RTE_ETH_FOREACH_DEV_SIBLING(sibling, port_id) {
> + RTE_ETH_FOREACH_DEV_OF(sibling, dev) {
> /* reset mapping between old ports and removed device */
> rte_eth_devices[sibling].device = NULL;
> if (ports[sibling].port_status != RTE_PORT_CLOSED) {
> --
> 1.8.3.1
>
