diff options
author | Avi Kivity <avi@qumranet.com> | 2008-08-27 13:01:04 -0400 |
---|---|---|
committer | Avi Kivity <avi@qumranet.com> | 2008-10-15 04:15:24 -0400 |
commit | 3201b5d9f0f7ef392886cd76dcd2c69186d9d5cd (patch) | |
tree | 94c726dfc773f551a9b8ad600193f815b6619d2c /arch/x86/kvm/mmu.c | |
parent | 171d595d3b3254b9a952af8d1f6965d2e85dcbaa (diff) |
KVM: MMU: Fix setting the accessed bit on non-speculative sptes
The accessed bit was accidentally turned on in a random flag word, rather
than, the spte itself, which was lucky, since it used the non-EPT compatible
PT_ACCESSED_MASK.
Fix by turning the bit on in the spte and changing it to use the portable
accessed mask.
Signed-off-by: Avi Kivity <avi@qumranet.com>
Diffstat (limited to 'arch/x86/kvm/mmu.c')
-rw-r--r-- | arch/x86/kvm/mmu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c index 853a2889b202..866d7133cad8 100644 --- a/arch/x86/kvm/mmu.c +++ b/arch/x86/kvm/mmu.c | |||
@@ -1192,7 +1192,7 @@ static void mmu_set_spte(struct kvm_vcpu *vcpu, u64 *shadow_pte, | |||
1192 | */ | 1192 | */ |
1193 | spte = shadow_base_present_pte | shadow_dirty_mask; | 1193 | spte = shadow_base_present_pte | shadow_dirty_mask; |
1194 | if (!speculative) | 1194 | if (!speculative) |
1195 | pte_access |= PT_ACCESSED_MASK; | 1195 | spte |= shadow_accessed_mask; |
1196 | if (!dirty) | 1196 | if (!dirty) |
1197 | pte_access &= ~ACC_WRITE_MASK; | 1197 | pte_access &= ~ACC_WRITE_MASK; |
1198 | if (pte_access & ACC_EXEC_MASK) | 1198 | if (pte_access & ACC_EXEC_MASK) |