|
 |
26ba25 |
From 0a6e593487844fd00920c0caab64aa98eac368a1 Mon Sep 17 00:00:00 2001
|
|
|
26ba25 |
From: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
26ba25 |
Date: Thu, 26 Jul 2018 17:19:04 +0100
|
|
|
26ba25 |
Subject: [PATCH 14/14] i386: Allow TOPOEXT to be enabled on older kernels
|
|
|
26ba25 |
|
|
|
26ba25 |
RH-Author: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
26ba25 |
Message-id: <20180726171904.27418-12-ehabkost@redhat.com>
|
|
|
26ba25 |
Patchwork-id: 81534
|
|
|
26ba25 |
O-Subject: [qemu-kvm RHEL8/virt212 PATCH v2 11/11] i386: Allow TOPOEXT to be enabled on older kernels
|
|
|
26ba25 |
Bugzilla: 1597739
|
|
|
26ba25 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
26ba25 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
26ba25 |
RH-Acked-by: Igor Mammedov <imammedo@redhat.com>
|
|
|
26ba25 |
|
|
|
26ba25 |
From: Babu Moger <babu.moger@amd.com>
|
|
|
26ba25 |
|
|
|
26ba25 |
Enabling TOPOEXT feature might cause compatibility issues if
|
|
|
26ba25 |
older kernels does not set this feature. Lets set this feature
|
|
|
26ba25 |
unconditionally.
|
|
|
26ba25 |
|
|
|
26ba25 |
Signed-off-by: Babu Moger <babu.moger@amd.com>
|
|
|
26ba25 |
Message-Id: <1528939107-17193-2-git-send-email-babu.moger@amd.com>
|
|
|
26ba25 |
[ehabkost: rewrite comment and commit message]
|
|
|
26ba25 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
26ba25 |
(cherry picked from commit f98bbd8304112187cafc3e636c31b2a3865d2717)
|
|
|
26ba25 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
26ba25 |
|
|
|
26ba25 |
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
|
|
|
26ba25 |
---
|
|
|
26ba25 |
target/i386/kvm.c | 7 +++++++
|
|
|
26ba25 |
1 file changed, 7 insertions(+)
|
|
|
26ba25 |
|
|
|
26ba25 |
diff --git a/target/i386/kvm.c b/target/i386/kvm.c
|
|
|
26ba25 |
index 6e66f9c..bb5e485 100644
|
|
|
26ba25 |
--- a/target/i386/kvm.c
|
|
|
26ba25 |
+++ b/target/i386/kvm.c
|
|
|
26ba25 |
@@ -371,6 +371,13 @@ uint32_t kvm_arch_get_supported_cpuid(KVMState *s, uint32_t function,
|
|
|
26ba25 |
if (host_tsx_blacklisted()) {
|
|
|
26ba25 |
ret &= ~(CPUID_7_0_EBX_RTM | CPUID_7_0_EBX_HLE);
|
|
|
26ba25 |
}
|
|
|
26ba25 |
+ } else if (function == 0x80000001 && reg == R_ECX) {
|
|
|
26ba25 |
+ /*
|
|
|
26ba25 |
+ * It's safe to enable TOPOEXT even if it's not returned by
|
|
|
26ba25 |
+ * GET_SUPPORTED_CPUID. Unconditionally enabling TOPOEXT here allows
|
|
|
26ba25 |
+ * us to keep CPU models including TOPOEXT runnable on older kernels.
|
|
|
26ba25 |
+ */
|
|
|
26ba25 |
+ ret |= CPUID_EXT3_TOPOEXT;
|
|
|
26ba25 |
} else if (function == 0x80000001 && reg == R_EDX) {
|
|
|
26ba25 |
/* On Intel, kvm returns cpuid according to the Intel spec,
|
|
|
26ba25 |
* so add missing bits according to the AMD spec:
|
|
|
26ba25 |
--
|
|
|
26ba25 |
1.8.3.1
|
|
|
26ba25 |
|