|
 |
619821 |
From 6e553d9996682836a35a1020e9d992c856236342 Mon Sep 17 00:00:00 2001
|
|
|
619821 |
From: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
619821 |
Date: Thu, 23 Feb 2017 14:29:40 +0100
|
|
|
619821 |
Subject: [PATCH 10/17] target-i386: Make TCG feature filtering more readable
|
|
|
619821 |
MIME-Version: 1.0
|
|
|
619821 |
Content-Type: text/plain; charset=UTF-8
|
|
|
619821 |
Content-Transfer-Encoding: 8bit
|
|
|
619821 |
|
|
|
619821 |
RH-Author: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
619821 |
Message-id: <20170223142945.17790-10-ehabkost@redhat.com>
|
|
|
619821 |
Patchwork-id: 74041
|
|
|
619821 |
O-Subject: [RHEL-7.4 qemu-kvm PATCH v2 09/14] target-i386: Make TCG feature filtering more readable
|
|
|
619821 |
Bugzilla: 1382122
|
|
|
619821 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
619821 |
RH-Acked-by: Igor Mammedov <imammedo@redhat.com>
|
|
|
619821 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
619821 |
|
|
|
619821 |
Instead of an #ifdef in the middle of the code, just set
|
|
|
619821 |
TCG_EXT2_FEATURES to a different value depending on TARGET_X86_64.
|
|
|
619821 |
|
|
|
619821 |
Reviewed-by: Richard Henderson <rth@twiddle.net>
|
|
|
619821 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
619821 |
Signed-off-by: Andreas Färber <afaerber@suse.de>
|
|
|
619821 |
(cherry picked from commit a42d9938a162c3e3c9e441d1927dca5bd59167d9)
|
|
|
619821 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
619821 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
619821 |
---
|
|
|
619821 |
target-i386/cpu.c | 16 ++++++++++------
|
|
|
619821 |
1 file changed, 10 insertions(+), 6 deletions(-)
|
|
|
619821 |
|
|
|
619821 |
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
|
|
|
619821 |
index bb8abf7..e32d4d7 100644
|
|
|
619821 |
--- a/target-i386/cpu.c
|
|
|
619821 |
+++ b/target-i386/cpu.c
|
|
|
619821 |
@@ -486,9 +486,17 @@ typedef struct x86_def_t {
|
|
|
619821 |
CPUID_EXT_X2APIC, CPUID_EXT_TSC_DEADLINE_TIMER, CPUID_EXT_XSAVE,
|
|
|
619821 |
CPUID_EXT_OSXSAVE, CPUID_EXT_AVX, CPUID_EXT_F16C,
|
|
|
619821 |
CPUID_EXT_RDRAND */
|
|
|
619821 |
+
|
|
|
619821 |
+#ifdef TARGET_X86_64
|
|
|
619821 |
+#define TCG_EXT2_X86_64_FEATURES (CPUID_EXT2_SYSCALL | CPUID_EXT2_LM)
|
|
|
619821 |
+#else
|
|
|
619821 |
+#define TCG_EXT2_X86_64_FEATURES 0
|
|
|
619821 |
+#endif
|
|
|
619821 |
+
|
|
|
619821 |
#define TCG_EXT2_FEATURES ((TCG_FEATURES & CPUID_EXT2_AMD_ALIASES) | \
|
|
|
619821 |
CPUID_EXT2_NX | CPUID_EXT2_MMXEXT | CPUID_EXT2_RDTSCP | \
|
|
|
619821 |
- CPUID_EXT2_3DNOW | CPUID_EXT2_3DNOWEXT)
|
|
|
619821 |
+ CPUID_EXT2_3DNOW | CPUID_EXT2_3DNOWEXT | \
|
|
|
619821 |
+ TCG_EXT2_X86_64_FEATURES)
|
|
|
619821 |
/* missing:
|
|
|
619821 |
CPUID_EXT2_PDPE1GB */
|
|
|
619821 |
#define TCG_EXT3_FEATURES (CPUID_EXT3_LAHF_LM | CPUID_EXT3_SVM | \
|
|
|
619821 |
@@ -2632,11 +2640,7 @@ static void x86_cpu_realizefn(DeviceState *dev, Error **errp)
|
|
|
619821 |
if (!kvm_enabled()) {
|
|
|
619821 |
env->features[FEAT_1_EDX] &= TCG_FEATURES;
|
|
|
619821 |
env->features[FEAT_1_ECX] &= TCG_EXT_FEATURES;
|
|
|
619821 |
- env->features[FEAT_8000_0001_EDX] &= (TCG_EXT2_FEATURES
|
|
|
619821 |
-#ifdef TARGET_X86_64
|
|
|
619821 |
- | CPUID_EXT2_SYSCALL | CPUID_EXT2_LM
|
|
|
619821 |
-#endif
|
|
|
619821 |
- );
|
|
|
619821 |
+ env->features[FEAT_8000_0001_EDX] &= TCG_EXT2_FEATURES;
|
|
|
619821 |
env->features[FEAT_8000_0001_ECX] &= TCG_EXT3_FEATURES;
|
|
|
619821 |
env->features[FEAT_SVM] &= TCG_SVM_FEATURES;
|
|
|
619821 |
env->features[FEAT_XSAVE] = 0;
|
|
|
619821 |
--
|
|
|
619821 |
1.8.3.1
|
|
|
619821 |
|