KVM: MMU: Eliminate pointless temporary 'ac'
authorAvi Kivity <avi@redhat.com>
Wed, 19 Sep 2012 16:33:48 +0000 (19:33 +0300)
committerAvi Kivity <avi@redhat.com>
Thu, 20 Sep 2012 10:00:10 +0000 (13:00 +0300)
'ac' essentially reconstructs the 'access' variable we already
have, except for the PFERR_PRESENT_MASK and PFERR_RSVD_MASK.  As
these are not used by callees, just use 'access' directly.

Reviewed-by: Xiao Guangrong <xiaoguangrong@linux.vnet.ibm.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
arch/x86/kvm/paging_tmpl.h

index 810c1da2ee44f26d9dd9939e53ef0e76aad20b12..714e2c01a6fe0fd714f66146c8613f17f31f4a68 100644 (file)
@@ -160,7 +160,6 @@ static int FNAME(walk_addr_generic)(struct guest_walker *walker,
        u16 errcode = 0;
        gpa_t real_gpa;
        gfn_t gfn;
-       u32 ac;
 
        trace_kvm_mmu_pagetable_walk(addr, access);
 retry_walk:
@@ -242,9 +241,7 @@ retry_walk:
        if (PTTYPE == 32 && walker->level == PT_DIRECTORY_LEVEL && is_cpuid_PSE36())
                gfn += pse36_gfn_delta(pte);
 
-       ac = write_fault | fetch_fault | user_fault;
-
-       real_gpa = mmu->translate_gpa(vcpu, gfn_to_gpa(gfn), ac);
+       real_gpa = mmu->translate_gpa(vcpu, gfn_to_gpa(gfn), access);
        if (real_gpa == UNMAPPED_GVA)
                return 0;