KVM: MMU: cleanup for FNAME(fetch)
authorXiao Guangrong <xiaoguangrong@cn.fujitsu.com>
Mon, 11 Jul 2011 19:25:19 +0000 (03:25 +0800)
committerAvi Kivity <avi@redhat.com>
Sun, 24 Jul 2011 08:50:29 +0000 (11:50 +0300)
gw->pte_access is the final access permission, since it is unified with
gw->pt_access when we walked guest page table:

FNAME(walk_addr_generic):
pte_access = pt_access & FNAME(gpte_access)(vcpu, pte, true);

Signed-off-by: Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
arch/x86/kvm/paging_tmpl.h

index c9fe97b..5c2aa40 100644 (file)
@@ -479,7 +479,7 @@ static u64 *FNAME(fetch)(struct kvm_vcpu *vcpu, gva_t addr,
        if (!is_present_gpte(gw->ptes[gw->level - 1]))
                return NULL;
 
-       direct_access = gw->pt_access & gw->pte_access;
+       direct_access = gw->pte_access;
 
        top_level = vcpu->arch.mmu.root_level;
        if (top_level == PT32E_ROOT_LEVEL)
@@ -537,7 +537,7 @@ static u64 *FNAME(fetch)(struct kvm_vcpu *vcpu, gva_t addr,
                link_shadow_page(it.sptep, sp);
        }
 
-       mmu_set_spte(vcpu, it.sptep, access, gw->pte_access & access,
+       mmu_set_spte(vcpu, it.sptep, access, gw->pte_access,
                     user_fault, write_fault, ptwrite, it.level,
                     gw->gfn, pfn, prefault, map_writable);
        FNAME(pte_prefetch)(vcpu, gw, it.sptep);