[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <48DE8851.9070900@linux.intel.com>
Date: Sat, 27 Sep 2008 12:24:01 -0700
From: Arjan van de Ven <arjan@...ux.intel.com>
To: Alan Cox <alan@...rguk.ukuu.org.uk>
CC: Jeremy Fitzhardinge <jeremy@...p.org>,
Suresh Siddha <suresh.b.siddha@...el.com>,
jbarnes@...tuousgeek.org, mingo@...e.hu, tglx@...utronix.de,
hpa@...or.com, torvalds@...ux-foundation.org,
akpm@...ux-foundation.org, linux-kernel@...r.kernel.org
Subject: Re: [patch] ioremap sanity check to catch mapping requests exceeding
the BAR sizes
Alan Cox wrote:
> On Sat, 27 Sep 2008 07:43:55 -0700
> Jeremy Fitzhardinge <jeremy@...p.org> wrote:
>
>> Alan Cox wrote:
>>>> Any attempt to use ioremap on memory is a bug, so you should warn about
>>>> that too.
>>>>
>>> We use it to ioremap things like the BIOS...
>> Yeah, that's fine. I mean using ioremap on system memory is a bug.
>
> On system memory mapped by the kernel which I assume is what you mean -
> the BIOS is often shadowed system memory and we have platforms where
> memory pages not mapped into or managed the OS are ioremap() targets.
>
> That however is getting pedantic - I just didn't want anyone to overdo
> the sanity checks
the existing sanity check checks to see if the kernel thinks if the memory is usable
for its own use, eg it uses the e820 table, the same one we use for feeding all memory
into the page allocator.
reserved and other similar types is not what ioremap complains about
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists