Remove unneeded variable used to store return value. Replace `gpa >> PAGE_SHIFT` with `gpa_to_gfn(gpa)` No functional change intended. Suggested-by: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx> Signed-off-by: Quanfa Fu <quanfafu@xxxxxxxxx> --- arch/x86/kvm/mmu/mmu.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/arch/x86/kvm/mmu/mmu.c b/arch/x86/kvm/mmu/mmu.c index e2e1d012df22..61e35170ee2e 100644 --- a/arch/x86/kvm/mmu/mmu.c +++ b/arch/x86/kvm/mmu/mmu.c @@ -2553,16 +2553,13 @@ int kvm_mmu_unprotect_page(struct kvm *kvm, gfn_t gfn) static int kvm_mmu_unprotect_page_virt(struct kvm_vcpu *vcpu, gva_t gva) { gpa_t gpa; - int r; if (vcpu->arch.mmu->direct_map) return 0; gpa = kvm_mmu_gva_to_gpa_read(vcpu, gva, NULL); - r = kvm_mmu_unprotect_page(vcpu->kvm, gpa >> PAGE_SHIFT); - - return r; + return kvm_mmu_unprotect_page(vcpu->kvm, gpa_to_gfn(gpa)); } static void kvm_unsync_page(struct kvm_vcpu *vcpu, struct kvm_mmu_page *sp) -- 2.25.1