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

Thomas Monjalon thomas at monjalon.net
Tue May 22 16:38:06 CEST 2018


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 recursive.
> > 
> > Fixes: 35f462839b69 ("bus/vdev: add lock on device list")
> > 
> > Suggested-by: Anatoly Burakov <anatoly.burakov at intel.com>
> > Signed-off-by: Thomas Monjalon <thomas at monjalon.net>
> > ---
> > v3: try recursive lock
> > WARNING: not yet tested!
> > ---
> 
> LGTM
> 
> Acked-by: Anatoly Burakov <anatoly.burakov at intel.com>

Tested-by: Matan Azrad <matan at mellanox.com>

Applied




More information about the dev mailing list