[dpdk-dev] [PATCH v2] bus/vdev: fix scope of device list lock

[Thomas Monjalon]

The lock vdev_device_list_lock was taken before calling
"remove" function for the device.
So it prevents to remove sub-devices (as in failsafe) inside
its own "remove" function, because of a deadlock.

The lock is now only protecting the device list inside
the bus driver.

Fixes: 35f462839b69 ("bus/vdev: add lock on device list")

Signed-off-by: Thomas Monjalon <tho...@monjalon.net>
---
v2: reduce scope more by moving unlock
---
 drivers/bus/vdev/vdev.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c
index 099b9ff85..470cff46c 100644
--- a/drivers/bus/vdev/vdev.c
+++ b/drivers/bus/vdev/vdev.c
@@ -293,25 +293,24 @@ rte_vdev_uninit(const char *name)
        if (name == NULL)
                return -EINVAL;
 
-       rte_spinlock_lock(&vdev_device_list_lock);
-
        dev = find_vdev(name);
        if (!dev) {
                ret = -ENOENT;
-               goto unlock;
+               return ret;
        }
 
        ret = vdev_remove_driver(dev);
        if (ret)
-               goto unlock;
+               return ret;
 
+       rte_spinlock_lock(&vdev_device_list_lock);
        TAILQ_REMOVE(&vdev_device_list, dev, next);
+       rte_spinlock_unlock(&vdev_device_list_lock);
+
        devargs = dev->device.devargs;
        rte_devargs_remove(devargs->bus->name, devargs->name);
        free(dev);
 
-unlock:
-       rte_spinlock_unlock(&vdev_device_list_lock);
        return ret;
 }
 
-- 
2.16.2