* [PATCH m68k resent] allow ioremapping top of memory
@ 2009-12-06 19:28 Philippe De Muyter
2009-12-22 19:08 ` Geert Uytterhoeven
0 siblings, 1 reply; 2+ messages in thread
From: Philippe De Muyter @ 2009-12-06 19:28 UTC (permalink / raw)
To: linux-kernel, linux-m68k
Hi Geert and all,
The test in __ioremap to reject memory ranges crossing the 0 boundary
rejects also memory ranges ending at the end of the memory. Fix that.
Signed-off-by: Philippe De Muyter <phdm@macqel.be>
diff --git a/arch/m68k/mm/kmap.c b/arch/m68k/mm/kmap.c
--- a/arch/m68k/mm/kmap.c
+++ b/arch/m68k/mm/kmap.c
@@ -116,7 +115,7 @@ void __iomem *__ioremap(unsigned long ph
/*
* Don't allow mappings that wrap..
*/
- if (!size || size > physaddr + size)
+ if (!size || physaddr > (unsigned long)(-size))
return NULL;
#ifdef CONFIG_AMIGA
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH m68k resent] allow ioremapping top of memory
2009-12-06 19:28 [PATCH m68k resent] allow ioremapping top of memory Philippe De Muyter
@ 2009-12-22 19:08 ` Geert Uytterhoeven
0 siblings, 0 replies; 2+ messages in thread
From: Geert Uytterhoeven @ 2009-12-22 19:08 UTC (permalink / raw)
To: Philippe De Muyter; +Cc: linux-kernel, linux-m68k
On Sun, Dec 6, 2009 at 20:28, Philippe De Muyter <phdm@macqel.be> wrote:
> The test in __ioremap to reject memory ranges crossing the 0 boundary
> rejects also memory ranges ending at the end of the memory. Fix that.
Thanks, will take it.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-12-22 19:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-12-06 19:28 [PATCH m68k resent] allow ioremapping top of memory Philippe De Muyter
2009-12-22 19:08 ` Geert Uytterhoeven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox