[PATCH] Updated: fix-pci-mmap-on-ppc-and-ppc64.patch

[PATCH] Updated: fix-pci-mmap-on-ppc-and-ppc64.patch

[Michael Ellerman]

Hi Andrew,

This is an updated version of Ben's fix-pci-mmap-on-ppc-and-ppc64.patch
which is in 2.6.12-rc4-mm1.

It fixes the patch to work on PPC iSeries, removes some debug printks
at Ben's request, and incorporates your 
fix-pci-mmap-on-ppc-and-ppc64-fix.patch also.

cheers

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>

--
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>

This patch was discussed at length on linux-pci and so far, the last
iteration of it didn't raise any comment.  It's effect is a nop on
architecture that don't define the new pci_resource_to_user() callback
anyway.  It allows architecture like ppc who put weird things inside of PCI
resource structures to convert to some different value for user visible
ones.  It also fixes mmap'ing of IO space on those archs.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>

 arch/ppc/kernel/pci.c   |   21 +++++++++++++++++++--
 arch/ppc64/kernel/pci.c |   22 ++++++++++++++++++++--
 drivers/pci/pci-sysfs.c |   26 +++++++++++++++++++++-----
 drivers/pci/proc.c      |   14 ++++++++++----
 include/asm-ppc/pci.h   |    6 ++++++
 include/asm-ppc64/pci.h |    7 +++++++
 include/linux/pci.h     |   14 ++++++++++++++
 7 files changed, 97 insertions(+), 13 deletions(-)

Index: 2.6.12-rc4-mm1/arch/ppc64/kernel/pci.c
===================================================================
--- 2.6.12-rc4-mm1.orig/arch/ppc64/kernel/pci.c	2005-05-13 15:39:55.000000000 +1000
+++ 2.6.12-rc4-mm1/arch/ppc64/kernel/pci.c	2005-05-13 15:40:19.000000000 +1000
@@ -351,7 +351,7 @@
 		*offset += hose->pci_mem_offset;
 		res_bit = IORESOURCE_MEM;
 	} else {
-		io_offset = (unsigned long)hose->io_base_virt;
+		io_offset = (unsigned long)hose->io_base_virt - pci_io_base;
 		*offset += io_offset;
 		res_bit = IORESOURCE_IO;
 	}
@@ -378,7 +378,7 @@
 
 		/* found it! construct the final physical address */
 		if (mmap_state == pci_mmap_io)
-			*offset += hose->io_base_phys - io_offset;
+		       	*offset += hose->io_base_phys - io_offset;
 		return rp;
 	}
 
@@ -941,4 +941,22 @@
 }
 EXPORT_SYMBOL(pci_read_irq_line);
 
+void pci_resource_to_user(const struct pci_dev *dev, int bar,
+			  const struct resource *rsrc,
+			  u64 *start, u64 *end)
+{
+	struct pci_controller *hose = pci_bus_to_host(dev->bus);
+	unsigned long offset = 0;
+
+	if (hose == NULL)
+		return;
+
+	if (rsrc->flags & IORESOURCE_IO)
+		offset = pci_io_base - (unsigned long)hose->io_base_virt +
+			hose->io_base_phys;
+
+	*start = rsrc->start + offset;
+	*end = rsrc->end + offset;
+}
+
 #endif /* CONFIG_PPC_MULTIPLATFORM */
Index: 2.6.12-rc4-mm1/arch/ppc/kernel/pci.c
===================================================================
--- 2.6.12-rc4-mm1.orig/arch/ppc/kernel/pci.c	2005-05-13 15:39:55.000000000 +1000
+++ 2.6.12-rc4-mm1/arch/ppc/kernel/pci.c	2005-05-13 15:40:19.000000000 +1000
@@ -1495,7 +1495,7 @@
 		*offset += hose->pci_mem_offset;
 		res_bit = IORESOURCE_MEM;
 	} else {
-		io_offset = (unsigned long)hose->io_base_virt;
+		io_offset = hose->io_base_virt - ___IO_BASE;
 		*offset += io_offset;
 		res_bit = IORESOURCE_IO;
 	}
@@ -1522,7 +1522,7 @@
 
 		/* found it! construct the final physical address */
 		if (mmap_state == pci_mmap_io)
-			*offset += hose->io_base_phys - _IO_BASE;
+			*offset += hose->io_base_phys - io_offset;
 		return rp;
 	}
 
@@ -1739,6 +1739,23 @@
 	return result;
 }
 
