]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
KVM: arm64: Remove extra argument for __pvkm_host_{share,unshare}_hyp()
authorAlexandru Elisei <alexandru.elisei@arm.com>
Tue, 16 Dec 2025 10:30:52 +0000 (10:30 +0000)
committerOliver Upton <oupton@kernel.org>
Thu, 8 Jan 2026 20:56:17 +0000 (12:56 -0800)
__pvkm_host_share_hyp() and __pkvm_host_unshare_hyp() both have one
parameter, the pfn, not two. Even though correctness isn't impacted because
the SMCCC handlers pass the first argument and ignore the second one, let's
call the functions with the proper number of arguments.

Reviewed-by: Fuad Tabba <tabba@google.com>
Signed-off-by: Alexandru Elisei <alexandru.elisei@arm.com>
Reviewed-by: Marc Zyngier <maz@kernel.org>
Tested-by: Fuad Tabba <tabba@google.com>
Link: https://msgid.link/20251216103053.47224-4-alexandru.elisei@arm.com
Signed-off-by: Oliver Upton <oupton@kernel.org>
arch/arm64/kvm/mmu.c

index 48d7c372a4cd02da5ea7a7cc0f685e8617a682dd..124404eb208d3b5af06fdee7415a80caff6a565f 100644 (file)
@@ -497,7 +497,7 @@ static int share_pfn_hyp(u64 pfn)
        this->count = 1;
        rb_link_node(&this->node, parent, node);
        rb_insert_color(&this->node, &hyp_shared_pfns);
-       ret = kvm_call_hyp_nvhe(__pkvm_host_share_hyp, pfn, 1);
+       ret = kvm_call_hyp_nvhe(__pkvm_host_share_hyp, pfn);
 unlock:
        mutex_unlock(&hyp_shared_pfns_lock);
 
@@ -523,7 +523,7 @@ static int unshare_pfn_hyp(u64 pfn)
 
        rb_erase(&this->node, &hyp_shared_pfns);
        kfree(this);
-       ret = kvm_call_hyp_nvhe(__pkvm_host_unshare_hyp, pfn, 1);
+       ret = kvm_call_hyp_nvhe(__pkvm_host_unshare_hyp, pfn);
 unlock:
        mutex_unlock(&hyp_shared_pfns_lock);