Messages in this thread | | | Date | Wed, 9 Jan 2008 01:30:34 +0100 | From | Ingo Molnar <> | Subject | Re: [PATCH 00 of 10] x86: unify asm/pgtable.h |
| |
* Ingo Molnar <mingo@elte.hu> wrote:
> > >>> #define __PAGE_KERNEL_EXEC \ > > >>> - (_PAGE_PRESENT | _PAGE_RW | _PAGE_DIRTY | _PAGE_ACCESSED | _PAGE_GLOBAL) > > >>> + (_PAGE_PRESENT | _PAGE_RW | _PAGE_DIRTY | _PAGE_ACCESSED) > > >>> > > > > > > This shouldn't be necessary. The old 64-bit code defined everything > > > without _PAGE_GLOBAL, but then used a MAKE_GLOBAL() macro to OR it > > > in later. This seemed a bit roundabout to me, so I just put it in > > > from the outset. > > actually, this is wrong. > > a couple of places use __PAGE_* values, which you've now changed to > include the _PAGE_GLOBAL flag.
yep, fixing this resolves the crash.
Ingo
| |