+void pci_resource_to_user(const struct pci_dev *dev, int bar,
+			  const struct resource *rsrc,
+			  u64 *start, u64 *end)
+{
+	struct pci_controller *hose = pci_bus_to_hose(dev->bus->number);
+	unsigned long offset = 0;
+
+	if (hose == NULL)
+		return;
+
+	if (rsrc->flags & IORESOURCE_IO)
+		offset = ___IO_BASE - hose->io_base_virt + hose->io_base_phys;
+
+	*start = rsrc->start + offset;
+	*end = rsrc->end + offset;
+}
+
 void __init
 pci_init_resource(struct resource *res, unsigned long start, unsigned long end,
 		  int flags, char *name)
Index: 2.6.12-rc4-mm1/drivers/pci/pci-sysfs.c
===================================================================
--- 2.6.12-rc4-mm1.orig/drivers/pci/pci-sysfs.c	2005-05-13 15:39:55.000000000 +1000
+++ 2.6.12-rc4-mm1/drivers/pci/pci-sysfs.c	2005-05-13 15:40:19.000000000 +1000
@@ -60,15 +60,18 @@
 	char * str = buf;
 	int i;
 	int max = 7;
+	u64 start, end;
 
 	if (pci_dev->subordinate)
 		max = DEVICE_COUNT_RESOURCE;
 
 	for (i = 0; i < max; i++) {
-		str += sprintf(str,"0x%016lx 0x%016lx 0x%016lx\n",
-			       pci_resource_start(pci_dev,i),
-			       pci_resource_end(pci_dev,i),
-			       pci_resource_flags(pci_dev,i));
+		struct resource *res =  &pci_dev->resource[i];
+		pci_resource_to_user(pci_dev, i, res, &start, &end);
+		str += sprintf(str,"0x%016llx 0x%016llx 0x%016llx\n",
+			       (unsigned long long)start,
+			       (unsigned long long)end,
+			       (unsigned long long)res->flags);
 	}
 	return (str - buf);
 }
@@ -301,8 +304,21 @@
 						       struct device, kobj));
 	struct resource *res = (struct resource *)attr->attr.private;
 	enum pci_mmap_state mmap_type;
+	u64 start, end;
+	int i;
 
-	vma->vm_pgoff += res->start >> PAGE_SHIFT;
+	for (i = 0; i < PCI_ROM_RESOURCE; i++)
+		if (res == &pdev->resource[i])
+			break;
+	if (i >= PCI_ROM_RESOURCE)
+		return -ENODEV;
+
+	/* pci_mmap_page_range() expects the same kind of entry as coming
+	 * from /proc/bus/pci/ which is a "user visible" value. If this is
+	 * different from the resource itself, arch will do necessary fixup.
+	 */
+	pci_resource_to_user(pdev, i, res, &start, &end);
+	vma->vm_pgoff += start >> PAGE_SHIFT;
 	mmap_type = res->flags & IORESOURCE_MEM ? pci_mmap_mem : pci_mmap_io;
 
 	return pci_mmap_page_range(pdev, vma, mmap_type, 0);
Index: 2.6.12-rc4-mm1/drivers/pci/proc.c
===================================================================
--- 2.6.12-rc4-mm1.orig/drivers/pci/proc.c	2005-05-13 15:39:55.000000000 +1000
+++ 2.6.12-rc4-mm1/drivers/pci/proc.c	2005-05-13 15:40:19.000000000 +1000
@@ -355,14 +355,20 @@
 			dev->device,
 			dev->irq);
 	/* Here should be 7 and not PCI_NUM_RESOURCES as we need to preserve compatibility */
-	for(i=0; i<7; i++)
+	for(i=0; i<7; i++) {
+		u64 start, end;
+		pci_resource_to_user(dev, i, &dev->resource[i], &start, &end);
 		seq_printf(m, LONG_FORMAT,
-			dev->resource[i].start |
+			((unsigned long)start) |
 			(dev->resource[i].flags & PCI_REGION_FLAG_MASK));
-	for(i=0; i<7; i++)
+	}
+	for(i=0; i<7; i++) {
+		u64 start, end;
+		pci_resource_to_user(dev, i, &dev->resource[i], &start, &end);
 		seq_printf(m, LONG_FORMAT,
 			dev->resource[i].start < dev->resource[i].end ?
-			dev->resource[i].end - dev->resource[i].start + 1 : 0);
+			(unsigned long)(end - start) + 1 : 0);
+	}
 	seq_putc(m, '\t');
 	if (drv)
 		seq_printf(m, "%s", drv->name);
