|
 |
76daa3 |
From b4307873bd68ead49bdccdde06c079dcc6a7e3ca Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
76daa3 |
Date: Tue, 6 Jun 2017 13:03:15 +0200
|
|
|
76daa3 |
Subject: [PATCH 14/17] pc: Use "min[x]level" on compat_props on RHEL
|
|
|
76daa3 |
machine-types
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
76daa3 |
Message-id: <20170606130315.14990-1-ehabkost@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75501
|
|
|
76daa3 |
O-Subject: [RHV-7.4 qemu-kvm-rhev PATCH] pc: Use "min[x]level" on compat_props on RHEL machine-types
|
|
|
76daa3 |
Bugzilla: 1454641
|
|
|
76daa3 |
RH-Acked-by: Marcel Apfelbaum <marcel@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Igor Mammedov <imammedo@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
Since the automatic cpuid-level code was introduced in commit
|
|
|
76daa3 |
c39c0edf9bb3b968ba95484465a50c7b19f4aa3a ("target-i386: Automatically
|
|
|
76daa3 |
set level/xlevel/xlevel2 when needed"), the CPU model tables just define
|
|
|
76daa3 |
the default CPUID level code (set using "min-level"). Setting
|
|
|
76daa3 |
"[x]level" forces CPUID level to a specific value and disable the
|
|
|
76daa3 |
automatic-level logic.
|
|
|
76daa3 |
|
|
|
76daa3 |
However, the PC compat code was not updated and the existing "[x]level"
|
|
|
76daa3 |
compat properties broke compatibility for people using features that
|
|
|
76daa3 |
triggered the auto-level code. To keep previous behavior, we should set
|
|
|
76daa3 |
"min-[x]level" instead of "[x]level" on compat_props.
|
|
|
76daa3 |
|
|
|
76daa3 |
This was not a problem for most cases, because old machine-types don't
|
|
|
76daa3 |
have full-cpuid-auto-level enabled. The only common use case it broke
|
|
|
76daa3 |
was the CPUID[7] auto-level code, that was already enabled since the
|
|
|
76daa3 |
first CPUID[7] feature was introduced (in QEMU 1.4.0).
|
|
|
76daa3 |
|
|
|
76daa3 |
This causes the regression reported at:
|
|
|
76daa3 |
https://bugzilla.redhat.com/show_bug.cgi?id=1454641
|
|
|
76daa3 |
|
|
|
76daa3 |
Change the RHEL compat code to use "min-[x]level" instead of "[x]level"
|
|
|
76daa3 |
on compat_props.
|
|
|
76daa3 |
|
|
|
76daa3 |
Upstream notes:
|
|
|
76daa3 |
|
|
|
76daa3 |
There's an upstream patch in a pending pull request already (1f43571604
|
|
|
76daa3 |
"pc: Use "min-[x]level" on compat_props"), but we don't need it to be
|
|
|
76daa3 |
backported because it only touches the upstream machine-types.
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
hw/i386/pc_piix.c | 10 +++++-----
|
|
|
76daa3 |
include/hw/i386/pc.h | 36 ++++++++++++++++++------------------
|
|
|
76daa3 |
2 files changed, 23 insertions(+), 23 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
|
|
|
76daa3 |
index 038dda5..a9d4769 100644
|
|
|
76daa3 |
--- a/hw/i386/pc_piix.c
|
|
|
76daa3 |
+++ b/hw/i386/pc_piix.c
|
|
|
76daa3 |
@@ -1354,11 +1354,11 @@ DEFINE_PC_MACHINE(rhel700, "pc-i440fx-rhel7.0.0", pc_init_rhel700,
|
|
|
76daa3 |
.value = "off",\
|
|
|
76daa3 |
},{\
|
|
|
76daa3 |
.driver = "qemu32-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0),\
|
|
|
76daa3 |
},{\
|
|
|
76daa3 |
.driver = "486-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(0),\
|
|
|
76daa3 |
},{\
|
|
|
76daa3 |
.driver = "qemu32-" TYPE_X86_CPU,\
|
|
|
76daa3 |
@@ -1684,15 +1684,15 @@ DEFINE_PC_MACHINE(rhel640, "rhel6.4.0", pc_init_rhel640,
|
|
|
76daa3 |
#define PC_RHEL6_3_COMPAT \
|
|
|
76daa3 |
{\
|
|
|
76daa3 |
.driver = "Conroe-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(2),\
|
|
|
76daa3 |
},{\
|
|
|
76daa3 |
.driver = "Penryn-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(2),\
|
|
|
76daa3 |
},{\
|
|
|
76daa3 |
.driver = "Nehalem-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(2),\
|
|
|
76daa3 |
},{\
|
|
|
76daa3 |
.driver = "e1000",\
|
|
|
76daa3 |
diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
|
|
|
76daa3 |
index e05d838..3944e40 100644
|
|
|
76daa3 |
--- a/include/hw/i386/pc.h
|
|
|
76daa3 |
+++ b/include/hw/i386/pc.h
|
|
|
76daa3 |
@@ -1237,75 +1237,75 @@ extern void igd_passthrough_isa_bridge_create(PCIBus *bus, uint16_t gpu_dev_id);
|
|
|
76daa3 |
},\
|
|
|
76daa3 |
{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "qemu64" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(4),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "kvm64" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(5),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "pentium3" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(2),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "n270" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(5),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Conroe" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(4),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Penryn" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(4),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Nehalem" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "level",\
|
|
|
76daa3 |
+ .property = "min-level",\
|
|
|
76daa3 |
.value = stringify(4),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "n270" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Penryn" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Conroe" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Nehalem" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Westmere" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "SandyBridge" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "IvyBridge" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Haswell" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Haswell-noTSX" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Broadwell" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},{ /* PC_RHEL7_1_COMPAT */ \
|
|
|
76daa3 |
.driver = "Broadwell-noTSX" "-" TYPE_X86_CPU,\
|
|
|
76daa3 |
- .property = "xlevel",\
|
|
|
76daa3 |
+ .property = "min-xlevel",\
|
|
|
76daa3 |
.value = stringify(0x8000000a),\
|
|
|
76daa3 |
},
|
|
|
76daa3 |
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|