diff mbox series

[v4,8/9b] target/loongarch: Implement set vcpu intr for kvm

Message ID 20240110094152.52138-2-philmd@linaro.org
State New
Headers show
Series [v4,8/9b] target/loongarch: Implement set vcpu intr for kvm | expand

Commit Message

Philippe Mathieu-Daudé Jan. 10, 2024, 9:41 a.m. UTC
From: Tianrui Zhao <zhaotianrui@loongson.cn>

Implement loongarch kvm set vcpu interrupt interface,
when a irq is set in vcpu, we use the KVM_INTERRUPT
ioctl to set intr into kvm.

Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn>
Signed-off-by: xianglai li <lixianglai@loongson.cn>
Reviewed-by: Song Gao <gaosong@loongson.cn>
Message-ID: <20240105075804.1228596-9-zhaotianrui@loongson.cn>
[PMD: Split from bigger patch, part 2]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 target/loongarch/kvm/kvm_loongarch.h | 16 ++++++++++++++++
 target/loongarch/cpu.c               |  9 ++++++++-
 target/loongarch/kvm/kvm.c           | 15 +++++++++++++++
 target/loongarch/trace-events        |  1 +
 4 files changed, 40 insertions(+), 1 deletion(-)
 create mode 100644 target/loongarch/kvm/kvm_loongarch.h

Comments

Song Gao Jan. 11, 2024, 2:29 a.m. UTC | #1
Hi,

在 2024/1/10 下午5:41, Philippe Mathieu-Daudé 写道:
> From: Tianrui Zhao <zhaotianrui@loongson.cn>
>
> Implement loongarch kvm set vcpu interrupt interface,
> when a irq is set in vcpu, we use the KVM_INTERRUPT
> ioctl to set intr into kvm.
>
> Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn>
> Signed-off-by: xianglai li <lixianglai@loongson.cn>
> Reviewed-by: Song Gao <gaosong@loongson.cn>
> Message-ID: <20240105075804.1228596-9-zhaotianrui@loongson.cn>
> [PMD: Split from bigger patch, part 2]
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   target/loongarch/kvm/kvm_loongarch.h | 16 ++++++++++++++++
>   target/loongarch/cpu.c               |  9 ++++++++-
>   target/loongarch/kvm/kvm.c           | 15 +++++++++++++++
>   target/loongarch/trace-events        |  1 +
>   4 files changed, 40 insertions(+), 1 deletion(-)
>   create mode 100644 target/loongarch/kvm/kvm_loongarch.h
>
> diff --git a/target/loongarch/kvm/kvm_loongarch.h b/target/loongarch/kvm/kvm_loongarch.h
> new file mode 100644
> index 0000000000..d945b6bb82
> --- /dev/null
> +++ b/target/loongarch/kvm/kvm_loongarch.h
> @@ -0,0 +1,16 @@
> +/* SPDX-License-Identifier: GPL-2.0-or-later */
> +/*
> + * QEMU LoongArch kvm interface
> + *
> + * Copyright (c) 2023 Loongson Technology Corporation Limited
> + */
> +
> +#include "cpu.h"
> +
> +#ifndef QEMU_KVM_LOONGARCH_H
> +#define QEMU_KVM_LOONGARCH_H
> +
> +int  kvm_loongarch_set_interrupt(LoongArchCPU *cpu, int irq, int level);
> +void kvm_arch_reset_vcpu(CPULoongArchState *env);
> +
> +#endif
> diff --git a/target/loongarch/cpu.c b/target/loongarch/cpu.c
> index d9f8661cfd..d3a8a2f521 100644
> --- a/target/loongarch/cpu.c
> +++ b/target/loongarch/cpu.c
> @@ -12,6 +12,7 @@
>   #include "qemu/module.h"
>   #include "sysemu/qtest.h"
>   #include "sysemu/tcg.h"
> +#include "sysemu/kvm.h"
>   #include "exec/exec-all.h"
>   #include "cpu.h"
>   #include "internals.h"
> @@ -21,6 +22,10 @@
>   #include "sysemu/reset.h"
>   #endif
>   #include "vec.h"
> +#ifdef CONFIG_KVM
> +#include "kvm/kvm_loongarch.h"

