b971b8
From 7d0790fb5061cb956390ce5f7a2b92ac9702ada6 Mon Sep 17 00:00:00 2001
b971b8
Message-Id: <7d0790fb5061cb956390ce5f7a2b92ac9702ada6@dist-git>
b971b8
From: Jiri Denemark <jdenemar@redhat.com>
b971b8
Date: Tue, 26 May 2020 10:59:39 +0200
b971b8
Subject: [PATCH] cpu_map: Add pschange-mc-no bit in IA32_ARCH_CAPABILITIES MSR
b971b8
MIME-Version: 1.0
b971b8
Content-Type: text/plain; charset=UTF-8
b971b8
Content-Transfer-Encoding: 8bit
b971b8
b971b8
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
b971b8
Reviewed-by: Pavel Hrdina <phrdina@redhat.com>
b971b8
(cherry picked from commit 12eb0c9496e802bad9e5ec71cde888b8fdb1b0b4)
b971b8
b971b8
https://bugzilla.redhat.com/show_bug.cgi?id=1781878
b971b8
b971b8
Conflicts:
b971b8
	tests/cputestdata/x86_64-cpuid-Ryzen-9-3900X-12-Core-enabled.xml
b971b8
	tests/cputestdata/x86_64-cpuid-Ryzen-9-3900X-12-Core-guest.xml
b971b8
	tests/cputestdata/x86_64-cpuid-Ryzen-9-3900X-12-Core-host.xml
b971b8
	tests/cputestdata/x86_64-cpuid-Ryzen-9-3900X-12-Core-json.xml
b971b8
	tests/domaincapsdata/qemu_5.1.0-q35.x86_64.xml
b971b8
	tests/domaincapsdata/qemu_5.1.0.x86_64.xml
b971b8
            - test data files do not exist downstream
b971b8
b971b8
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
b971b8
Message-Id: <d106df21f8423a9acff9eb5bf75ab25611c1de1f.1590483392.git.jdenemar@redhat.com>
b971b8
Reviewed-by: Ján Tomko <jtomko@redhat.com>
b971b8
---
b971b8
 src/cpu_map/x86_features.xml                                  | 3 +++
b971b8
 tests/cputestdata/x86_64-cpuid-Cooperlake-guest.xml           | 1 +
b971b8
 tests/cputestdata/x86_64-cpuid-Cooperlake-host.xml            | 1 +
b971b8
 tests/cputestdata/x86_64-cpuid-Cooperlake-json.xml            | 1 +
b971b8
 tests/cputestdata/x86_64-cpuid-Core-i7-8550U-enabled.xml      | 2 +-
b971b8
 tests/cputestdata/x86_64-cpuid-Core-i7-8550U-guest.xml        | 1 +
b971b8
 tests/cputestdata/x86_64-cpuid-Core-i7-8550U-host.xml         | 1 +
b971b8
 tests/cputestdata/x86_64-cpuid-Core-i7-8550U-json.xml         | 1 +
b971b8
 tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-enabled.xml | 2 +-
b971b8
 tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-json.xml    | 1 +
b971b8
 tests/domaincapsdata/qemu_4.2.0-q35.x86_64.xml                | 1 +
b971b8
 tests/domaincapsdata/qemu_4.2.0.x86_64.xml                    | 1 +
b971b8
 tests/domaincapsdata/qemu_5.0.0-q35.x86_64.xml                | 1 +
b971b8
 tests/domaincapsdata/qemu_5.0.0.x86_64.xml                    | 1 +
b971b8
 14 files changed, 16 insertions(+), 2 deletions(-)
b971b8
b971b8
diff --git a/src/cpu_map/x86_features.xml b/src/cpu_map/x86_features.xml
b971b8
index 2c4c29dc99..8525ae0fa5 100644
b971b8
--- a/src/cpu_map/x86_features.xml
b971b8
+++ b/src/cpu_map/x86_features.xml
b971b8
@@ -509,6 +509,9 @@
b971b8
   <feature name='mds-no'>
b971b8
     <msr index='0x10a' edx='0x00000000' eax='0x00000020'/>
b971b8
   </feature>
b971b8
+  <feature name='pschange-mc-no'>
b971b8
+    <msr index='0x10a' edx='0x00000000' eax='0x00000040'/>
b971b8
+  </feature>
b971b8
   <feature name='tsx-ctrl'>
b971b8
     <msr index='0x10a' edx='0x00000000' eax='0x00000080'/>
b971b8
   </feature>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Cooperlake-guest.xml b/tests/cputestdata/x86_64-cpuid-Cooperlake-guest.xml
