igb_uio: remove device reset in release
[dpdk.git] / lib / librte_eal / linuxapp / igb_uio / igb_uio.c
index da71e05..a3a98c1 100644 (file)
@@ -34,9 +34,6 @@
 #include <linux/version.h>
 #include <linux/slab.h>
 
-#ifdef CONFIG_XEN_DOM0
-#include <xen/xen.h>
-#endif
 #include <rte_pci_dev_features.h>
 
 #include "compat.h"
@@ -52,7 +49,6 @@ struct rte_uio_pci_dev {
 
 static char *intr_mode;
 static enum rte_intr_mode igbuio_intr_mode_preferred = RTE_INTR_MODE_MSIX;
-
 /* sriov sysfs */
 static ssize_t
 show_max_vfs(struct device *dev, struct device_attribute *attr,
@@ -93,6 +89,75 @@ static const struct attribute_group dev_attr_grp = {
        .attrs = dev_attrs,
 };
 
+#ifndef HAVE_PCI_MSI_MASK_IRQ
+/*
+ * It masks the msix on/off of generating MSI-X messages.
+ */
+static void
+igbuio_msix_mask_irq(struct msi_desc *desc, s32 state)
+{
+       u32 mask_bits = desc->masked;
+       unsigned int offset = desc->msi_attrib.entry_nr * PCI_MSIX_ENTRY_SIZE +
+                                               PCI_MSIX_ENTRY_VECTOR_CTRL;
+
+       if (state != 0)
+               mask_bits &= ~PCI_MSIX_ENTRY_CTRL_MASKBIT;
+       else
+               mask_bits |= PCI_MSIX_ENTRY_CTRL_MASKBIT;
+
+       if (mask_bits != desc->masked) {
+               writel(mask_bits, desc->mask_base + offset);
+               readl(desc->mask_base);
+               desc->masked = mask_bits;
+       }
+}
+
+/*
+ * It masks the msi on/off of generating MSI messages.
+ */
+static void
+igbuio_msi_mask_irq(struct pci_dev *pdev, struct msi_desc *desc, int32_t state)
+{
+       u32 mask_bits = desc->masked;
+       u32 offset = desc->irq - pdev->irq;
+       u32 mask = 1 << offset;
+
+       if (!desc->msi_attrib.maskbit)
+               return;
+
+       if (state != 0)
+               mask_bits &= ~mask;
+       else
+               mask_bits |= mask;
+
+       if (mask_bits != desc->masked) {
+               pci_write_config_dword(pdev, desc->mask_pos, mask_bits);
+               desc->masked = mask_bits;
+       }
+}
+
+static void
+igbuio_mask_irq(struct pci_dev *pdev, enum rte_intr_mode mode, s32 irq_state)
+{
+       struct msi_desc *desc;
+       struct list_head *msi_list;
+
+#ifdef HAVE_MSI_LIST_IN_GENERIC_DEVICE
+       msi_list = &pdev->dev.msi_list;
+#else
+       msi_list = &pdev->msi_list;
+#endif
+
+       if (mode == RTE_INTR_MODE_MSIX) {
+               list_for_each_entry(desc, msi_list, list)
+                       igbuio_msix_mask_irq(desc, irq_state);
+       } else if (mode == RTE_INTR_MODE_MSI) {
+               list_for_each_entry(desc, msi_list, list)
+                       igbuio_msi_mask_irq(pdev, desc, irq_state);
+       }
+}
+#endif
+
 /**
  * This is the irqcontrol callback to be registered to uio_info.
  * It can be used to disable/enable interrupt from user space processes.
@@ -112,10 +177,8 @@ igbuio_pci_irqcontrol(struct uio_info *info, s32 irq_state)
        struct rte_uio_pci_dev *udev = info->priv;
        struct pci_dev *pdev = udev->pdev;
 
-#ifdef HAVE_IRQ_DATA
+#ifdef HAVE_PCI_MSI_MASK_IRQ
        struct irq_data *irq = irq_get_irq_data(udev->info.irq);
-#else
-       unsigned int irq = udev->info.irq;
 #endif
 
        pci_cfg_access_lock(pdev);
@@ -127,10 +190,7 @@ igbuio_pci_irqcontrol(struct uio_info *info, s32 irq_state)
                else
                        pci_msi_mask_irq(irq);
 #else
-               if (irq_state == 1)
-                       unmask_msi_irq(irq);
-               else
-                       mask_msi_irq(irq);
+               igbuio_mask_irq(pdev, udev->mode, irq_state);
 #endif
        }
 
@@ -147,157 +207,22 @@ igbuio_pci_irqcontrol(struct uio_info *info, s32 irq_state)
  * If yes, disable it here and will be enable later.
  */
 static irqreturn_t
-igbuio_pci_irqhandler(int irq, struct uio_info *info)
+igbuio_pci_irqhandler(int irq, void *dev_id)
 {
-       struct rte_uio_pci_dev *udev = info->priv;
+       struct rte_uio_pci_dev *udev = (struct rte_uio_pci_dev *)dev_id;
+       struct uio_info *info = &udev->info;
 
        /* Legacy mode need to mask in hardware */
        if (udev->mode == RTE_INTR_MODE_LEGACY &&
            !pci_check_and_mask_intx(udev->pdev))
                return IRQ_NONE;
 
+       uio_event_notify(info);
+
        /* Message signal mode, no share IRQ and automasked */
        return IRQ_HANDLED;
 }
 
-/**
- * This gets called while opening uio device file.
- */
-static int
-igbuio_pci_open(struct uio_info *info, struct inode *inode)
-{
-       struct rte_uio_pci_dev *udev = info->priv;
-       struct pci_dev *dev = udev->pdev;
-
-       pci_reset_function(dev);
-
-       /* set bus master, which was cleared by the reset function */
-       pci_set_master(dev);
-
-       return 0;
-}
-
-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;
-
-       /* stop the device from further DMA */
-       pci_clear_master(dev);
-
-       pci_reset_function(dev);
-
-       return 0;
-}
-
-#ifdef CONFIG_XEN_DOM0
-static int
-igbuio_dom0_mmap_phys(struct uio_info *info, struct vm_area_struct *vma)
-{
-       int idx;
-
-       idx = (int)vma->vm_pgoff;
-       vma->vm_page_prot = pgprot_noncached(vma->vm_page_prot);
-#ifdef HAVE_PTE_MASK_PAGE_IOMAP
-       vma->vm_page_prot.pgprot |= _PAGE_IOMAP;
-#endif
-
-       return remap_pfn_range(vma,
-                       vma->vm_start,
-                       info->mem[idx].addr >> PAGE_SHIFT,
-                       vma->vm_end - vma->vm_start,
-                       vma->vm_page_prot);
-}
-
-/**
- * This is uio device mmap method which will use igbuio mmap for Xen
- * Dom0 environment.
- */
-static int
-igbuio_dom0_pci_mmap(struct uio_info *info, struct vm_area_struct *vma)
-{
-       int idx;
-
-       if (vma->vm_pgoff >= MAX_UIO_MAPS)
-               return -EINVAL;
-
-       if (info->mem[vma->vm_pgoff].size == 0)
-               return -EINVAL;
-
-       idx = (int)vma->vm_pgoff;
-       switch (info->mem[idx].memtype) {
-       case UIO_MEM_PHYS:
-               return igbuio_dom0_mmap_phys(info, vma);
-       case UIO_MEM_LOGICAL:
-       case UIO_MEM_VIRTUAL:
-       default:
-               return -EINVAL;
-       }
-}
-#endif
-
-/* Remap pci resources described by bar #pci_bar in uio resource n. */
-static int
-igbuio_pci_setup_iomem(struct pci_dev *dev, struct uio_info *info,
-                      int n, int pci_bar, const char *name)
-{
-       unsigned long addr, len;
-       void *internal_addr;
-
-       if (n >= ARRAY_SIZE(info->mem))
-               return -EINVAL;
-
-       addr = pci_resource_start(dev, pci_bar);
-       len = pci_resource_len(dev, pci_bar);
-       if (addr == 0 || len == 0)
-               return -1;
-       internal_addr = ioremap(addr, len);
-       if (internal_addr == NULL)
-               return -1;
-       info->mem[n].name = name;
-       info->mem[n].addr = addr;
-       info->mem[n].internal_addr = internal_addr;
-       info->mem[n].size = len;
-       info->mem[n].memtype = UIO_MEM_PHYS;
-       return 0;
-}
-
-/* Get pci port io resources described by bar #pci_bar in uio resource n. */
-static int
-igbuio_pci_setup_ioport(struct pci_dev *dev, struct uio_info *info,
-               int n, int pci_bar, const char *name)
-{
-       unsigned long addr, len;
-
-       if (n >= ARRAY_SIZE(info->port))
-               return -EINVAL;
-
-       addr = pci_resource_start(dev, pci_bar);
-       len = pci_resource_len(dev, pci_bar);
-       if (addr == 0 || len == 0)
-               return -EINVAL;
-
-       info->port[n].name = name;
-       info->port[n].start = addr;
-       info->port[n].size = len;
-       info->port[n].porttype = UIO_PORT_X86;
-
-       return 0;
-}
-
-/* Unmap previously ioremap'd resources */
-static void
-igbuio_pci_release_iomem(struct uio_info *info)
-{
-       int i;
-
-       for (i = 0; i < MAX_UIO_MAPS; i++) {
-               if (info->mem[i].internal_addr)
-                       iounmap(info->mem[i].internal_addr);
-       }
-}
-
 static int
 igbuio_pci_enable_interrupts(struct rte_uio_pci_dev *udev)
 {
@@ -327,6 +252,7 @@ igbuio_pci_enable_interrupts(struct rte_uio_pci_dev *udev)
                        break;
                }
 #endif
+
        /* fall back to MSI */
        case RTE_INTR_MODE_MSI:
 #ifndef HAVE_ALLOC_IRQ_VECTORS
@@ -365,15 +291,28 @@ igbuio_pci_enable_interrupts(struct rte_uio_pci_dev *udev)
        default:
                dev_err(&udev->pdev->dev, "invalid IRQ mode %u",
                        igbuio_intr_mode_preferred);
+               udev->info.irq = UIO_IRQ_NONE;
                err = -EINVAL;
        }
 
