igb_uio: fix build
[dpdk.git] / lib / librte_eal / linuxapp / igb_uio / igb_uio.c
index ba38d23..865a276 100644 (file)
 #include <linux/io.h>
 #include <linux/msi.h>
 #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"
+
 #ifdef RTE_PCI_CONFIG
 #define PCI_SYS_FILE_BUF_SIZE      10
 #define PCI_DEV_CAP_REG            0xA4
@@ -52,7 +55,6 @@
 struct rte_uio_pci_dev {
        struct uio_info info;
        struct pci_dev *pdev;
-       spinlock_t lock; /* spinlock for accessing PCI config space or msix data in multi tasks/isr */
        enum rte_intr_mode mode;
 };
 
@@ -66,25 +68,6 @@ igbuio_get_uio_pci_dev(struct uio_info *info)
 }
 
 /* sriov sysfs */
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 34)
-static int pci_num_vf(struct pci_dev *dev)
-       struct iov {
-               int pos;
-               int nres;
-               u32 cap;
-               u16 ctrl;
-               u16 total;
-               u16 initial;
-               u16 nr_virtfn;
-       } *iov = (struct iov *)dev->sriov;
-
-       if (!dev->is_physfn)
-               return 0;
-
-       return iov->nr_virtfn;
-}
-#endif
-
 static ssize_t
 show_max_vfs(struct device *dev, struct device_attribute *attr,
             char *buf)
@@ -101,7 +84,7 @@ store_max_vfs(struct device *dev, struct device_attribute *attr,
        unsigned long max_vfs;
        struct pci_dev *pdev = container_of(dev, struct pci_dev, dev);
 
-       if (0 != strict_strtoul(buf, 0, &max_vfs))
+       if (0 != kstrtoul(buf, 0, &max_vfs))
                return -EINVAL;
 
        if (0 == max_vfs)
@@ -192,7 +175,7 @@ store_max_read_request_size(struct device *dev,
        unsigned long size = 0;
        int ret;
 
-       if (strict_strtoul(buf, 0, &size) != 0)
+       if (0 != kstrtoul(buf, 0, &size))
                return -EINVAL;
 
        ret = pcie_set_readrq(pci_dev, (int)size);
@@ -223,37 +206,13 @@ static struct attribute *dev_attrs[] = {
 static const struct attribute_group dev_attr_grp = {
        .attrs = dev_attrs,
 };
-
-static inline int
-pci_lock(struct pci_dev * pdev)
-{
-       /* Some function names changes between 3.2.0 and 3.3.0... */
-#if LINUX_VERSION_CODE < KERNEL_VERSION(3, 3, 0)
-       pci_block_user_cfg_access(pdev);
-       return 1;
-#else
-       return pci_cfg_access_trylock(pdev);
-#endif
-}
-
-static inline void
-pci_unlock(struct pci_dev * pdev)
-{
-       /* Some function names changes between 3.2.0 and 3.3.0... */
-#if LINUX_VERSION_CODE < KERNEL_VERSION(3, 3, 0)
-       pci_unblock_user_cfg_access(pdev);
-#else
-       pci_cfg_access_unlock(pdev);
-#endif
-}
-
-/**
+/*
  * It masks the msix on/off of generating MSI-X messages.
  */
-static int
+static void
 igbuio_msix_mask_irq(struct msi_desc *desc, int32_t state)
 {
-       uint32_t mask_bits = desc->masked;
+       u32 mask_bits = desc->masked;
        unsigned offset = desc->msi_attrib.entry_nr * PCI_MSIX_ENTRY_SIZE +
                                                PCI_MSIX_ENTRY_VECTOR_CTRL;
 
@@ -267,48 +226,6 @@ igbuio_msix_mask_irq(struct msi_desc *desc, int32_t state)
                readl(desc->mask_base);
                desc->masked = mask_bits;
        }
-
-       return 0;
-}
-
-/**
- * This function sets/clears the masks for generating LSC interrupts.
- *
- * @param info
- *   The pointer to struct uio_info.
- * @param on
- *   The on/off flag of masking LSC.
- * @return
- *   -On success, zero value.
- *   -On failure, a negative value.
- */
-static int
-igbuio_set_interrupt_mask(struct rte_uio_pci_dev *udev, int32_t state)
-{
-       struct pci_dev *pdev = udev->pdev;
-
-       if (udev->mode == RTE_INTR_MODE_MSIX) {
-               struct msi_desc *desc;
-
-               list_for_each_entry(desc, &pdev->msi_list, list) {
-                       igbuio_msix_mask_irq(desc, state);
-               }
-       } else if (udev->mode == RTE_INTR_MODE_LEGACY) {
-               uint32_t status;
-               uint16_t old, new;
-
-               pci_read_config_dword(pdev, PCI_COMMAND, &status);
-               old = status;
-               if (state != 0)
-                       new = old & (~PCI_COMMAND_INTX_DISABLE);
-               else
-                       new = old | PCI_COMMAND_INTX_DISABLE;
-
-               if (old != new)
-                       pci_write_config_word(pdev, PCI_COMMAND, new);
-       }
-
-       return 0;
 }
 
 /**
@@ -327,20 +244,20 @@ igbuio_set_interrupt_mask(struct rte_uio_pci_dev *udev, int32_t state)
 static int
 igbuio_pci_irqcontrol(struct uio_info *info, s32 irq_state)
 {
-       unsigned long flags;
        struct rte_uio_pci_dev *udev = igbuio_get_uio_pci_dev(info);
        struct pci_dev *pdev = udev->pdev;
 
-       spin_lock_irqsave(&udev->lock, flags);
-       if (!pci_lock(pdev)) {
-               spin_unlock_irqrestore(&udev->lock, flags);
-               return -1;
-       }
+       pci_cfg_access_lock(pdev);
+       if (udev->mode == RTE_INTR_MODE_LEGACY)
+               pci_intx(pdev, !!irq_state);
 
-       igbuio_set_interrupt_mask(udev, irq_state);
+       else if (udev->mode == RTE_INTR_MODE_MSIX) {
+               struct msi_desc *desc;
 
-       pci_unlock(pdev);
-       spin_unlock_irqrestore(&udev->lock, flags);
+               list_for_each_entry(desc, &pdev->msi_list, list)
+                       igbuio_msix_mask_irq(desc, irq_state);
+       }
+       pci_cfg_access_unlock(pdev);
 
        return 0;
 }
@@ -352,37 +269,15 @@ igbuio_pci_irqcontrol(struct uio_info *info, s32 irq_state)
 static irqreturn_t
 igbuio_pci_irqhandler(int irq, struct uio_info *info)
 {
-       irqreturn_t ret = IRQ_NONE;
-       unsigned long flags;
        struct rte_uio_pci_dev *udev = igbuio_get_uio_pci_dev(info);
-       struct pci_dev *pdev = udev->pdev;
-       uint32_t cmd_status_dword;
-       uint16_t status;
-
-       spin_lock_irqsave(&udev->lock, flags);
-       /* block userspace PCI config reads/writes */
-       if (!pci_lock(pdev))
-               goto spin_unlock;
-
-       /* for legacy mode, interrupt maybe shared */
-       if (udev->mode == RTE_INTR_MODE_LEGACY) {
-               pci_read_config_dword(pdev, PCI_COMMAND, &cmd_status_dword);
-               status = cmd_status_dword >> 16;
-               /* interrupt is not ours, goes to out */
-               if (!(status & PCI_STATUS_INTERRUPT))
-                       goto done;
-       }
 
-       igbuio_set_interrupt_mask(udev, 0);
-       ret = IRQ_HANDLED;
-done:
-       /* unblock userspace PCI config reads/writes */
-       pci_unlock(pdev);
-spin_unlock:
-       spin_unlock_irqrestore(&udev->lock, flags);
-       pr_info("irq 0x%x %s\n", irq, (ret == IRQ_HANDLED) ? "handled" : "not handled");
+       /* Legacy mode need to mask in hardware */
+       if (udev->mode == RTE_INTR_MODE_LEGACY &&
+           !pci_check_and_mask_intx(udev->pdev))
+               return IRQ_NONE;
 
-       return ret;
+       /* Message signal mode, no share IRQ and automasked */
+       return IRQ_HANDLED;
 }
 
 #ifdef CONFIG_XEN_DOM0
@@ -393,7 +288,9 @@ igbuio_dom0_mmap_phys(struct uio_info *info, struct vm_area_struct *vma)
 
        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,
@@ -598,40 +495,40 @@ igbuio_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
 #endif
        udev->info.priv = udev;
        udev->pdev = dev;
-       udev->mode = RTE_INTR_MODE_LEGACY;
-       spin_lock_init(&udev->lock);
 
-       /* check if it need to try msix first */
-       if (igbuio_intr_mode_preferred == RTE_INTR_MODE_MSIX) {
+       switch (igbuio_intr_mode_preferred) {
+       case RTE_INTR_MODE_MSIX:
                /* Only 1 msi-x vector needed */
                msix_entry.entry = 0;
                if (pci_enable_msix(dev, &msix_entry, 1) == 0) {
                        dev_dbg(&dev->dev, "using MSI-X");
+                       udev->info.irq = msix_entry.vector;
                        udev->mode = RTE_INTR_MODE_MSIX;
+                       break;
                }
-               else {
-                       pci_disable_msix(udev->pdev);
-                       pr_info("fail to enable pci msix, or not enough msix entries\n");
-               }
-       }
-       switch (udev->mode) {
-       case RTE_INTR_MODE_MSIX:
-               udev->info.irq_flags = 0;
-               udev->info.irq = msix_entry.vector;
-               break;
-       case RTE_INTR_MODE_MSI:
-               break;
+               /* fall back to INTX */
        case RTE_INTR_MODE_LEGACY:
-               udev->info.irq_flags = IRQF_SHARED;
-               udev->info.irq = dev->irq;
+               if (pci_intx_mask_supported(dev)) {
+                       dev_dbg(&dev->dev, "using INTX");
+                       udev->info.irq_flags = IRQF_SHARED;
+                       udev->info.irq = dev->irq;
+                       udev->mode = RTE_INTR_MODE_LEGACY;
+                       break;
+               }
+               dev_notice(&dev->dev, "PCI INTX mask not supported\n");
+               /* fall back to no IRQ */
+       case RTE_INTR_MODE_NONE:
+               udev->mode = RTE_INTR_MODE_NONE;
+               udev->info.irq = 0;
                break;
+
        default:
-               break;
+               dev_err(&dev->dev, "invalid IRQ mode %u",
+                       igbuio_intr_mode_preferred);
+               err = -EINVAL;
+               goto fail_release_iomem;
        }
 
-       pci_set_drvdata(dev, udev);
-       igbuio_pci_irqcontrol(&udev->info, 0);
-
        err = sysfs_create_group(&dev->dev.kobj, &dev_attr_grp);
        if (err != 0)
                goto fail_release_iomem;
@@ -641,7 +538,10 @@ igbuio_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
        if (err != 0)
                goto fail_remove_group;
 
-       pr_info("uio device registered with irq %lx\n", udev->info.irq);
+       pci_set_drvdata(dev, udev);
+
+       dev_info(&dev->dev, "uio device registered with irq %lx\n",
+                udev->info.irq);
 
        return 0;