[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 5/7] target/i386: Handle kvm_auto_* compat in x86_cp
From: |
Eduardo Habkost |
Subject: |
[Qemu-devel] [PATCH 5/7] target/i386: Handle kvm_auto_* compat in x86_cpu_expand_features() |
Date: |
Fri, 6 Oct 2017 18:52:42 -0300 |
x86_cpu_expand_features() is run after global properties are
applied, so we will be able to convert the kvm_auto_* global
variables to QOM properties controlled by
MachineClass::compat_props.
When doing that, we need to use x86_cpu_expand_feature() so we
don't override properties set explicitly by the user.
Signed-off-by: Eduardo Habkost <address@hidden>
---
target/i386/cpu.c | 41 +++++++++++++++++++++++++----------------
1 file changed, 25 insertions(+), 16 deletions(-)
diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index f8d317f1f9..83e234cef4 100644
--- a/target/i386/cpu.c
+++ b/target/i386/cpu.c
@@ -2423,22 +2423,6 @@ static void x86_cpu_load_def(X86CPU *cpu,
X86CPUDefinition *def, Error **errp)
/* Special cases not set in the X86CPUDefinition structs: */
if (kvm_enabled()) {
- if (!kvm_irqchip_in_kernel()) {
- object_property_set_bool(OBJECT(cpu), false, "x2apic",
- &error_abort);
- } else if (kvm_auto_enable_x2apic) {
- object_property_set_bool(OBJECT(cpu), true, "x2apic",
- &error_abort);
- }
- if (kvm_auto_disable_svm) {
- object_property_set_bool(OBJECT(cpu), false, "svm",
- &error_abort);
- }
- if (kvm_auto_enable_pv_eoi) {
- object_property_set_bool(OBJECT(cpu), true, "kvm-pv-eoi",
- &error_abort);
- }
-
x86_cpu_apply_props(cpu, kvm_default_props);
} else if (tcg_enabled()) {
x86_cpu_apply_props(cpu, tcg_default_props);
@@ -3511,6 +3495,31 @@ static void x86_cpu_expand_features(X86CPU *cpu, Error
**errp)
FeatureWord w;
GList *l;
Error *local_err = NULL;
+ X86CPUClass *xcc = X86_CPU_GET_CLASS(cpu);
+
+ /*
+ * Note: "base" is a static CPU model and shouldn't auto-enable KVM
+ * defaults, so we check xcc->cpu_def here.
+ */
+ if (xcc->cpu_def && kvm_enabled()) {
+ /* KVM-specific defaults that depend on compatibility globals: */
+
+ if (!kvm_irqchip_in_kernel()) {
+ x86_cpu_expand_feature(cpu, FEAT_1_ECX, CPUID_EXT_X2APIC, 0);
+ } else if (kvm_auto_enable_x2apic) {
+ x86_cpu_expand_feature(cpu, FEAT_1_ECX, CPUID_EXT_X2APIC,
+ CPUID_EXT_X2APIC);
+ }
+
+ if (kvm_auto_disable_svm) {
+ x86_cpu_expand_feature(cpu, FEAT_8000_0001_ECX, CPUID_EXT3_SVM, 0);
+ }
+
+ if (kvm_auto_enable_pv_eoi) {
+ x86_cpu_expand_feature(cpu, FEAT_KVM, (1 << KVM_FEATURE_PV_EOI),
+ (1 << KVM_FEATURE_PV_EOI));
+ }
+ }
/*TODO: Now cpu->max_features doesn't overwrite features
* set using QOM properties, and we can convert
--
2.13.6
- [Qemu-devel] [PATCH 0/7] x86: Rework KVM-defaults compat code, enable kvm_pv_unhalt by default, Eduardo Habkost, 2017/10/06
- [Qemu-devel] [PATCH 1/7] qemu-doc: Document minimum kernel version for KVM in x86_64, Eduardo Habkost, 2017/10/06
- [Qemu-devel] [PATCH 2/7] target/i386: x86_cpu_expand_feature() helper, Eduardo Habkost, 2017/10/06
- [Qemu-devel] [PATCH 3/7] target/i386: Use global variables to control KVM defaults, Eduardo Habkost, 2017/10/06
- [Qemu-devel] [PATCH 4/7] kvm: Define KVM_FEAT_* even if CONFIG_KVM is not defined, Eduardo Habkost, 2017/10/06
- [Qemu-devel] [PATCH 5/7] target/i386: Handle kvm_auto_* compat in x86_cpu_expand_features(),
Eduardo Habkost <=
- [Qemu-devel] [PATCH 6/7] pc: Use compat_props to control KVM defaults compatibility, Eduardo Habkost, 2017/10/06
- [Qemu-devel] [PATCH 7/7] target/i386: Enable kvm_pv_unhalt by default, Eduardo Habkost, 2017/10/06
- Re: [Qemu-devel] [PATCH 0/7] x86: Rework KVM-defaults compat code, enable kvm_pv_unhalt by default, Paolo Bonzini, 2017/10/09
- Re: [Qemu-devel] [PATCH 0/7] x86: Rework KVM-defaults compat code, enable kvm_pv_unhalt by default, Waiman Long, 2017/10/09
- Re: [Qemu-devel] [PATCH 0/7] x86: Rework KVM-defaults compat code, enable kvm_pv_unhalt by default, Paolo Bonzini, 2017/10/09
- Re: [Qemu-devel] [PATCH 0/7] x86: Rework KVM-defaults compat code, enable kvm_pv_unhalt by default, Eduardo Habkost, 2017/10/10
- Re: [Qemu-devel] [PATCH 0/7] x86: Rework KVM-defaults compat code, enable kvm_pv_unhalt by default, Waiman Long, 2017/10/10
- Re: [Qemu-devel] [PATCH 0/7] x86: Rework KVM-defaults compat code, enable kvm_pv_unhalt by default, Eduardo Habkost, 2017/10/10