The ebreak self test enables/disables guest debugging as a part of the
test. However the KVM_SET_GUEST_DEBUG ioctl doesn't actually do it.
Fixing it by calling kvm_riscv_vcpu_config_guest_debug.

Fixes: 6ed523e2b612 ("RISC-V: KVM: Factor-out VCPU config into separate sources")
Signed-off-by: Mayuresh Chitale <mayuresh.chitale@oss.qualcomm.com>
---
 arch/riscv/kvm/vcpu.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/riscv/kvm/vcpu.c b/arch/riscv/kvm/vcpu.c
index a73690eda84b..cf6e231e76e2 100644
--- a/arch/riscv/kvm/vcpu.c
+++ b/arch/riscv/kvm/vcpu.c
@@ -538,6 +538,7 @@ int kvm_arch_vcpu_ioctl_set_guest_debug(struct kvm_vcpu *vcpu,
 	else
 		vcpu->guest_debug = 0;
 
+	kvm_riscv_vcpu_config_guest_debug(vcpu);
 	return 0;
 }
 
-- 
2.43.0

On Mon, May 25, 2026 at 3:29 PM Mayuresh Chitale
<mayuresh.chitale@oss.qualcomm.com> wrote:
>
> The ebreak self test enables/disables guest debugging as a part of the
> test. However the KVM_SET_GUEST_DEBUG ioctl doesn't actually do it.
> Fixing it by calling kvm_riscv_vcpu_config_guest_debug.
>
> Fixes: 6ed523e2b612 ("RISC-V: KVM: Factor-out VCPU config into separate sources")
> Signed-off-by: Mayuresh Chitale <mayuresh.chitale@oss.qualcomm.com>

LGTM.

Reviewed-by: Anup Patel <anup@brainfault.org>

Queued this patch for Linux-7.2

Thanks,
Anup

> ---
>  arch/riscv/kvm/vcpu.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/riscv/kvm/vcpu.c b/arch/riscv/kvm/vcpu.c
> index a73690eda84b..cf6e231e76e2 100644
> --- a/arch/riscv/kvm/vcpu.c
> +++ b/arch/riscv/kvm/vcpu.c
> @@ -538,6 +538,7 @@ int kvm_arch_vcpu_ioctl_set_guest_debug(struct kvm_vcpu *vcpu,
>         else
>                 vcpu->guest_debug = 0;
>
> +       kvm_riscv_vcpu_config_guest_debug(vcpu);
>         return 0;
>  }
>
> --
> 2.43.0
>