nEPT: Redefine EPT-specific link_shadow_page()
authorYang Zhang <yang.z.zhang@Intel.com>
Mon, 5 Aug 2013 08:07:13 +0000 (11:07 +0300)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 7 Aug 2013 13:57:39 +0000 (15:57 +0200)
Since nEPT doesn't support A/D bit, so we should not set those bit
when build shadow page table.

Reviewed-by: Xiao Guangrong <xiaoguangrong@linux.vnet.ibm.com>
Signed-off-by: Yang Zhang <yang.z.zhang@Intel.com>
Signed-off-by: Gleb Natapov <gleb@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/mmu.c
arch/x86/kvm/paging_tmpl.h

index 2988df5c1e3ea16a3832abd5c8124d899485336f..68d0f086de6cbc1ed587ebf3aeec4605dea07d8f 100644 (file)
@@ -2047,12 +2047,18 @@ static void shadow_walk_next(struct kvm_shadow_walk_iterator *iterator)
        return __shadow_walk_next(iterator, *iterator->sptep);
 }
 
-static void link_shadow_page(u64 *sptep, struct kvm_mmu_page *sp)
+static void link_shadow_page(u64 *sptep, struct kvm_mmu_page *sp, bool accessed)
 {
        u64 spte;
 
+       BUILD_BUG_ON(VMX_EPT_READABLE_MASK != PT_PRESENT_MASK ||
+                       VMX_EPT_WRITABLE_MASK != PT_WRITABLE_MASK);
+
        spte = __pa(sp->spt) | PT_PRESENT_MASK | PT_WRITABLE_MASK |
-              shadow_user_mask | shadow_x_mask | shadow_accessed_mask;
+              shadow_user_mask | shadow_x_mask;
+
+       if (accessed)
+               spte |= shadow_accessed_mask;
 
        mmu_spte_set(sptep, spte);
 }
@@ -2677,7 +2683,7 @@ static int __direct_map(struct kvm_vcpu *vcpu, gpa_t v, int write,
                                              iterator.level - 1,
                                              1, ACC_ALL, iterator.sptep);
 
-                       link_shadow_page(iterator.sptep, sp);
+                       link_shadow_page(iterator.sptep, sp, true);
                }
        }
        return emulate;
index 893768825725e853e3496b9ffe9c2abb8101306b..50b8679d4dd8946c0efe247f88d774dab3825d55 100644 (file)
@@ -555,7 +555,7 @@ static int FNAME(fetch)(struct kvm_vcpu *vcpu, gva_t addr,
                        goto out_gpte_changed;
 
                if (sp)
-                       link_shadow_page(it.sptep, sp);
+                       link_shadow_page(it.sptep, sp, PT_GUEST_ACCESSED_MASK);
        }
 
        for (;
@@ -575,7 +575,7 @@ static int FNAME(fetch)(struct kvm_vcpu *vcpu, gva_t addr,
 
                sp = kvm_mmu_get_page(vcpu, direct_gfn, addr, it.level-1,
                                      true, direct_access, it.sptep);
-               link_shadow_page(it.sptep, sp);
+               link_shadow_page(it.sptep, sp, PT_GUEST_ACCESSED_MASK);
        }
 
        clear_sp_write_flooding_count(it.sptep);