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