Re: [patch] x86, pat: fix reserve_memtype() for legacy 1MB range

From: Pavel Machek
Date: Sun Jan 18 2009 - 03:26:59 EST


On Sat 2009-01-17 15:24:39, H. Peter Anvin wrote:
> Pavel Machek wrote:
> > On Fri 2009-01-09 14:48:04, Suresh Siddha wrote:
> >> On Fri, Jan 09, 2009 at 02:39:31PM -0800, H. Peter Anvin wrote:
> >>> Suresh Siddha wrote:
> >>>> Here X is trying to map first 8KB of memory using /dev/mem. Existing
> >>>> code treats first 0-4KB of memory as non-RAM and 4KB-8KB as RAM. Recent
> >>>> code changes don't allow to map memory with different attributes
> >>>> at the same time.
> >>>>
> >>> Why was 0-4 KB marked as non-RAM? It is most definitely RAM, and should
> >>> be WB.
> >> While in reality it is RAM, we have CONFIG_STRICT_DEVMEM which doesn't allow
> >> apps to map RAM pages using /dev/mem. And to allow app's to map the
> >> legacy 0-4KB bios data page, we consider it as non-RAM.
> >
> > Fix config_strict_devmem? Ram is ram, and we should not li for
> > strict_devmem benefit...
>
> *As far as I understand* this is only considered non-RAM for the purpose
> of strict_devmem?

I did not look *too* closely, but the patch at the beggining of this
thread was for pat.c or something like that. I'd expect it to change
/dev/mem code...
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/