PCI: Add arch_can_pci_mmap_io() on architectures which can mmap() I/O space
authorDavid Woodhouse <dwmw@amazon.co.uk>
Wed, 12 Apr 2017 12:25:56 +0000 (13:25 +0100)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 18 Apr 2017 18:02:26 +0000 (13:02 -0500)
This is relatively esoteric, and knowing that we don't have it makes life
easier in some cases rather than just an eventual -EINVAL from
pci_mmap_page_range().

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Documentation/filesystems/sysfs-pci.txt
arch/microblaze/include/asm/pci.h
arch/powerpc/include/asm/pci.h
arch/sparc/include/asm/pci_64.h
arch/xtensa/include/asm/pci.h
drivers/pci/pci-sysfs.c
drivers/pci/proc.c
include/linux/pci.h

index 25b7f1c1b868abf24ee570d4ca430585552946cc..46b95d82c4fd4a7696a31001e6bd3f0db835ec59 100644 (file)
@@ -119,7 +119,8 @@ useful return codes should be provided.
 
 Platforms which support write-combining maps of PCI resources must define
 arch_can_pci_mmap_wc() which shall evaluate to non-zero at runtime when
-write-combining is permitted.
+write-combining is permitted. Platforms which support maps of I/O resources
+define arch_can_pci_mmap_io() similarly.
 
 Legacy resources are protected by the HAVE_PCI_LEGACY define.  Platforms
 wishing to support legacy functionality should define it and provide
index ab381d283fb9200465c9e8168ba2bd0f9f7d2941..efd4983cb6972a2b774950617141b9ace491230f 100644 (file)
@@ -48,7 +48,8 @@ extern int pci_proc_domain(struct pci_bus *bus);
 struct vm_area_struct;
 
 /* Tell drivers/pci/proc.c that we have pci_mmap_page_range() */
-#define HAVE_PCI_MMAP  1
+#define HAVE_PCI_MMAP          1
+#define arch_can_pci_mmap_io() 1
 
 extern int pci_legacy_read(struct pci_bus *bus, loff_t port, u32 *val,
                           size_t count);
index 55887d1712053290e1d1a0261bbe3f098e19036a..c8975dac535fc7ea907bc7577b13aee8fd989a8a 100644 (file)
@@ -80,6 +80,7 @@ struct vm_area_struct;
 
 /* Tell drivers/pci/proc.c that we have pci_mmap_page_range() and it does WC */
 #define HAVE_PCI_MMAP          1
+#define arch_can_pci_mmap_io() 1
 #define arch_can_pci_mmap_wc() 1
 
 extern int pci_legacy_read(struct pci_bus *bus, loff_t port, u32 *val,
index 516fda70a1bb457dd8e3293c9caff098cd2def2a..b957ca5527a3d23c603a53eab461991536cc66ff 100644 (file)
@@ -42,6 +42,7 @@ static inline int pci_proc_domain(struct pci_bus *bus)
 /* Platform support for /proc/bus/pci/X/Y mmap()s. */
 
 #define HAVE_PCI_MMAP
+#define arch_can_pci_mmap_io() 1
 #define HAVE_ARCH_PCI_GET_UNMAPPED_AREA
 #define get_pci_unmapped_area get_fb_unmapped_area
 
index bb5510b28754ef686c462bb07846c782ade2cb8d..e4f366a488d3dc2fd7ba292d7ae4e2ccbe1fcae3 100644 (file)
@@ -47,7 +47,8 @@ struct pci_dev;
 #define PCI_DMA_BUS_IS_PHYS    (1)
 
 /* Tell drivers/pci/proc.c that we have pci_mmap_page_range() */
-#define HAVE_PCI_MMAP  1
+#define HAVE_PCI_MMAP          1
+#define arch_can_pci_mmap_io() 1
 
 #endif /* __KERNEL__ */
 
index 7d494bd66a97f01bc7145d3691939fc7418de523..cb04bc29943f12ebeda69aaa688dabdb31c801c1 100644 (file)
@@ -1174,11 +1174,14 @@ static int pci_create_attr(struct pci_dev *pdev, int num, int write_combine)
        } else {
                pdev->res_attr[num] = res_attr;
                sprintf(res_attr_name, "resource%d", num);
-               res_attr->mmap = pci_mmap_resource_uc;
-       }
-       if (pci_resource_flags(pdev, num) & IORESOURCE_IO) {
-               res_attr->read = pci_read_resource_io;
-               res_attr->write = pci_write_resource_io;
+               if (pci_resource_flags(pdev, num) & IORESOURCE_IO) {
+                       res_attr->read = pci_read_resource_io;
+                       res_attr->write = pci_write_resource_io;
+                       if (arch_can_pci_mmap_io())
+                               res_attr->mmap = pci_mmap_resource_uc;
+               } else {
+                       res_attr->mmap = pci_mmap_resource_uc;
+               }
        }
        res_attr->attr.name = res_attr_name;
        res_attr->attr.mode = S_IRUSR | S_IWUSR;
index a2aa58a8fb9632613b65389ea249aec96eff4423..45e5cf7e91931056bdcba5ed5a282cd0ea2e3c90 100644 (file)
@@ -202,6 +202,8 @@ static long proc_bus_pci_ioctl(struct file *file, unsigned int cmd,
 
 #ifdef HAVE_PCI_MMAP
        case PCIIOC_MMAP_IS_IO:
+               if (!arch_can_pci_mmap_io())
+                       return -EINVAL;
                fpriv->mmap_state = pci_mmap_io;
                break;
 
@@ -232,15 +234,16 @@ static int proc_bus_pci_mmap(struct file *file, struct vm_area_struct *vma)
 {
        struct pci_dev *dev = PDE_DATA(file_inode(file));
        struct pci_filp_private *fpriv = file->private_data;
-       int i, ret, write_combine = 0, res_bit;
+       int i, ret, write_combine = 0, res_bit = IORESOURCE_MEM;
 
        if (!capable(CAP_SYS_RAWIO))
                return -EPERM;
 
-       if (fpriv->mmap_state == pci_mmap_io)
+       if (fpriv->mmap_state == pci_mmap_io) {
+               if (!arch_can_pci_mmap_io())
+                       return -EINVAL;
                res_bit = IORESOURCE_IO;
-       else
-               res_bit = IORESOURCE_MEM;
+       }
 
        /* Make sure the caller is mapping a real resource for this device */
        for (i = 0; i < PCI_ROM_RESOURCE; i++) {
index e7bb4b62cc971b56f6a4071df31c852e6c8e1d02..590cfcf6acf59361a7de2e7bab4f44d872997e03 100644 (file)
@@ -1636,6 +1636,9 @@ int pci_mmap_page_range(struct pci_dev *pdev, struct vm_area_struct *vma,
 #ifndef arch_can_pci_mmap_wc
 #define arch_can_pci_mmap_wc()         0
 #endif
+#ifndef arch_can_pci_mmap_io
+#define arch_can_pci_mmap_io()         0
+#endif
 
 #ifndef pci_root_bus_fwnode
 #define pci_root_bus_fwnode(bus)       NULL