igb_uio: fix unexpected removal for hot-unplug
authorJeff Guo <jia.guo@intel.com>
Thu, 18 Oct 2018 06:27:15 +0000 (14:27 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Thu, 25 Oct 2018 00:11:22 +0000 (02:11 +0200)
When a device is hot-unplugged, pci_remove will be invoked unexpectedly
before pci_release, it will caused kernel hung issue which will throw the
error info of "Trying to free already-free IRQ XXX". And on the other hand,
if pci_remove before pci_release, the interrupt will not got chance to be
disabled. So this patch aim to fix this issue by adding pci_release call
in pci_remove, it will gurranty that all pci clean up will be done before
pci removal.

Signed-off-by: Jeff Guo <jia.guo@intel.com>
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
kernel/linux/igb_uio/igb_uio.c

index fede66c..3cf394b 100644 (file)
@@ -570,6 +570,8 @@ igbuio_pci_remove(struct pci_dev *dev)
 {
        struct rte_uio_pci_dev *udev = pci_get_drvdata(dev);
 
+       igbuio_pci_release(&udev->info, NULL);
+
        sysfs_remove_group(&dev->dev.kobj, &dev_attr_grp);
        uio_unregister_device(&udev->info);
        igbuio_pci_release_iomem(&udev->info);