b971b8
index b5137e3c03..db5598740c 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Cooperlake-guest.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Cooperlake-guest.xml
b971b8
@@ -35,6 +35,7 @@
b971b8
   <feature policy='require' name='ibrs-all'/>
b971b8
   <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
   <feature policy='require' name='mds-no'/>
b971b8
+  <feature policy='require' name='pschange-mc-no'/>
b971b8
   <feature policy='require' name='tsx-ctrl'/>
b971b8
   <feature policy='require' name='taa-no'/>
b971b8
 </cpu>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Cooperlake-host.xml b/tests/cputestdata/x86_64-cpuid-Cooperlake-host.xml
b971b8
index 52b352f4f1..6926b7c975 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Cooperlake-host.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Cooperlake-host.xml
b971b8
@@ -36,6 +36,7 @@
b971b8
   <feature name='ibrs-all'/>
b971b8
   <feature name='skip-l1dfl-vmentry'/>
b971b8
   <feature name='mds-no'/>
b971b8
+  <feature name='pschange-mc-no'/>
b971b8
   <feature name='tsx-ctrl'/>
b971b8
   <feature name='taa-no'/>
b971b8
 </cpu>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Cooperlake-json.xml b/tests/cputestdata/x86_64-cpuid-Cooperlake-json.xml
b971b8
index e323d6205e..fc52805b2f 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Cooperlake-json.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Cooperlake-json.xml
b971b8
@@ -18,6 +18,7 @@
b971b8
   <feature policy='require' name='ibrs-all'/>
b971b8
   <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
   <feature policy='require' name='mds-no'/>
b971b8
+  <feature policy='require' name='pschange-mc-no'/>
b971b8
   <feature policy='require' name='tsx-ctrl'/>
b971b8
   <feature policy='require' name='taa-no'/>
b971b8
 </cpu>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-enabled.xml b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-enabled.xml
b971b8
index 6c480eeacf..57f8ebabba 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-enabled.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-enabled.xml
b971b8
@@ -5,5 +5,5 @@
b971b8
   <cpuid eax_in='0x00000007' ecx_in='0x00' eax='0x00000000' ebx='0x009c47ab' ecx='0x00000004' edx='0xac000400'/>
b971b8
   <cpuid eax_in='0x0000000d' ecx_in='0x01' eax='0x0000000f' ebx='0x00000000' ecx='0x00000000' edx='0x00000000'/>
b971b8
   <cpuid eax_in='0x80000001' ecx_in='0x00' eax='0x00000000' ebx='0x00000000' ecx='0x00000121' edx='0x2c100800'/>
b971b8
-  <msr index='0x10a' edx='0x00000000' eax='0x00000008'/>
b971b8
+  <msr index='0x10a' edx='0x00000000' eax='0x00000048'/>
b971b8
 </cpudata>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-guest.xml b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-guest.xml
b971b8
index 92404e4d03..ed06515e99 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-guest.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-guest.xml
b971b8
@@ -26,6 +26,7 @@
b971b8
   <feature policy='require' name='pdpe1gb'/>
b971b8
   <feature policy='require' name='invtsc'/>
b971b8
   <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
+  <feature policy='require' name='pschange-mc-no'/>
b971b8
   <feature policy='disable' name='hle'/>
b971b8
   <feature policy='disable' name='rtm'/>
b971b8
 </cpu>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-host.xml b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-host.xml
b971b8
index 7f6fe2eac3..7681c94649 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-host.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-host.xml
b971b8
@@ -27,4 +27,5 @@
b971b8
   <feature name='pdpe1gb'/>
b971b8
   <feature name='invtsc'/>
b971b8
   <feature name='skip-l1dfl-vmentry'/>
b971b8
+  <feature name='pschange-mc-no'/>
b971b8
 </cpu>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-json.xml b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-json.xml
b971b8
index 645c0934c2..4774d39c7e 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-json.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Core-i7-8550U-json.xml
b971b8
@@ -14,6 +14,7 @@
b971b8
   <feature policy='require' name='xsaves'/>
b971b8
   <feature policy='require' name='pdpe1gb'/>
b971b8
   <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
+  <feature policy='require' name='pschange-mc-no'/>
b971b8
   <feature policy='disable' name='hle'/>
b971b8
   <feature policy='disable' name='rtm'/>
b971b8
 </cpu>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-enabled.xml b/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-enabled.xml
b971b8
index 43c3a93a16..cd082feffe 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-enabled.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-enabled.xml
b971b8
@@ -6,5 +6,5 @@
b971b8
   <cpuid eax_in='0x0000000d' ecx_in='0x01' eax='0x0000000f' ebx='0x00000000' ecx='0x00000000' edx='0x00000000'/>