Index: 2.6.12-rc4-mm1/include/asm-ppc64/pci.h
===================================================================
--- 2.6.12-rc4-mm1.orig/include/asm-ppc64/pci.h	2005-05-13 15:39:55.000000000 +1000
+++ 2.6.12-rc4-mm1/include/asm-ppc64/pci.h	2005-05-13 15:41:51.000000000 +1000
@@ -136,6 +136,13 @@
 					 unsigned long size,
 					 pgprot_t prot);
 
+#ifdef CONFIG_PPC_MULTIPLATFORM
+#define HAVE_ARCH_PCI_RESOURCE_TO_USER
+extern void pci_resource_to_user(const struct pci_dev *dev, int bar,
+				 const struct resource *rsrc,
+				 u64 *start, u64 *end);
+#endif /* CONFIG_PPC_MULTIPLATFORM */
+
 
 #endif	/* __KERNEL__ */
 
Index: 2.6.12-rc4-mm1/include/asm-ppc/pci.h
===================================================================
--- 2.6.12-rc4-mm1.orig/include/asm-ppc/pci.h	2005-05-13 15:39:55.000000000 +1000
+++ 2.6.12-rc4-mm1/include/asm-ppc/pci.h	2005-05-13 15:40:19.000000000 +1000
@@ -103,6 +103,12 @@
 					 unsigned long size,
 					 pgprot_t prot);
 
+#define HAVE_ARCH_PCI_RESOURCE_TO_USER
+extern void pci_resource_to_user(const struct pci_dev *dev, int bar,
+				 const struct resource *rsrc,
+				 u64 *start, u64 *end);
+
+
 #endif	/* __KERNEL__ */
 
 #endif /* __PPC_PCI_H */
Index: 2.6.12-rc4-mm1/include/linux/pci.h
===================================================================
--- 2.6.12-rc4-mm1.orig/include/linux/pci.h	2005-05-13 15:39:55.000000000 +1000
+++ 2.6.12-rc4-mm1/include/linux/pci.h	2005-05-13 15:43:33.000000000 +1000
@@ -1020,6 +1020,20 @@
 #define pci_pretty_name(dev) ""
 #endif
 
+
+/* Some archs don't want to expose struct resource to userland as-is
+ * in sysfs and /proc
+ */
+#ifndef HAVE_ARCH_PCI_RESOURCE_TO_USER
+static inline void pci_resource_to_user(const struct pci_dev *dev, int bar,
+                const struct resource *rsrc, u64 *start, u64 *end)
+{
+	*start = rsrc->start;
+	*end = rsrc->end;
+}
+#endif /* HAVE_ARCH_PCI_RESOURCE_TO_USER */
+
+
 /*
  *  The world is not perfect and supplies us with broken PCI devices.
  *  For at least a part of these bugs we need a work-around, so both

Re: [PATCH] Updated: fix-pci-mmap-on-ppc-and-ppc64.patch

[Benjamin Herrenschmidt]

On Fri, 2005-05-13 at 17:44 +1000, Michael Ellerman wrote:
> Hi Andrew,
> 
> This is an updated version of Ben's fix-pci-mmap-on-ppc-and-ppc64.patch
> which is in 2.6.12-rc4-mm1.
> 
> It fixes the patch to work on PPC iSeries, removes some debug printks
> at Ben's request, and incorporates your 
> fix-pci-mmap-on-ppc-and-ppc64-fix.patch also.
> 
> cheers
> 
> Signed-off-by: Michael Ellerman <michael@ellerman.id.au>

Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Re: [PATCH] Updated: fix-pci-mmap-on-ppc-and-ppc64.patch

[Greg KH]

On Fri, May 13, 2005 at 05:44:10PM +1000, Michael Ellerman wrote:
> Hi Andrew,

Hm, why not send this to the pci maintainer?

> This is an updated version of Ben's fix-pci-mmap-on-ppc-and-ppc64.patch
> which is in 2.6.12-rc4-mm1.
> 
> It fixes the patch to work on PPC iSeries, removes some debug printks
> at Ben's request, and incorporates your 
> fix-pci-mmap-on-ppc-and-ppc64-fix.patch also.

I'll add it to my queue.

greg k-h

Re: [PATCH] Updated: fix-pci-mmap-on-ppc-and-ppc64.patch

[Benjamin Herrenschmidt]

On Fri, 2005-05-13 at 09:18 -0700, Greg KH wrote:
> On Fri, May 13, 2005 at 05:44:10PM +1000, Michael Ellerman wrote:
> > Hi Andrew,
> 
> Hm, why not send this to the pci maintainer?

Because I told him to send it to Andrew as it's just an arch bugfix on
top of my patch that is already in mm ;)

> I'll add it to my queue.

Thanks,
Ben.