From: Jeff Guo
> hi matan
>
> On 11/6/2018 2:22 PM, Matan Azrad wrote:
> > Hi Jeff
> >
> > Can you detail more in the commit log that we can understand the
> deadlock scenario. And how does this commit fix it?
>
>
> Before i add more detail in the commit log of next version, i would explain to
> you here at first here.
>
> When the device be hot-unplugged, the hot-unplug handler will be invoked
> and the device will be detached, at this time if the interrupt still not
> disable
> soon and the second
>
> remove event come again(kernel will sent pci remove event after sent uio
> remove event) , the bus->find_device will return null and return, at this
> place
> lack of an unlock.
>
> Without this unlock, it will block the next remove or add event detection. So
> it definitely need an unlock here to avoid dead lock.
>
Makes sense.
Thanks
>
> >> -----Original Message-----
> >> From: Jeff Guo <jia....@intel.com>
> >> Sent: Tuesday, November 6, 2018 8:07 AM
> >> To: konstantin.anan...@intel.com; anatoly.bura...@intel.com; Thomas
> >> Monjalon <tho...@monjalon.net>; bernard.iremon...@intel.com;
> >> jingjing...@intel.com; wenzhuo...@intel.com
> >> Cc: ferruh.yi...@intel.com; dev@dpdk.org; jia....@intel.com;
> >> helin.zh...@intel.com; Matan Azrad <ma...@mellanox.com>;
> >> shaopeng...@intel.com
> >> Subject: [PATCH 1/3] eal: fix lock issue for hot-unplug
> >>
> >> This patch will add missing unlock for hot-unplug handler, without this
> patch
> >> potential dead lock will occur when device be hotplug-in after device be
> hot-
> >> unplugged.
> >>
> >> Fixes: 0fc54536b14a ("eal: add failure handling for hot-unplug")
> >> Signed-off-by: Jeff Guo <jia....@intel.com>
> >> ---
> >> lib/librte_eal/linuxapp/eal/eal_dev.c | 12 ++++++++----
> >> 1 file changed, 8 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/lib/librte_eal/linuxapp/eal/eal_dev.c
> >> b/lib/librte_eal/linuxapp/eal/eal_dev.c
> >> index d589c69..2830c86 100644
> >> --- a/lib/librte_eal/linuxapp/eal/eal_dev.c
> >> +++ b/lib/librte_eal/linuxapp/eal/eal_dev.c
> >> @@ -258,7 +258,7 @@ dev_uev_handler(__rte_unused void *param)
> >> if (bus == NULL) {
> >> RTE_LOG(ERR, EAL, "Cannot find bus (%s)\n",
> >> busname);
> >> - return;
> >> + goto failure_handle_err;
> >> }
> >>
> >> dev = bus->find_device(NULL, cmp_dev_name, @@
> >> -266,19 +266,23 @@ dev_uev_handler(__rte_unused void *param)
> >> if (dev == NULL) {
> >> RTE_LOG(ERR, EAL, "Cannot find device (%s)
> >> on "
> >> "bus (%s)\n", uevent.devname,
> >> busname);
> >> - return;
> >> + goto failure_handle_err;
> >> }
> >>
> >> ret = bus->hot_unplug_handler(dev);
> >> - rte_spinlock_unlock(&failure_handle_lock);
> >> if (ret) {
> >> RTE_LOG(ERR, EAL, "Can not handle hot-
> >> unplug "
> >> "for device (%s)\n", dev->name);
> >> - return;
> >> }
> >> + rte_spinlock_unlock(&failure_handle_lock);
> >> }
> >> rte_dev_event_callback_process(uevent.devname,
> >> uevent.type);
> >> }
> >> +
> >> + return;
> >> +
> >> +failure_handle_err:
> >> + rte_spinlock_unlock(&failure_handle_lock);
> >> }
> >>
> >> int __rte_experimental
> >> --
> >> 2.7.4
