Re: [dpdk-dev] [PATCH v3] bus/vdev: replace device list lock by a recursive one

2018-05-22 Thread Thomas Monjalon
22/05/2018 15:34, Burakov, Anatoly: > On 22-May-18 12:37 PM, Thomas Monjalon wrote: > > A device like failsafe can manage sub-devices. > > When removing such device, it removes its sub-devices > > and try to take the same vdev_device_list_lock. > > It was causing a deadlock because the lock was not

Re: [dpdk-dev] [PATCH v3] bus/vdev: replace device list lock by a recursive one

2018-05-22 Thread Burakov, Anatoly
On 22-May-18 12:37 PM, Thomas Monjalon wrote: A device like failsafe can manage sub-devices. When removing such device, it removes its sub-devices and try to take the same vdev_device_list_lock. It was causing a deadlock because the lock was not recursive. Fixes: 35f462839b69 ("bus/vdev: add loc

Re: [dpdk-dev] [PATCH v3] bus/vdev: replace device list lock by a recursive one

2018-05-22 Thread Matan Azrad
From: Thomas Monjalon > A device like failsafe can manage sub-devices. > When removing such device, it removes its sub-devices and try to take the > same vdev_device_list_lock. > It was causing a deadlock because the lock was not recursive. > > Fixes: 35f462839b69 ("bus/vdev: add lock on device

[dpdk-dev] [PATCH v3] bus/vdev: replace device list lock by a recursive one

2018-05-22 Thread Thomas Monjalon
A device like failsafe can manage sub-devices. When removing such device, it removes its sub-devices and try to take the same vdev_device_list_lock. It was causing a deadlock because the lock was not recursive. Fixes: 35f462839b69 ("bus/vdev: add lock on device list") Suggested-by: Anatoly Burako