This broken  tcg 'loongarch64-softmmu' build on X86 host, :-[

../target/loongarch/cpu.c: In function ‘loongarch_cpu_set_irq’:
../target/loongarch/cpu.c:122:9: error: implicit declaration of function 
‘kvm_loongarch_set_interrupt’ [-Werror=implicit-function-declaration]
   122 |         kvm_loongarch_set_interrupt(cpu, irq, level);
       |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~
../target/loongarch/cpu.c:122:9: error: nested extern declaration of 
‘kvm_loongarch_set_interrupt’ [-Werror=nested-externs]
../target/loongarch/cpu.c: In function ‘loongarch_cpu_reset_hold’:
../target/loongarch/cpu.c:557:9: error: implicit declaration of function 
‘kvm_arch_reset_vcpu’; did you mean ‘kvm_arch_init_vcpu’? 
[-Werror=implicit-function-declaration]
   557 |         kvm_arch_reset_vcpu(env);
       |         ^~~~~~~~~~~~~~~~~~~
       |         kvm_arch_init_vcpu
../target/loongarch/cpu.c:557:9: error: nested extern declaration of 
‘kvm_arch_reset_vcpu’ [-Werror=nested-externs]
cc1: all warnings being treated as errors

I will move it out of  '#ifdef CONFIG_KVM'

Thanks.
Song Gao
> +#include <linux/kvm.h>
> +#endif
>   #ifdef CONFIG_TCG
>   #include "exec/cpu_ldst.h"
>   #include "tcg/tcg.h"
> @@ -113,7 +118,9 @@ void loongarch_cpu_set_irq(void *opaque, int irq, int level)
>           return;
>       }
>   
> -    if (tcg_enabled()) {
> +    if (kvm_enabled()) {
> +        kvm_loongarch_set_interrupt(cpu, irq, level);
> +    } else if (tcg_enabled()) {
>           env->CSR_ESTAT = deposit64(env->CSR_ESTAT, irq, 1, level != 0);
>           if (FIELD_EX64(env->CSR_ESTAT, CSR_ESTAT, IS)) {
>               cpu_interrupt(cs, CPU_INTERRUPT_HARD);
> diff --git a/target/loongarch/kvm/kvm.c b/target/loongarch/kvm/kvm.c
> index d2dab3fef4..bd33ec2114 100644
> --- a/target/loongarch/kvm/kvm.c
> +++ b/target/loongarch/kvm/kvm.c
> @@ -748,6 +748,21 @@ int kvm_arch_handle_exit(CPUState *cs, struct kvm_run *run)
>       return ret;
>   }
>   
> +int kvm_loongarch_set_interrupt(LoongArchCPU *cpu, int irq, int level)
> +{
> +    struct kvm_interrupt intr;
> +    CPUState *cs = CPU(cpu);
> +
> +    if (level) {
> +        intr.irq = irq;
> +    } else {
> +        intr.irq = -irq;
> +    }
> +
> +    trace_kvm_set_intr(irq, level);
> +    return kvm_vcpu_ioctl(cs, KVM_INTERRUPT, &intr);
> +}
> +
>   void kvm_arch_accel_class_init(ObjectClass *oc)
>   {
>   }
> diff --git a/target/loongarch/trace-events b/target/loongarch/trace-events
> index 021839880e..dea11edc0f 100644
> --- a/target/loongarch/trace-events
> +++ b/target/loongarch/trace-events
> @@ -12,3 +12,4 @@ kvm_failed_put_counter(const char *msg) "Failed to put counter into KVM: %s"
>   kvm_failed_get_cpucfg(const char *msg) "Failed to get cpucfg from KVM: %s"
>   kvm_failed_put_cpucfg(const char *msg) "Failed to put cpucfg into KVM: %s"
>   kvm_arch_handle_exit(int num) "kvm arch handle exit, the reason number: %d"
> +kvm_set_intr(int irq, int level) "kvm set interrupt, irq num: %d, level: %d"
Philippe Mathieu-Daudé Jan. 11, 2024, 8:24 a.m. UTC | #2
On 11/1/24 03:29, gaosong wrote:
> Hi,
> 
> 在 2024/1/10 下午5:41, Philippe Mathieu-Daudé 写道:
>> From: Tianrui Zhao <zhaotianrui@loongson.cn>
>>
>> Implement loongarch kvm set vcpu interrupt interface,
>> when a irq is set in vcpu, we use the KVM_INTERRUPT
>> ioctl to set intr into kvm.
>>
>> Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn>
>> Signed-off-by: xianglai li <lixianglai@loongson.cn>
>> Reviewed-by: Song Gao <gaosong@loongson.cn>
>> Message-ID: <20240105075804.1228596-9-zhaotianrui@loongson.cn>
>> [PMD: Split from bigger patch, part 2]
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>> ---
>>   target/loongarch/kvm/kvm_loongarch.h | 16 ++++++++++++++++
>>   target/loongarch/cpu.c               |  9 ++++++++-
>>   target/loongarch/kvm/kvm.c           | 15 +++++++++++++++
>>   target/loongarch/trace-events        |  1 +
>>   4 files changed, 40 insertions(+), 1 deletion(-)
>>   create mode 100644 target/loongarch/kvm/kvm_loongarch.h


>> diff --git a/target/loongarch/cpu.c b/target/loongarch/cpu.c
>> index d9f8661cfd..d3a8a2f521 100644
>> --- a/target/loongarch/cpu.c
>> +++ b/target/loongarch/cpu.c
>> @@ -12,6 +12,7 @@
>>   #include "qemu/module.h"
>>   #include "sysemu/qtest.h"
>>   #include "sysemu/tcg.h"
>> +#include "sysemu/kvm.h"
>>   #include "exec/exec-all.h"
>>   #include "cpu.h"
>>   #include "internals.h"
>> @@ -21,6 +22,10 @@
>>   #include "sysemu/reset.h"
>>   #endif
>>   #include "vec.h"
>> +#ifdef CONFIG_KVM
>> +#include "kvm/kvm_loongarch.h"
> 
> This broken  tcg 'loongarch64-softmmu' build on X86 host, :-[
> 
> ../target/loongarch/cpu.c: In function ‘loongarch_cpu_set_irq’:
> ../target/loongarch/cpu.c:122:9: error: implicit declaration of function 
> ‘kvm_loongarch_set_interrupt’ [-Werror=implicit-function-declaration]
>    122 |         kvm_loongarch_set_interrupt(cpu, irq, level);
>        |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> ../target/loongarch/cpu.c:122:9: error: nested extern declaration of 
> ‘kvm_loongarch_set_interrupt’ [-Werror=nested-externs]
> ../target/loongarch/cpu.c: In function ‘loongarch_cpu_reset_hold’:
> ../target/loongarch/cpu.c:557:9: error: implicit declaration of function 
> ‘kvm_arch_reset_vcpu’; did you mean ‘kvm_arch_init_vcpu’? 
> [-Werror=implicit-function-declaration]
>    557 |         kvm_arch_reset_vcpu(env);
>        |         ^~~~~~~~~~~~~~~~~~~
>        |         kvm_arch_init_vcpu
> ../target/loongarch/cpu.c:557:9: error: nested extern declaration of 
> ‘kvm_arch_reset_vcpu’ [-Werror=nested-externs]
> cc1: all warnings being treated as errors
> 
> I will move it out of  '#ifdef CONFIG_KVM'

Oh, right.

(Sorry I only tested on loongarch64 host).

Thanks!

Phil.
diff mbox series

Patch

diff --git a/target/loongarch/kvm/kvm_loongarch.h b/target/loongarch/kvm/kvm_loongarch.h
new file mode 100644
index 0000000000..d945b6bb82
--- /dev/null
+++ b/target/loongarch/kvm/kvm_loongarch.h
@@ -0,0 +1,16 @@ 
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/*
+ * QEMU LoongArch kvm interface
+ *
+ * Copyright (c) 2023 Loongson Technology Corporation Limited
+ */
+
+#include "cpu.h"
+
+#ifndef QEMU_KVM_LOONGARCH_H
+#define QEMU_KVM_LOONGARCH_H
+
+int  kvm_loongarch_set_interrupt(LoongArchCPU *cpu, int irq, int level);
+void kvm_arch_reset_vcpu(CPULoongArchState *env);
+
+#endif
diff --git a/target/loongarch/cpu.c b/target/loongarch/cpu.c
index d9f8661cfd..d3a8a2f521 100644
--- a/target/loongarch/cpu.c
+++ b/target/loongarch/cpu.c
@@ -12,6 +12,7 @@ 
 #include "qemu/module.h"
 #include "sysemu/qtest.h"
 #include "sysemu/tcg.h"
+#include "sysemu/kvm.h"
 #include "exec/exec-all.h"
 #include "cpu.h"
 #include "internals.h"
@@ -21,6 +22,10 @@ 
 #include "sysemu/reset.h"
 #endif
 #include "vec.h"
+#ifdef CONFIG_KVM
+#include "kvm/kvm_loongarch.h"
+#include <linux/kvm.h>
+#endif
 #ifdef CONFIG_TCG
 #include "exec/cpu_ldst.h"
 #include "tcg/tcg.h"
@@ -113,7 +118,9 @@  void loongarch_cpu_set_irq(void *opaque, int irq, int level)
         return;
     }
 
-    if (tcg_enabled()) {
+    if (kvm_enabled()) {
+        kvm_loongarch_set_interrupt(cpu, irq, level);
+    } else if (tcg_enabled()) {
         env->CSR_ESTAT = deposit64(env->CSR_ESTAT, irq, 1, level != 0);
         if (FIELD_EX64(env->CSR_ESTAT, CSR_ESTAT, IS)) {
             cpu_interrupt(cs, CPU_INTERRUPT_HARD);
diff --git a/target/loongarch/kvm/kvm.c b/target/loongarch/kvm/kvm.c
index d2dab3fef4..bd33ec2114 100644
--- a/target/loongarch/kvm/kvm.c
+++ b/target/loongarch/kvm/kvm.c
@@ -748,6 +748,21 @@  int kvm_arch_handle_exit(CPUState *cs, struct kvm_run *run)
     return ret;
 }
 
+int kvm_loongarch_set_interrupt(LoongArchCPU *cpu, int irq, int level)
+{
+    struct kvm_interrupt intr;
+    CPUState *cs = CPU(cpu);
+
+    if (level) {
+        intr.irq = irq;
+    } else {
+        intr.irq = -irq;
+    }
+
+    trace_kvm_set_intr(irq, level);
+    return kvm_vcpu_ioctl(cs, KVM_INTERRUPT, &intr);
+}
+
 void kvm_arch_accel_class_init(ObjectClass *oc)
 {
 }
diff --git a/target/loongarch/trace-events b/target/loongarch/trace-events
index 021839880e..dea11edc0f 100644
--- a/target/loongarch/trace-events
+++ b/target/loongarch/trace-events
@@ -12,3 +12,4 @@  kvm_failed_put_counter(const char *msg) "Failed to put counter into KVM: %s"
 kvm_failed_get_cpucfg(const char *msg) "Failed to get cpucfg from KVM: %s"
 kvm_failed_put_cpucfg(const char *msg) "Failed to put cpucfg into KVM: %s"
 kvm_arch_handle_exit(int num) "kvm arch handle exit, the reason number: %d"
+kvm_set_intr(int irq, int level) "kvm set interrupt, irq num: %d, level: %d"