From: William Grant Date: Sun, 24 Aug 2014 05:13:48 +0000 (+1000) Subject: target-i386: Don't forbid NX bit on PAE PDEs and PTEs X-Git-Tag: v2.1.1~24 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=72c9c9a05e79d4638bb248bc1cc903839b8f676f;p=thirdparty%2Fqemu.git target-i386: Don't forbid NX bit on PAE PDEs and PTEs Commit e8f6d00c30ed88910d0d985f4b2bf41654172ceb ("target-i386: raise page fault for reserved physical address bits") added a check that the NX bit is not set on PAE PDPEs, but it also added it to rsvd_mask for the rest of the function. This caused any PDEs or PTEs with NX set to be erroneously rejected, making PAE guests with NX support unusable. Signed-off-by: William Grant Cc: qemu-stable@nongnu.org Signed-off-by: Paolo Bonzini (cherry picked from commit 1844e68ecabbdfdf0228774bcd5cf0f63ffc2e57) Signed-off-by: Michael Roth --- diff --git a/target-i386/helper.c b/target-i386/helper.c index 47b982b4376..30cb0d01432 100644 --- a/target-i386/helper.c +++ b/target-i386/helper.c @@ -615,8 +615,8 @@ int x86_cpu_handle_mmu_fault(CPUState *cs, vaddr addr, if (!(pdpe & PG_PRESENT_MASK)) { goto do_fault; } - rsvd_mask |= PG_HI_USER_MASK | PG_NX_MASK; - if (pdpe & rsvd_mask) { + rsvd_mask |= PG_HI_USER_MASK; + if (pdpe & (rsvd_mask | PG_NX_MASK)) { goto do_fault_rsvd; } ptep = PG_NX_MASK | PG_USER_MASK | PG_RW_MASK;