On Sun, Sep 10, 2023 at 04:29:11AM -0400, guoren@xxxxxxxxxx wrote:
> From: Guo Ren <guoren@xxxxxxxxxxxxxxxxx>
>
> We only need to call the kvm_vcpu_kick() and bring target_vcpu
> from the halt state. No irq raised, no other request, just a pure
> vcpu_kick.
>
> Signed-off-by: Guo Ren <guoren@xxxxxxxxxxxxxxxxx>
> Signed-off-by: Guo Ren <guoren@xxxxxxxxxx>
> ---
> arch/riscv/kvm/vcpu_sbi_pvlock.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/arch/riscv/kvm/vcpu_sbi_pvlock.c b/arch/riscv/kvm/vcpu_sbi_pvlock.c
> index 544a456c5041..914fc58aedfe 100644
> --- a/arch/riscv/kvm/vcpu_sbi_pvlock.c
> +++ b/arch/riscv/kvm/vcpu_sbi_pvlock.c
> @@ -12,6 +12,24 @@
> #include <asm/sbi.h>
> #include <asm/kvm_vcpu_sbi.h>
>
> +static int kvm_sbi_ext_pvlock_kick_cpu(struct kvm_vcpu *vcpu)
> +{
> + struct kvm_cpu_context *cp = &vcpu->arch.guest_context;
> + struct kvm *kvm = vcpu->kvm;
> + struct kvm_vcpu *target;
> +
> + target = kvm_get_vcpu_by_id(kvm, cp->a0);
> + if (!target)
> + return SBI_ERR_INVALID_PARAM;
> +
> + kvm_vcpu_kick(target);
> +
> + if (READ_ONCE(target->ready))
> + kvm_vcpu_yield_to(target);
> +
> + return SBI_SUCCESS;
> +}
> +
> static int kvm_sbi_ext_pvlock_handler(struct kvm_vcpu *vcpu, struct kvm_run *run,
> struct kvm_vcpu_sbi_return *retdata)
> {
> @@ -21,6 +39,7 @@ static int kvm_sbi_ext_pvlock_handler(struct kvm_vcpu *vcpu, struct kvm_run *run
>
> switch (funcid) {
> case SBI_EXT_PVLOCK_KICK_CPU:
> + ret = kvm_sbi_ext_pvlock_kick_cpu(vcpu);
> break;
> default:
> ret = SBI_ERR_NOT_SUPPORTED;
> --
> 2.36.1
>
LGTM:
Reviewed-by: Leonardo Bras <leobras@xxxxxxxxxx>
Thanks!
Leo
