Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1622091
| From | David Woodhouse <dwmw2@infradead.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | [PATCH v2 05/27] pci: Add arch_can_pci_mmap_wc() macro |
| Date | 2017-04-12 14:40 +0200 |
| Message-ID | <tvpIZ-4rx-1@gated-at.bofh.it> (permalink) |
| References | <tvpzk-4op-3@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
From: David Woodhouse <dwmw@amazon.co.uk>
Most of the almost-identical versions of pci_mmap_page_range() silently
ignore the 'write_combine' argument and give uncached mappings.
Yet we allow the PCIIOC_WRITE_COMBINE ioctl in /proc/bus/pci, expose the
'resourceX_wc' file in sysfs, and allow an attempted mapping to apparently
succeed.
To fix this, introduce a macro arch_can_pci_mmap_wc() which indicates
whether the platform can do a write-combining mapping. On x86 this ends
up being pat_enabled(), while the few other platforms that support it
can just set it to a literal '1'.
Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
---
Documentation/filesystems/sysfs-pci.txt | 4 ++++
arch/ia64/include/asm/pci.h | 2 ++
arch/powerpc/include/asm/pci.h | 5 +++--
arch/x86/include/asm/pci.h | 2 ++
drivers/pci/pci-sysfs.c | 4 ++--
drivers/pci/proc.c | 15 ++++++++-------
include/linux/pci.h | 4 ++++
7 files changed, 25 insertions(+), 11 deletions(-)
diff --git a/Documentation/filesystems/sysfs-pci.txt b/Documentation/filesystems/sysfs-pci.txt
index 6ea1ced..25b7f1c 100644
--- a/Documentation/filesystems/sysfs-pci.txt
+++ b/Documentation/filesystems/sysfs-pci.txt
@@ -117,6 +117,10 @@ code must define HAVE_PCI_MMAP and provide a pci_mmap_page_range function.
Platforms are free to only support subsets of the mmap functionality, but
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.
+
Legacy resources are protected by the HAVE_PCI_LEGACY define. Platforms
wishing to support legacy functionality should define it and provide
pci_legacy_read, pci_legacy_write and pci_mmap_legacy_page_range functions.
diff --git a/arch/ia64/include/asm/pci.h b/arch/ia64/include/asm/pci.h
index c0835b0..6283758 100644
--- a/arch/ia64/include/asm/pci.h
+++ b/arch/ia64/include/asm/pci.h
@@ -51,6 +51,8 @@ extern unsigned long ia64_max_iommu_merge_mask;
#define PCI_DMA_BUS_IS_PHYS (ia64_max_iommu_merge_mask == ~0UL)
#define HAVE_PCI_MMAP
+#define arch_can_pci_mmap_wc() 1
+
extern int pci_mmap_page_range (struct pci_dev *dev, struct vm_area_struct *vma,
enum pci_mmap_state mmap_state, int write_combine);
#define HAVE_PCI_LEGACY
diff --git a/arch/powerpc/include/asm/pci.h b/arch/powerpc/include/asm/pci.h
index 93eded8..b5b68c6 100644
--- a/arch/powerpc/include/asm/pci.h
+++ b/arch/powerpc/include/asm/pci.h
@@ -81,8 +81,9 @@ struct vm_area_struct;
int pci_mmap_page_range(struct pci_dev *pdev, struct vm_area_struct *vma,
enum pci_mmap_state mmap_state, int write_combine);
-/* Tell drivers/pci/proc.c that we have pci_mmap_page_range() */
-#define HAVE_PCI_MMAP 1
+/* 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_wc() 1
extern int pci_legacy_read(struct pci_bus *bus, loff_t port, u32 *val,
size_t count);
diff --git a/arch/x86/include/asm/pci.h b/arch/x86/include/asm/pci.h
index 1411dbe..f6e22c2 100644
--- a/arch/x86/include/asm/pci.h
+++ b/arch/x86/include/asm/pci.h
@@ -7,6 +7,7 @@
#include <linux/string.h>
#include <linux/scatterlist.h>
#include <asm/io.h>
+#include <asm/pat.h>
#include <asm/x86_init.h>
#ifdef __KERNEL__
@@ -102,6 +103,7 @@ int pcibios_set_irq_routing(struct pci_dev *dev, int pin, int irq);
#define HAVE_PCI_MMAP
+#define arch_can_pci_mmap_wc() pat_enabled()
extern int pci_mmap_page_range(struct pci_dev *dev, struct vm_area_struct *vma,
enum pci_mmap_state mmap_state,
int write_combine);
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
index 7ac258f..7d494bd 100644
--- a/drivers/pci/pci-sysfs.c
+++ b/drivers/pci/pci-sysfs.c
@@ -1211,9 +1211,9 @@ static int pci_create_resource_files(struct pci_dev *pdev)
retval = pci_create_attr(pdev, i, 0);
/* for prefetchable resources, create a WC mappable file */
- if (!retval && pdev->resource[i].flags & IORESOURCE_PREFETCH)
+ if (!retval && arch_can_pci_mmap_wc() &&
+ pdev->resource[i].flags & IORESOURCE_PREFETCH)
retval = pci_create_attr(pdev, i, 1);
-
if (retval) {
pci_remove_resource_files(pdev);
return retval;
diff --git a/drivers/pci/proc.c b/drivers/pci/proc.c
index dc8912e..a2aa58a 100644
--- a/drivers/pci/proc.c
+++ b/drivers/pci/proc.c
@@ -210,14 +210,15 @@ static long proc_bus_pci_ioctl(struct file *file, unsigned int cmd,
break;
case PCIIOC_WRITE_COMBINE:
- if (arg)
- fpriv->write_combine = 1;
- else
- fpriv->write_combine = 0;
- break;
-
+ if (arch_can_pci_mmap_wc()) {
+ if (arg)
+ fpriv->write_combine = 1;
+ else
+ fpriv->write_combine = 0;
+ break;
+ }
+ /* If arch decided it can't, fall through... */
#endif /* HAVE_PCI_MMAP */
-
default:
ret = -EINVAL;
break;
diff --git a/include/linux/pci.h b/include/linux/pci.h
index eb3da1a..e614fb4 100644
--- a/include/linux/pci.h
+++ b/include/linux/pci.h
@@ -1626,6 +1626,10 @@ static inline int pci_get_new_domain_nr(void) { return -ENOSYS; }
#include <asm/pci.h>
+#ifndef arch_can_pci_mmap_wc
+#define arch_can_pci_mmap_wc() 0
+#endif
+
#ifndef pci_root_bus_fwnode
#define pci_root_bus_fwnode(bus) NULL
#endif
--
2.9.3
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v2 00/27] PCI resource mmap cleanup David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 18/27] x86: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 06/27] pci: Move multiple declarations of pci_mmap_page_range() to <linux/pci.h> David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 07/27] pci: Add arch_can_pci_mmap_io() on architectures which can mmap() I/O space David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 01/27] pci: Fix pci_mmap_fits() for HAVE_PCI_RESOURCE_TO_USER platforms David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 21/27] microblaze: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 04/27] xtensa: Do not mmap PCI BARs to userspace as write-through David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 19/27] pci: Add I/O BAR support to generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:30 +0200
[PATCH v2 05/27] pci: Add arch_can_pci_mmap_wc() macro David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 22/27] xtensa: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 14/27] mn10300: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
Re: [PATCH v2 14/27] mn10300: Use generic pci_mmap_resource_range() David Howells <dhowells@redhat.com> - 2017-04-13 11:30 +0200
[PATCH v2 02/27] pci: Fix another sanity check bug in /proc/pci mmap David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 08/27] pci: Use BAR index in sysfs attr->private instead of resource pointer David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 17/27] unicore: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 25/27] ia64: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
Re: [PATCH v2 25/27] ia64: Use generic pci_mmap_resource_range() Tony Luck <tony.luck@gmail.com> - 2017-04-12 23:50 +0200
Re: [PATCH v2 25/27] ia64: Use generic pci_mmap_resource_range() Thomas Gleixner <tglx@linutronix.de> - 2017-04-13 00:00 +0200
Re: [PATCH v2 25/27] ia64: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-13 10:40 +0200
[PATCH v2 10/27] pci: Add pci_mmap_resource_range() and use it for ARM64 David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
Re: [PATCH v2 10/27] pci: Add pci_mmap_resource_range() and use it for ARM64 David Howells <dhowells@redhat.com> - 2017-04-13 11:30 +0200
[PATCH v2 09/27] pci: Add BAR index argument to pci_mmap_page_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 26/27] sparc: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 20/27] powerpc: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 13/27] mips: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 11/27] arm: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 16/27] sh: Use generic pci_mmap_resource_range() David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
[PATCH v2 03/27] pci: Only allow WC mmap on prefetchable resources David Woodhouse <dwmw2@infradead.org> - 2017-04-12 14:40 +0200
Re: [PATCH v2 00/27] PCI resource mmap cleanup Bjorn Helgaas <helgaas@kernel.org> - 2017-04-18 20:30 +0200
Re: [PATCH v2 00/27] PCI resource mmap cleanup Bjorn Helgaas <helgaas@kernel.org> - 2017-04-18 21:00 +0200
Re: [PATCH v2 00/27] PCI resource mmap cleanup David Woodhouse <dwmw2@infradead.org> - 2017-04-19 09:30 +0200
Re: [PATCH v2 00/27] PCI resource mmap cleanup Bjorn Helgaas <bhelgaas@google.com> - 2017-04-19 20:40 +0200
csiph-web