b971b8
   <cpuid eax_in='0x80000001' ecx_in='0x00' eax='0x00000000' ebx='0x00000000' ecx='0x00000121' edx='0x2c100800'/>
b971b8
   <cpuid eax_in='0x80000008' ecx_in='0x00' eax='0x00000000' ebx='0x01001000' ecx='0x00000000' edx='0x00000000'/>
b971b8
-  <msr index='0x10a' edx='0x00000000' eax='0x000000ab'/>
b971b8
+  <msr index='0x10a' edx='0x00000000' eax='0x000000eb'/>
b971b8
 </cpudata>
b971b8
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-json.xml b/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-json.xml
b971b8
index 36e95029be..10c256ee1e 100644
b971b8
--- a/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-json.xml
b971b8
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-Platinum-9242-json.xml
b971b8
@@ -17,5 +17,6 @@
b971b8
   <feature policy='require' name='ibrs-all'/>
b971b8
   <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
   <feature policy='require' name='mds-no'/>
b971b8
+  <feature policy='require' name='pschange-mc-no'/>
b971b8
   <feature policy='require' name='tsx-ctrl'/>
b971b8
 </cpu>
b971b8
diff --git a/tests/domaincapsdata/qemu_4.2.0-q35.x86_64.xml b/tests/domaincapsdata/qemu_4.2.0-q35.x86_64.xml
b971b8
index e257657981..4d9616cb69 100644
b971b8
--- a/tests/domaincapsdata/qemu_4.2.0-q35.x86_64.xml
b971b8
+++ b/tests/domaincapsdata/qemu_4.2.0-q35.x86_64.xml
b971b8
@@ -47,6 +47,7 @@
b971b8
       <feature policy='require' name='pdpe1gb'/>
b971b8
       <feature policy='require' name='invtsc'/>
b971b8
       <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
+      <feature policy='require' name='pschange-mc-no'/>
b971b8
     </mode>
b971b8
     <mode name='custom' supported='yes'>
b971b8
       <model usable='yes'>qemu64</model>
b971b8
diff --git a/tests/domaincapsdata/qemu_4.2.0.x86_64.xml b/tests/domaincapsdata/qemu_4.2.0.x86_64.xml
b971b8
index 2918ba9715..5210c917aa 100644
b971b8
--- a/tests/domaincapsdata/qemu_4.2.0.x86_64.xml
b971b8
+++ b/tests/domaincapsdata/qemu_4.2.0.x86_64.xml
b971b8
@@ -46,6 +46,7 @@
b971b8
       <feature policy='require' name='pdpe1gb'/>
b971b8
       <feature policy='require' name='invtsc'/>
b971b8
       <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
+      <feature policy='require' name='pschange-mc-no'/>
b971b8
     </mode>
b971b8
     <mode name='custom' supported='yes'>
b971b8
       <model usable='yes'>qemu64</model>
b971b8
diff --git a/tests/domaincapsdata/qemu_5.0.0-q35.x86_64.xml b/tests/domaincapsdata/qemu_5.0.0-q35.x86_64.xml
b971b8
index 7b2a929738..b7e2c8a4fe 100644
b971b8
--- a/tests/domaincapsdata/qemu_5.0.0-q35.x86_64.xml
b971b8
+++ b/tests/domaincapsdata/qemu_5.0.0-q35.x86_64.xml
b971b8
@@ -47,6 +47,7 @@
b971b8
       <feature policy='require' name='pdpe1gb'/>
b971b8
       <feature policy='require' name='invtsc'/>
b971b8
       <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
+      <feature policy='require' name='pschange-mc-no'/>
b971b8
     </mode>
b971b8
     <mode name='custom' supported='yes'>
b971b8
       <model usable='yes'>qemu64</model>
b971b8
diff --git a/tests/domaincapsdata/qemu_5.0.0.x86_64.xml b/tests/domaincapsdata/qemu_5.0.0.x86_64.xml
b971b8
index 7609d49020..d2741375b4 100644
b971b8
--- a/tests/domaincapsdata/qemu_5.0.0.x86_64.xml
b971b8
+++ b/tests/domaincapsdata/qemu_5.0.0.x86_64.xml
b971b8
@@ -46,6 +46,7 @@
b971b8
       <feature policy='require' name='pdpe1gb'/>
b971b8
       <feature policy='require' name='invtsc'/>
b971b8
       <feature policy='require' name='skip-l1dfl-vmentry'/>
b971b8
+      <feature policy='require' name='pschange-mc-no'/>
b971b8
     </mode>
b971b8
     <mode name='custom' supported='yes'>
b971b8
       <model usable='yes'>qemu64</model>
b971b8
-- 
b971b8
2.26.2
b971b8