Messages in this thread | | | Subject | RE: [patch 02/11] PAT x86: Map only usable memory in x86_64 identity map and kernel text | Date | Thu, 10 Jan 2008 11:17:07 -0800 | From | "Pallipadi, Venkatesh" <> |
| |
>-----Original Message----- >From: linux-kernel-owner@vger.kernel.org >[mailto:linux-kernel-owner@vger.kernel.org] On Behalf Of Andi Kleen >Sent: Thursday, January 10, 2008 11:07 AM >To: Pallipadi, Venkatesh >Cc: ebiederm@xmission.com; rdreier@cisco.com; >torvalds@linux-foundation.org; gregkh@suse.de; >airlied@skynet.ie; davej@redhat.com; mingo@elte.hu; >tglx@linutronix.de; hpa@zytor.co; >linux-kernel@vger.kernel.org; Siddha, Suresh B >Subject: Re: [patch 02/11] PAT x86: Map only usable memory in >x86_64 identity map and kernel text > >venkatesh.pallipadi@intel.com writes: > >> x86_64: Map only usable memory in identity map. > >I don't think that is needed or makes sense for reserved/ACPI * etc. >Only e820 holes should be truly unmapped because only those should >contain mmio.
Do you mean just the regions that are not listed in e820 at all? We should also not map anything marked "RESERVED" in e820. Right?
>> All reserved memory maps to a >> zero page. > >Why zero page? Why not unmap.
I had it unmapped first. Then thought of zero mapping for dd of devmem to continue working. May be there are apps that depend on that? Also, dd of devmem seems to be already broken with big memory without any of these changes.
>Anyways you could make that a zillion times more simple by >just rounding >the e820 areas to 2MB -- for the holes only that should be ok I think; >i would expect them to be near always already suitably aligned. > >In short this can be all done much simpler.
On systems I tested, ACPI regions are typically not 2MB aligned. And on some systems there are few 4k pages of reserved holes just before 0xa0000. PCI reserved regions are 2MB aligned however. I agree that making this 2MB aligned will make this patch a lot simpler. But, not all reserved regions seems to be aligned that way.
Thanks, Venki
| |