+       if (udev->info.irq != UIO_IRQ_NONE)
+               err = request_irq(udev->info.irq, igbuio_pci_irqhandler,
+                                 udev->info.irq_flags, udev->info.name,
+                                 udev);
+       dev_info(&udev->pdev->dev, "uio device registered with irq %lx\n",
+                udev->info.irq);
+
        return err;
 }
 
 static void
 igbuio_pci_disable_interrupts(struct rte_uio_pci_dev *udev)
 {
+       if (udev->info.irq) {
+               free_irq(udev->info.irq, udev);
+               udev->info.irq = 0;
+       }
+
 #ifndef HAVE_ALLOC_IRQ_VECTORS
        if (udev->mode == RTE_INTR_MODE_MSIX)
                pci_disable_msix(udev->pdev);
@@ -386,6 +325,105 @@ igbuio_pci_disable_interrupts(struct rte_uio_pci_dev *udev)
 #endif
 }
 
+
+/**
+ * This gets called while opening uio device file.
+ */
+static int
+igbuio_pci_open(struct uio_info *info, struct inode *inode)
+{
+       struct rte_uio_pci_dev *udev = info->priv;
+       struct pci_dev *dev = udev->pdev;
+       int err;
+
+       /* set bus master, which was cleared by the reset function */
+       pci_set_master(dev);
+
+       /* enable interrupts */
+       err = igbuio_pci_enable_interrupts(udev);
+       if (err) {
+               dev_err(&dev->dev, "Enable interrupt fails\n");
+               return err;
+       }
+       return 0;
+}
+
+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;
+
+       /* disable interrupts */
+       igbuio_pci_disable_interrupts(udev);
+
+       /* stop the device from further DMA */
+       pci_clear_master(dev);
+
+       return 0;
+}
+
+/* Remap pci resources described by bar #pci_bar in uio resource n. */
+static int
+igbuio_pci_setup_iomem(struct pci_dev *dev, struct uio_info *info,
+                      int n, int pci_bar, const char *name)
+{
+       unsigned long addr, len;
+       void *internal_addr;
+
+       if (n >= ARRAY_SIZE(info->mem))
+               return -EINVAL;
+
+       addr = pci_resource_start(dev, pci_bar);
+       len = pci_resource_len(dev, pci_bar);
+       if (addr == 0 || len == 0)
+               return -1;
+       internal_addr = ioremap(addr, len);
+       if (internal_addr == NULL)
+               return -1;
+       info->mem[n].name = name;
+       info->mem[n].addr = addr;
+       info->mem[n].internal_addr = internal_addr;
+       info->mem[n].size = len;
+       info->mem[n].memtype = UIO_MEM_PHYS;
+       return 0;
+}
+
+/* Get pci port io resources described by bar #pci_bar in uio resource n. */
+static int
+igbuio_pci_setup_ioport(struct pci_dev *dev, struct uio_info *info,
+               int n, int pci_bar, const char *name)
+{
+       unsigned long addr, len;
+
+       if (n >= ARRAY_SIZE(info->port))
+               return -EINVAL;
+
+       addr = pci_resource_start(dev, pci_bar);
+       len = pci_resource_len(dev, pci_bar);
+       if (addr == 0 || len == 0)
+               return -EINVAL;
+
+       info->port[n].name = name;
+       info->port[n].start = addr;
+       info->port[n].size = len;
+       info->port[n].porttype = UIO_PORT_X86;
+
+       return 0;
+}
+
+/* Unmap previously ioremap'd resources */
+static void
+igbuio_pci_release_iomem(struct uio_info *info)
+{
+       int i;
+
+       for (i = 0; i < MAX_UIO_MAPS; i++) {
+               if (info->mem[i].internal_addr)
+                       iounmap(info->mem[i].internal_addr);
+       }
+}
+
 static int
 igbuio_setup_bars(struct pci_dev *dev, struct uio_info *info)
 {
@@ -476,25 +514,15 @@ igbuio_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
        /* fill uio infos */
        udev->info.name = "igb_uio";
        udev->info.version = "0.1";
-       udev->info.handler = igbuio_pci_irqhandler;
        udev->info.irqcontrol = igbuio_pci_irqcontrol;
        udev->info.open = igbuio_pci_open;
        udev->info.release = igbuio_pci_release;
-#ifdef CONFIG_XEN_DOM0
-       /* check if the driver run on Xen Dom0 */
-       if (xen_initial_domain())
-               udev->info.mmap = igbuio_dom0_pci_mmap;
-#endif
        udev->info.priv = udev;
        udev->pdev = dev;
 
-       err = igbuio_pci_enable_interrupts(udev);
-       if (err != 0)
-               goto fail_release_iomem;
-
        err = sysfs_create_group(&dev->dev.kobj, &dev_attr_grp);
        if (err != 0)
-               goto fail_disable_interrupts;
+               goto fail_release_iomem;
 
        /* register uio driver */
        err = uio_register_device(&dev->dev, &udev->info);
@@ -503,9 +531,6 @@ igbuio_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
 
        pci_set_drvdata(dev, udev);
 
-       dev_info(&dev->dev, "uio device registered with irq %lx\n",
-                udev->info.irq);
-
        /*
         * Doing a harmless dma mapping for attaching the device to
         * the iommu identity mapping if kernel boots with iommu=pt.
@@ -531,8 +556,6 @@ igbuio_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
 
 fail_remove_group:
        sysfs_remove_group(&dev->dev.kobj, &dev_attr_grp);
-fail_disable_interrupts:
-       igbuio_pci_disable_interrupts(udev);
 fail_release_iomem:
        igbuio_pci_release_iomem(&udev->info);
        pci_disable_device(dev);
@@ -549,7 +572,6 @@ igbuio_pci_remove(struct pci_dev *dev)
 
        sysfs_remove_group(&dev->dev.kobj, &dev_attr_grp);
        uio_unregister_device(&udev->info);
-       igbuio_pci_disable_interrupts(udev);
        igbuio_pci_release_iomem(&udev->info);
        pci_disable_device(dev);
        pci_set_drvdata(dev, NULL);