Commit 2311b1f2 authored by Michael Ellerman's avatar Michael Ellerman Committed by Greg Kroah-Hartman

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

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.

Originally 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: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: default avatarMichael Ellerman <michael@ellerman.id.au>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent a0d399a8
...@@ -1495,7 +1495,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev, ...@@ -1495,7 +1495,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev,
*offset += hose->pci_mem_offset; *offset += hose->pci_mem_offset;
res_bit = IORESOURCE_MEM; res_bit = IORESOURCE_MEM;
} else { } else {
io_offset = (unsigned long)hose->io_base_virt; io_offset = hose->io_base_virt - ___IO_BASE;
*offset += io_offset; *offset += io_offset;
res_bit = IORESOURCE_IO; res_bit = IORESOURCE_IO;
} }
...@@ -1522,7 +1522,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev, ...@@ -1522,7 +1522,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev,
/* found it! construct the final physical address */ /* found it! construct the final physical address */
if (mmap_state == pci_mmap_io) if (mmap_state == pci_mmap_io)
*offset += hose->io_base_phys - _IO_BASE; *offset += hose->io_base_phys - io_offset;
return rp; return rp;
} }
...@@ -1739,6 +1739,23 @@ long sys_pciconfig_iobase(long which, unsigned long bus, unsigned long devfn) ...@@ -1739,6 +1739,23 @@ long sys_pciconfig_iobase(long which, unsigned long bus, unsigned long devfn)
return result; 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 void __init
pci_init_resource(struct resource *res, unsigned long start, unsigned long end, pci_init_resource(struct resource *res, unsigned long start, unsigned long end,
int flags, char *name) int flags, char *name)
......
...@@ -351,7 +351,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev, ...@@ -351,7 +351,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev,
*offset += hose->pci_mem_offset; *offset += hose->pci_mem_offset;
res_bit = IORESOURCE_MEM; res_bit = IORESOURCE_MEM;
} else { } else {
io_offset = (unsigned long)hose->io_base_virt; io_offset = (unsigned long)hose->io_base_virt - pci_io_base;
*offset += io_offset; *offset += io_offset;
res_bit = IORESOURCE_IO; res_bit = IORESOURCE_IO;
} }
...@@ -378,7 +378,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev, ...@@ -378,7 +378,7 @@ static struct resource *__pci_mmap_make_offset(struct pci_dev *dev,
/* found it! construct the final physical address */ /* found it! construct the final physical address */
if (mmap_state == pci_mmap_io) if (mmap_state == pci_mmap_io)
*offset += hose->io_base_phys - io_offset; *offset += hose->io_base_phys - io_offset;
return rp; return rp;
} }
...@@ -944,4 +944,22 @@ int pci_read_irq_line(struct pci_dev *pci_dev) ...@@ -944,4 +944,22 @@ int pci_read_irq_line(struct pci_dev *pci_dev)
} }
EXPORT_SYMBOL(pci_read_irq_line); 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 */ #endif /* CONFIG_PPC_MULTIPLATFORM */
...@@ -60,15 +60,18 @@ resource_show(struct device * dev, struct device_attribute *attr, char * buf) ...@@ -60,15 +60,18 @@ resource_show(struct device * dev, struct device_attribute *attr, char * buf)
char * str = buf; char * str = buf;
int i; int i;
int max = 7; int max = 7;
u64 start, end;
if (pci_dev->subordinate) if (pci_dev->subordinate)
max = DEVICE_COUNT_RESOURCE; max = DEVICE_COUNT_RESOURCE;
for (i = 0; i < max; i++) { for (i = 0; i < max; i++) {
str += sprintf(str,"0x%016lx 0x%016lx 0x%016lx\n", struct resource *res = &pci_dev->resource[i];
pci_resource_start(pci_dev,i), pci_resource_to_user(pci_dev, i, res, &start, &end);
pci_resource_end(pci_dev,i), str += sprintf(str,"0x%016llx 0x%016llx 0x%016llx\n",
pci_resource_flags(pci_dev,i)); (unsigned long long)start,
(unsigned long long)end,
(unsigned long long)res->flags);
} }
return (str - buf); return (str - buf);
} }
...@@ -313,8 +316,21 @@ pci_mmap_resource(struct kobject *kobj, struct bin_attribute *attr, ...@@ -313,8 +316,21 @@ pci_mmap_resource(struct kobject *kobj, struct bin_attribute *attr,
struct device, kobj)); struct device, kobj));
struct resource *res = (struct resource *)attr->private; struct resource *res = (struct resource *)attr->private;
enum pci_mmap_state mmap_type; 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; mmap_type = res->flags & IORESOURCE_MEM ? pci_mmap_mem : pci_mmap_io;
return pci_mmap_page_range(pdev, vma, mmap_type, 0); return pci_mmap_page_range(pdev, vma, mmap_type, 0);
......
...@@ -355,14 +355,20 @@ static int show_device(struct seq_file *m, void *v) ...@@ -355,14 +355,20 @@ static int show_device(struct seq_file *m, void *v)
dev->device, dev->device,
dev->irq); dev->irq);
/* Here should be 7 and not PCI_NUM_RESOURCES as we need to preserve compatibility */ /* 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, seq_printf(m, LONG_FORMAT,
dev->resource[i].start | ((unsigned long)start) |
(dev->resource[i].flags & PCI_REGION_FLAG_MASK)); (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, seq_printf(m, LONG_FORMAT,
dev->resource[i].start < dev->resource[i].end ? 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'); seq_putc(m, '\t');
if (drv) if (drv)
seq_printf(m, "%s", drv->name); seq_printf(m, "%s", drv->name);
......
...@@ -103,6 +103,12 @@ extern pgprot_t pci_phys_mem_access_prot(struct file *file, ...@@ -103,6 +103,12 @@ extern pgprot_t pci_phys_mem_access_prot(struct file *file,
unsigned long size, unsigned long size,
pgprot_t prot); 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 /* __KERNEL__ */
#endif /* __PPC_PCI_H */ #endif /* __PPC_PCI_H */
...@@ -136,6 +136,13 @@ extern pgprot_t pci_phys_mem_access_prot(struct file *file, ...@@ -136,6 +136,13 @@ extern pgprot_t pci_phys_mem_access_prot(struct file *file,
unsigned long size, unsigned long size,
pgprot_t prot); 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__ */ #endif /* __KERNEL__ */
......
...@@ -1020,6 +1020,20 @@ static inline char *pci_name(struct pci_dev *pdev) ...@@ -1020,6 +1020,20 @@ static inline char *pci_name(struct pci_dev *pdev)
#define pci_pretty_name(dev) "" #define pci_pretty_name(dev) ""
#endif #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. * 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 * For at least a part of these bugs we need a work-around, so both
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment