Commit 26cdea48 authored by Alexandru Elisei's avatar Alexandru Elisei Committed by Oliver Upton
Browse files

KVM: arm64: Remove extra argument for __pvkm_host_{share,unshare}_hyp() 



__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: default avatarFuad Tabba <tabba@google.com>
Signed-off-by: default avatarAlexandru Elisei <alexandru.elisei@arm.com>
Reviewed-by: default avatarMarc Zyngier <maz@kernel.org>
Tested-by: default avatarFuad Tabba <tabba@google.com>
Link: https://msgid.link/20251216103053.47224-4-alexandru.elisei@arm.com


Signed-off-by: default avatarOliver Upton <oupton@kernel.org>
parent aba963cb

arch/arm64/kvm/mmu.c

+2 −2
Original line number Diff line number Diff line
@@ -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);