Provide a kvm specific vcpu property to override the default
(as of kernel v6.13 that would be PSCI v1.3) PSCI version emulated
by kvm. Current valid values are: 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
Signed-off-by: Sebastian Ott <sebott@redhat.com>
---
docs/system/arm/cpu-features.rst | 5 +++
target/arm/cpu.h | 6 +++
target/arm/kvm.c | 70 +++++++++++++++++++++++++++++++-
3 files changed, 80 insertions(+), 1 deletion(-)
diff --git a/docs/system/arm/cpu-features.rst b/docs/system/arm/cpu-features.rst
index 37d5dfd15b..1d32ce0fee 100644
--- a/docs/system/arm/cpu-features.rst
+++ b/docs/system/arm/cpu-features.rst
@@ -204,6 +204,11 @@ the list of KVM VCPU features and their descriptions.
the guest scheduler behavior and/or be exposed to the guest
userspace.
+``kvm-psci-version``
+ Override the default (as of kernel v6.13 that would be PSCI v1.3)
+ PSCI version emulated by the kernel. Current valid values are:
+ 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
+
TCG VCPU Features
=================
diff --git a/target/arm/cpu.h b/target/arm/cpu.h
index c15d79a106..44292aab32 100644
--- a/target/arm/cpu.h
+++ b/target/arm/cpu.h
@@ -974,6 +974,12 @@ struct ArchCPU {
*/
uint32_t psci_version;
+ /*
+ * Intermediate value used during property parsing.
+ * Once finalized, the value should be read from psci_version.
+ */
+ uint32_t prop_psci_version;
+
/* Current power state, access guarded by BQL */
ARMPSCIState power_state;
diff --git a/target/arm/kvm.c b/target/arm/kvm.c
index 6672344855..bc6073f395 100644
--- a/target/arm/kvm.c
+++ b/target/arm/kvm.c
@@ -483,6 +483,59 @@ static void kvm_steal_time_set(Object *obj, bool value, Error **errp)
ARM_CPU(obj)->kvm_steal_time = value ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;
}
+static char *kvm_get_psci_version(Object *obj, Error **errp)
+{
+ ARMCPU *cpu = ARM_CPU(obj);
+ const char *val;
+
+ switch (cpu->prop_psci_version) {
+ case QEMU_PSCI_VERSION_0_1:
+ val = "0.1";
+ break;
+ case QEMU_PSCI_VERSION_0_2:
+ val = "0.2";
+ break;
+ case QEMU_PSCI_VERSION_1_0:
+ val = "1.0";
+ break;
+ case QEMU_PSCI_VERSION_1_1:
+ val = "1.1";
+ break;
+ case QEMU_PSCI_VERSION_1_2:
+ val = "1.2";
+ break;
+ case QEMU_PSCI_VERSION_1_3:
+ val = "1.3";
+ break;
+ default:
+ val = "0.2";
+ break;
+ }
+ return g_strdup(val);
+}
+
+static void kvm_set_psci_version(Object *obj, const char *value, Error **errp)
+{
+ ARMCPU *cpu = ARM_CPU(obj);
+
+ if (!strcmp(value, "0.1")) {
+ cpu->prop_psci_version = QEMU_PSCI_VERSION_0_1;
+ } else if (!strcmp(value, "0.2")) {
+ cpu->prop_psci_version = QEMU_PSCI_VERSION_0_2;
+ } else if (!strcmp(value, "1.0")) {
+ cpu->prop_psci_version = QEMU_PSCI_VERSION_1_0;
+ } else if (!strcmp(value, "1.1")) {
+ cpu->prop_psci_version = QEMU_PSCI_VERSION_1_1;
+ } else if (!strcmp(value, "1.2")) {
+ cpu->prop_psci_version = QEMU_PSCI_VERSION_1_2;
+ } else if (!strcmp(value, "1.3")) {
+ cpu->prop_psci_version = QEMU_PSCI_VERSION_1_3;
+ } else {
+ error_setg(errp, "Invalid PSCI-version value");
+ error_append_hint(errp, "Valid values are 0.1, 0.2, 1.0, 1.1, 1.2, 1.3\n");
+ }
+}
+
/* KVM VCPU properties should be prefixed with "kvm-". */
void kvm_arm_add_vcpu_properties(ARMCPU *cpu)
{
@@ -504,6 +557,12 @@ void kvm_arm_add_vcpu_properties(ARMCPU *cpu)
kvm_steal_time_set);
object_property_set_description(obj, "kvm-steal-time",
"Set off to disable KVM steal time.");
+
+ object_property_add_str(obj, "kvm-psci-version", kvm_get_psci_version,
+ kvm_set_psci_version);
+ object_property_set_description(obj, "kvm-psci-version",
+ "Set PSCI version. "
+ "Valid values are 0.1, 0.2, 1.0, 1.1, 1.2, 1.3");
}
bool kvm_arm_pmu_supported(void)
@@ -1883,7 +1942,8 @@ int kvm_arch_init_vcpu(CPUState *cs)
if (cs->start_powered_off) {
cpu->kvm_init_features[0] |= 1 << KVM_ARM_VCPU_POWER_OFF;
}
- if (kvm_check_extension(cs->kvm_state, KVM_CAP_ARM_PSCI_0_2)) {
+ if (cpu->prop_psci_version != QEMU_PSCI_VERSION_0_1 &&
+ kvm_check_extension(cs->kvm_state, KVM_CAP_ARM_PSCI_0_2)) {
cpu->psci_version = QEMU_PSCI_VERSION_0_2;
cpu->kvm_init_features[0] |= 1 << KVM_ARM_VCPU_PSCI_0_2;
}
@@ -1922,6 +1982,14 @@ int kvm_arch_init_vcpu(CPUState *cs)
}
}
+ if (cpu->prop_psci_version) {
+ psciver = cpu->prop_psci_version;
+ ret = kvm_set_one_reg(cs, KVM_REG_ARM_PSCI_VERSION, &psciver);
+ if (ret) {
+ error_report("PSCI version %lx is not supported by KVM", psciver);
+ return ret;
+ }
+ }
/*
* KVM reports the exact PSCI version it is implementing via a
* special sysreg. If it is present, use its contents to determine
--
2.42.0On Thu, 11 Sept 2025 at 15:49, Sebastian Ott <sebott@redhat.com> wrote:
>
> Provide a kvm specific vcpu property to override the default
> (as of kernel v6.13 that would be PSCI v1.3) PSCI version emulated
> by kvm. Current valid values are: 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
>
> Signed-off-by: Sebastian Ott <sebott@redhat.com>
> ---
> docs/system/arm/cpu-features.rst | 5 +++
> target/arm/cpu.h | 6 +++
> target/arm/kvm.c | 70 +++++++++++++++++++++++++++++++-
> 3 files changed, 80 insertions(+), 1 deletion(-)
>
> diff --git a/docs/system/arm/cpu-features.rst b/docs/system/arm/cpu-features.rst
> index 37d5dfd15b..1d32ce0fee 100644
> --- a/docs/system/arm/cpu-features.rst
> +++ b/docs/system/arm/cpu-features.rst
> @@ -204,6 +204,11 @@ the list of KVM VCPU features and their descriptions.
> the guest scheduler behavior and/or be exposed to the guest
> userspace.
>
> +``kvm-psci-version``
> + Override the default (as of kernel v6.13 that would be PSCI v1.3)
> + PSCI version emulated by the kernel. Current valid values are:
> + 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
> +
> TCG VCPU Features
> =================
>
> diff --git a/target/arm/cpu.h b/target/arm/cpu.h
> index c15d79a106..44292aab32 100644
> --- a/target/arm/cpu.h
> +++ b/target/arm/cpu.h
> @@ -974,6 +974,12 @@ struct ArchCPU {
> */
> uint32_t psci_version;
>
> + /*
> + * Intermediate value used during property parsing.
> + * Once finalized, the value should be read from psci_version.
> + */
> + uint32_t prop_psci_version;
> +
> /* Current power state, access guarded by BQL */
> ARMPSCIState power_state;
>
> diff --git a/target/arm/kvm.c b/target/arm/kvm.c
> index 6672344855..bc6073f395 100644
> --- a/target/arm/kvm.c
> +++ b/target/arm/kvm.c
> @@ -483,6 +483,59 @@ static void kvm_steal_time_set(Object *obj, bool value, Error **errp)
> ARM_CPU(obj)->kvm_steal_time = value ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;
> }
>
> +static char *kvm_get_psci_version(Object *obj, Error **errp)
> +{
> + ARMCPU *cpu = ARM_CPU(obj);
> + const char *val;
> +
> + switch (cpu->prop_psci_version) {
> + case QEMU_PSCI_VERSION_0_1:
> + val = "0.1";
> + break;
> + case QEMU_PSCI_VERSION_0_2:
> + val = "0.2";
> + break;
> + case QEMU_PSCI_VERSION_1_0:
> + val = "1.0";
> + break;
> + case QEMU_PSCI_VERSION_1_1:
> + val = "1.1";
> + break;
> + case QEMU_PSCI_VERSION_1_2:
> + val = "1.2";
> + break;
> + case QEMU_PSCI_VERSION_1_3:
> + val = "1.3";
> + break;
> + default:
> + val = "0.2";
> + break;
> + }
> + return g_strdup(val);
> +}
> +
> +static void kvm_set_psci_version(Object *obj, const char *value, Error **errp)
> +{
> + ARMCPU *cpu = ARM_CPU(obj);
> +
> + if (!strcmp(value, "0.1")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_0_1;
> + } else if (!strcmp(value, "0.2")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_0_2;
> + } else if (!strcmp(value, "1.0")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_0;
> + } else if (!strcmp(value, "1.1")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_1;
> + } else if (!strcmp(value, "1.2")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_2;
> + } else if (!strcmp(value, "1.3")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_3;
We already have six values here and it's not implausible
we might end up with more in future; maybe we should make the
mapping between string and constant data-driven rather
than having code written out longhand in the get and set
functions?
> + } else {
> + error_setg(errp, "Invalid PSCI-version value");
> + error_append_hint(errp, "Valid values are 0.1, 0.2, 1.0, 1.1, 1.2, 1.3\n");
> + }
> +}
thanks
-- PMM
On Mon, 27 Oct 2025, Peter Maydell wrote:
> On Thu, 11 Sept 2025 at 15:49, Sebastian Ott <sebott@redhat.com> wrote:
>>
>> Provide a kvm specific vcpu property to override the default
>> (as of kernel v6.13 that would be PSCI v1.3) PSCI version emulated
>> by kvm. Current valid values are: 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
>>
>> Signed-off-by: Sebastian Ott <sebott@redhat.com>
>> ---
>> docs/system/arm/cpu-features.rst | 5 +++
>> target/arm/cpu.h | 6 +++
>> target/arm/kvm.c | 70 +++++++++++++++++++++++++++++++-
>> 3 files changed, 80 insertions(+), 1 deletion(-)
>>
>> diff --git a/docs/system/arm/cpu-features.rst b/docs/system/arm/cpu-features.rst
>> index 37d5dfd15b..1d32ce0fee 100644
>> --- a/docs/system/arm/cpu-features.rst
>> +++ b/docs/system/arm/cpu-features.rst
>> @@ -204,6 +204,11 @@ the list of KVM VCPU features and their descriptions.
>> the guest scheduler behavior and/or be exposed to the guest
>> userspace.
>>
>> +``kvm-psci-version``
>> + Override the default (as of kernel v6.13 that would be PSCI v1.3)
>> + PSCI version emulated by the kernel. Current valid values are:
>> + 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
>> +
>> TCG VCPU Features
>> =================
>>
>> diff --git a/target/arm/cpu.h b/target/arm/cpu.h
>> index c15d79a106..44292aab32 100644
>> --- a/target/arm/cpu.h
>> +++ b/target/arm/cpu.h
>> @@ -974,6 +974,12 @@ struct ArchCPU {
>> */
>> uint32_t psci_version;
>>
>> + /*
>> + * Intermediate value used during property parsing.
>> + * Once finalized, the value should be read from psci_version.
>> + */
>> + uint32_t prop_psci_version;
>> +
>> /* Current power state, access guarded by BQL */
>> ARMPSCIState power_state;
>>
>> diff --git a/target/arm/kvm.c b/target/arm/kvm.c
>> index 6672344855..bc6073f395 100644
>> --- a/target/arm/kvm.c
>> +++ b/target/arm/kvm.c
>> @@ -483,6 +483,59 @@ static void kvm_steal_time_set(Object *obj, bool value, Error **errp)
>> ARM_CPU(obj)->kvm_steal_time = value ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;
>> }
>>
>> +static char *kvm_get_psci_version(Object *obj, Error **errp)
>> +{
>> + ARMCPU *cpu = ARM_CPU(obj);
>> + const char *val;
>> +
>> + switch (cpu->prop_psci_version) {
>> + case QEMU_PSCI_VERSION_0_1:
>> + val = "0.1";
>> + break;
>> + case QEMU_PSCI_VERSION_0_2:
>> + val = "0.2";
>> + break;
>> + case QEMU_PSCI_VERSION_1_0:
>> + val = "1.0";
>> + break;
>> + case QEMU_PSCI_VERSION_1_1:
>> + val = "1.1";
>> + break;
>> + case QEMU_PSCI_VERSION_1_2:
>> + val = "1.2";
>> + break;
>> + case QEMU_PSCI_VERSION_1_3:
>> + val = "1.3";
>> + break;
>> + default:
>> + val = "0.2";
>> + break;
>> + }
>> + return g_strdup(val);
>> +}
>> +
>> +static void kvm_set_psci_version(Object *obj, const char *value, Error **errp)
>> +{
>> + ARMCPU *cpu = ARM_CPU(obj);
>> +
>> + if (!strcmp(value, "0.1")) {
>> + cpu->prop_psci_version = QEMU_PSCI_VERSION_0_1;
>> + } else if (!strcmp(value, "0.2")) {
>> + cpu->prop_psci_version = QEMU_PSCI_VERSION_0_2;
>> + } else if (!strcmp(value, "1.0")) {
>> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_0;
>> + } else if (!strcmp(value, "1.1")) {
>> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_1;
>> + } else if (!strcmp(value, "1.2")) {
>> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_2;
>> + } else if (!strcmp(value, "1.3")) {
>> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_3;
>
> We already have six values here and it's not implausible
> we might end up with more in future; maybe we should make the
> mapping between string and constant data-driven rather
> than having code written out longhand in the get and set
> functions?
Yes, sure. I'll send out a V2.
Thanks!
Sebastian
Hi Sebastian,
On 9/11/25 4:49 PM, Sebastian Ott wrote:
> Provide a kvm specific vcpu property to override the default
> (as of kernel v6.13 that would be PSCI v1.3) PSCI version emulated
> by kvm. Current valid values are: 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
>
> Signed-off-by: Sebastian Ott <sebott@redhat.com>
> ---
> docs/system/arm/cpu-features.rst | 5 +++
> target/arm/cpu.h | 6 +++
> target/arm/kvm.c | 70 +++++++++++++++++++++++++++++++-
> 3 files changed, 80 insertions(+), 1 deletion(-)
>
> diff --git a/docs/system/arm/cpu-features.rst b/docs/system/arm/cpu-features.rst
> index 37d5dfd15b..1d32ce0fee 100644
> --- a/docs/system/arm/cpu-features.rst
> +++ b/docs/system/arm/cpu-features.rst
> @@ -204,6 +204,11 @@ the list of KVM VCPU features and their descriptions.
> the guest scheduler behavior and/or be exposed to the guest
> userspace.
>
> +``kvm-psci-version``
> + Override the default (as of kernel v6.13 that would be PSCI v1.3)
> + PSCI version emulated by the kernel. Current valid values are:
> + 0.1, 0.2, 1.0, 1.1, 1.2, and 1.3
> +
> TCG VCPU Features
> =================
>
> diff --git a/target/arm/cpu.h b/target/arm/cpu.h
> index c15d79a106..44292aab32 100644
> --- a/target/arm/cpu.h
> +++ b/target/arm/cpu.h
> @@ -974,6 +974,12 @@ struct ArchCPU {
> */
> uint32_t psci_version;
>
> + /*
> + * Intermediate value used during property parsing.
> + * Once finalized, the value should be read from psci_version.
> + */
> + uint32_t prop_psci_version;
> +
> /* Current power state, access guarded by BQL */
> ARMPSCIState power_state;
>
> diff --git a/target/arm/kvm.c b/target/arm/kvm.c
> index 6672344855..bc6073f395 100644
> --- a/target/arm/kvm.c
> +++ b/target/arm/kvm.c
> @@ -483,6 +483,59 @@ static void kvm_steal_time_set(Object *obj, bool value, Error **errp)
> ARM_CPU(obj)->kvm_steal_time = value ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;
> }
>
> +static char *kvm_get_psci_version(Object *obj, Error **errp)
> +{
> + ARMCPU *cpu = ARM_CPU(obj);
> + const char *val;
> +
> + switch (cpu->prop_psci_version) {
> + case QEMU_PSCI_VERSION_0_1:
> + val = "0.1";
> + break;
> + case QEMU_PSCI_VERSION_0_2:
> + val = "0.2";
> + break;
> + case QEMU_PSCI_VERSION_1_0:
> + val = "1.0";
> + break;
> + case QEMU_PSCI_VERSION_1_1:
> + val = "1.1";
> + break;
> + case QEMU_PSCI_VERSION_1_2:
> + val = "1.2";
> + break;
> + case QEMU_PSCI_VERSION_1_3:
> + val = "1.3";
> + break;
> + default:
> + val = "0.2";
can you explain why you return 0.2 by default? Shouldn't we report the
default value exposed by KVM?
> + break;
> + }
> + return g_strdup(val);
> +}
> +
> +static void kvm_set_psci_version(Object *obj, const char *value, Error **errp)
> +{
> + ARMCPU *cpu = ARM_CPU(obj);
> +
> + if (!strcmp(value, "0.1")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_0_1;
> + } else if (!strcmp(value, "0.2")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_0_2;
> + } else if (!strcmp(value, "1.0")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_0;
> + } else if (!strcmp(value, "1.1")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_1;
> + } else if (!strcmp(value, "1.2")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_2;
> + } else if (!strcmp(value, "1.3")) {
> + cpu->prop_psci_version = QEMU_PSCI_VERSION_1_3;
> + } else {
> + error_setg(errp, "Invalid PSCI-version value");
> + error_append_hint(errp, "Valid values are 0.1, 0.2, 1.0, 1.1, 1.2, 1.3\n");
> + }
> +}
> +
> /* KVM VCPU properties should be prefixed with "kvm-". */
> void kvm_arm_add_vcpu_properties(ARMCPU *cpu)
> {
> @@ -504,6 +557,12 @@ void kvm_arm_add_vcpu_properties(ARMCPU *cpu)
> kvm_steal_time_set);
> object_property_set_description(obj, "kvm-steal-time",
> "Set off to disable KVM steal time.");
> +
> + object_property_add_str(obj, "kvm-psci-version", kvm_get_psci_version,
> + kvm_set_psci_version);
> + object_property_set_description(obj, "kvm-psci-version",
> + "Set PSCI version. "
> + "Valid values are 0.1, 0.2, 1.0, 1.1, 1.2, 1.3");
> }
>
> bool kvm_arm_pmu_supported(void)
> @@ -1883,7 +1942,8 @@ int kvm_arch_init_vcpu(CPUState *cs)
> if (cs->start_powered_off) {
> cpu->kvm_init_features[0] |= 1 << KVM_ARM_VCPU_POWER_OFF;
> }
> - if (kvm_check_extension(cs->kvm_state, KVM_CAP_ARM_PSCI_0_2)) {
> + if (cpu->prop_psci_version != QEMU_PSCI_VERSION_0_1 &&
> + kvm_check_extension(cs->kvm_state, KVM_CAP_ARM_PSCI_0_2)) {
I don't get why this change is needed. Please can you explain?
> cpu->psci_version = QEMU_PSCI_VERSION_0_2;
> cpu->kvm_init_features[0] |= 1 << KVM_ARM_VCPU_PSCI_0_2;
> }
> @@ -1922,6 +1982,14 @@ int kvm_arch_init_vcpu(CPUState *cs)
> }
> }
>
> + if (cpu->prop_psci_version) {
> + psciver = cpu->prop_psci_version;
> + ret = kvm_set_one_reg(cs, KVM_REG_ARM_PSCI_VERSION, &psciver);
> + if (ret) {
> + error_report("PSCI version %lx is not supported by KVM", psciver);
don't you need a PRIx64 here?
> + return ret;
> + }
> + }
> /*
> * KVM reports the exact PSCI version it is implementing via a
> * special sysreg. If it is present, use its contents to determine
Thanks
Eric
© 2016 - 2026 Red Hat, Inc.