From: Fuad Tabba <tabba@google.com>
Date: Mon, 31 Jul 2023 11:40:31 +0000 (+0100)
Subject: KVM: arm64: Remove redundant kvm_set_pfn_accessed() from user_mem_abort()
X-Git-Tag: v6.6.17~3972^2~9^2^2~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4460a7dc77d00340c59f7e4252c65efd5fbe877b;p=platform%2Fkernel%2Flinux-rpi.git

KVM: arm64: Remove redundant kvm_set_pfn_accessed() from user_mem_abort()

The function user_mem_abort() calls kvm_release_pfn_clean(),
which eventually calls kvm_set_page_accessed(). Therefore, remove
the redundant call to kvm_set_pfn_accessed().

Signed-off-by: Fuad Tabba <tabba@google.com>
Reviewed-by: Shaoqin Huang <shahuang@redhat.com>
Signed-off-by: Marc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20230731114110.2673451-1-tabba@google.com
---

diff --git a/arch/arm64/kvm/mmu.c b/arch/arm64/kvm/mmu.c
index d3b4fee..137b775 100644
--- a/arch/arm64/kvm/mmu.c
+++ b/arch/arm64/kvm/mmu.c
@@ -1541,7 +1541,6 @@ static int user_mem_abort(struct kvm_vcpu *vcpu, phys_addr_t fault_ipa,
 
 out_unlock:
 	read_unlock(&kvm->mmu_lock);
-	kvm_set_pfn_accessed(pfn);
 	kvm_release_pfn_clean(pfn);
 	return ret != -EAGAIN ? ret : 0;
 }