diff mbox

[dpdk-dev] igb_uio: reset PCI function on process exit

Message ID 3336123.hs2y1kGb2P@polaris (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK

Commit Message

Gregory Etelson June 6, 2017, 6:03 a.m. UTC 
  re-bind PCI function to igb_uio
after process termination

tested adapters: Intel 10G, Intel 40G

known issues: ENA adapters on AWS I3 instances fail to reset PCI function


Signed-off-by: Gregory Etelson <gregory@weka.io>
---
 lib/librte_eal/linuxapp/igb_uio/igb_uio.c |   12 ++++++++++++
 1 files changed, 12 insertions(+), 0 deletions(-)

Comments

Ferruh Yigit Oct. 6, 2017, 9:36 p.m. UTC | #1 
On 6/6/2017 7:03 AM, Gregory Etelson wrote:
> re-bind PCI function to igb_uio
> after process termination
> tested adapters: Intel 10G, Intel 40G
> known issues: ENA adapters on AWS I3 instances fail to reset PCI function
>
> Signed-off-by: Gregory Etelson <gregory@weka.io>

Shijith's patch [1] applied in 17.08 superseded this patch, updating it
in patchwork.

[1]
Commit: b58eedfc7dd5 ("igb_uio: issue FLR during open and release of
device file")
diff mbox

Patch

diff --git a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c b/lib/librte_eal/linuxapp/igb_uio/igb_uio.c
index b9d427c..f09ca0d 100644
--- a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c
+++ b/lib/librte_eal/linuxapp/igb_uio/igb_uio.c
@@ -317,6 +317,17 @@  struct rte_uio_pci_dev {
 	return (iom != 0 || iop != 0) ? ret : -ENOENT;
 }
 
+static int
+igbuio_pci_release(struct uio_info *info, struct inode *inode)
+{
+	struct rte_uio_pci_dev *udev = info->priv;
+	struct pci_dev *dev = udev->pdev;
+
+	int ret = __pci_reset_function(dev);
+	dev_info(&udev->pdev->dev, "PCI reset %s\n", !ret ? "done" : "failed");
+	return 0;
+}
+
 #if LINUX_VERSION_CODE < KERNEL_VERSION(3, 8, 0)
 static int __devinit
 #else
@@ -372,6 +383,7 @@  struct rte_uio_pci_dev {
 	udev->info.version = "0.1";
 	udev->info.handler = igbuio_pci_irqhandler;
 	udev->info.irqcontrol = igbuio_pci_irqcontrol;
+	udev->info.release = igbuio_pci_release;
 #ifdef CONFIG_XEN_DOM0
 	/* check if the driver run on Xen Dom0 */
 	if (xen_initial_domain())