diff options
author | Geert Uytterhoeven <geert@linux-m68k.org> | 2011-05-07 15:08:36 +0200 |
---|---|---|
committer | Geert Uytterhoeven <geert@linux-m68k.org> | 2011-07-30 21:21:38 +0200 |
commit | 88efd0bbc0fe403a9948e6f94cc48b9f15ee4861 (patch) | |
tree | 932a4f41f7ae679e3c0c35f694aa2b147546924a /arch/sparc/kernel/ioport.c | |
parent | 1dad6c7bd7dd158ef874f7382615cedc21a1f48d (diff) | |
download | op-kernel-dev-88efd0bbc0fe403a9948e6f94cc48b9f15ee4861.zip op-kernel-dev-88efd0bbc0fe403a9948e6f94cc48b9f15ee4861.tar.gz |
sparc: _sparc_find_resource() should check for exact matches
The address that's passed to _sparc_find_resource() should always be the
start address of a resource:
- iounmap() passes a page-aligned virtual address, while the original
address was created by adding the in-page offset to the resource's
start address,
- sbus_free_coherent() and pci32_free_coherent() should be passed an
address obtained from sbus_alloc_coherent() resp. pci32_alloc_coherent(),
which is always a resource's start address.
Hence replace the range check by a check for an exact match.
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'arch/sparc/kernel/ioport.c')
-rw-r--r-- | arch/sparc/kernel/ioport.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/sparc/kernel/ioport.c b/arch/sparc/kernel/ioport.c index 1c9c80a..b5e83be 100644 --- a/arch/sparc/kernel/ioport.c +++ b/arch/sparc/kernel/ioport.c @@ -723,12 +723,12 @@ static const struct file_operations sparc_io_proc_fops = { * This probably warrants some sort of hashing. */ static struct resource *_sparc_find_resource(struct resource *root, - unsigned long hit) + unsigned long start) { struct resource *tmp; for (tmp = root->child; tmp != 0; tmp = tmp->sibling) { - if (tmp->start <= hit && tmp->end >= hit) + if (tmp->start == start) return tmp; } return NULL; |