DPDK: Fix a crash in igb_uio driver when the device is removed.

This crash happens because the device still has MSI configured,
the fix is to free the IRQ. Adding pending further investigation.

Change-Id: Ifcd3404fcc25fe27dcccc67d0dcf9d8f0ec18582
Signed-off-by: Todd Foggoa (tfoggoa) <tfoggoa@cisco.com>
Signed-off-by: Ray Kinsella <ray.kinsella@intel.com>
This commit is contained in:
Ray Kinsella
2016-08-10 12:48:06 +01:00
committed by Damjan Marion
parent 348c3b70a4
commit 2bd44b7cdd

View File

@ -0,0 +1,38 @@
From 95c2d549d8d123aac37a372580122f1b043c6165 Mon Sep 17 00:00:00 2001
From: Ray Kinsella <ray.kinsella@intel.com>
Date: Wed, 10 Aug 2016 11:59:07 +0100
Subject: [PATCH] UIO: Fix a crash in igb_uio driver when the device is
removed.
This crash happens because the device still has MSI configured,
the fix is to free the IRQ.
Signed-off-by: Todd Foggoa (tfoggoa) <tfoggoa@cisco.com>
Signed-off-by: Ray Kinsella <ray.kinsella@intel.com>
---
lib/librte_eal/linuxapp/igb_uio/igb_uio.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c b/lib/librte_eal/linuxapp/igb_uio/igb_uio.c
index df41e45..69873e7 100644
--- a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c
+++ b/lib/librte_eal/linuxapp/igb_uio/igb_uio.c
@@ -442,8 +442,15 @@ static void
igbuio_pci_remove(struct pci_dev *dev)
{
struct rte_uio_pci_dev *udev = pci_get_drvdata(dev);
+ struct uio_info *info = pci_get_drvdata(dev);
sysfs_remove_group(&dev->dev.kobj, &dev_attr_grp);
+
+ if (info->irq && (info->irq != UIO_IRQ_CUSTOM)){
+ free_irq(info->irq, info->uio_dev);
+ info->irq = UIO_IRQ_NONE;
+ }
+
uio_unregister_device(&udev->info);
igbuio_pci_release_iomem(&udev->info);
if (udev->mode == RTE_INTR_MODE_MSIX)
--
1.9.1