[PATCH v2 3/4] KVM: arm64: Remove extra argument for __pvkm_host_{share,unshare}_hyp()

[Fuad Tabba]

On Mon, 15 Dec 2025 at 11:44, Alexandru Elisei <alexandru.elisei at arm.com> wrote:
>
> __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.
>
> Signed-off-by: Alexandru Elisei <alexandru.elisei at arm.com>
> ---

Reviewed-by: Fuad Tabba <tabba at google.com>

Cheers,
/fuad

>  arch/arm64/kvm/mmu.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/kvm/mmu.c b/arch/arm64/kvm/mmu.c
> index 48d7c372a4cd..124404eb208d 100644
> --- a/arch/arm64/kvm/mmu.c
> +++ b/arch/arm64/kvm/mmu.c
> @@ -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);
>
> --
> 2.52.0
>