a41c76
From f1926a0cb9147ce54484858f5e0f7376ee9ac648 Mon Sep 17 00:00:00 2001
a41c76
Message-Id: <f1926a0cb9147ce54484858f5e0f7376ee9ac648@dist-git>
a41c76
From: Jiri Denemark <jdenemar@redhat.com>
a41c76
Date: Tue, 26 May 2020 10:59:37 +0200
a41c76
Subject: [PATCH] cpu_map: Distinguish Cascadelake-Server from Skylake-Server
a41c76
MIME-Version: 1.0
a41c76
Content-Type: text/plain; charset=UTF-8
a41c76
Content-Transfer-Encoding: 8bit
a41c76
a41c76
The signatures of these two CPU model differ only in stepping as both
a41c76
report family 6 and model 85. Skylake-Server uses stepping 4 or less and
a41c76
Cascadelake-Server uses stepping 5..7.
a41c76
a41c76
https://bugzilla.redhat.com/show_bug.cgi?id=1761678
a41c76
a41c76
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
a41c76
Reviewed-by: Ján Tomko <jtomko@redhat.com>
a41c76
(cherry picked from commit 5d6059f8ec16d64f240dc5e6413ca55a3b46b3f7)
a41c76
a41c76
https://bugzilla.redhat.com/show_bug.cgi?id=1840010
a41c76
a41c76
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
a41c76
Message-Id: <8711c401083e2605e7d8734764a5fe1b092749b0.1590483392.git.jdenemar@redhat.com>
a41c76
Reviewed-by: Ján Tomko <jtomko@redhat.com>
a41c76
---
a41c76
 src/cpu_map/x86_Cascadelake-Server-noTSX.xml            | 2 +-
a41c76
 src/cpu_map/x86_Cascadelake-Server.xml                  | 2 +-
a41c76
 src/cpu_map/x86_Skylake-Server-IBRS.xml                 | 2 +-
a41c76
 src/cpu_map/x86_Skylake-Server-noTSX-IBRS.xml           | 2 +-
a41c76
 src/cpu_map/x86_Skylake-Server.xml                      | 2 +-
a41c76
 tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-guest.xml | 5 +++--
a41c76
 tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-json.xml  | 5 +++--
a41c76
 7 files changed, 11 insertions(+), 9 deletions(-)
a41c76
a41c76
diff --git a/src/cpu_map/x86_Cascadelake-Server-noTSX.xml b/src/cpu_map/x86_Cascadelake-Server-noTSX.xml
a41c76
index 459174a30d..bfd4629836 100644
a41c76
--- a/src/cpu_map/x86_Cascadelake-Server-noTSX.xml
a41c76
+++ b/src/cpu_map/x86_Cascadelake-Server-noTSX.xml
a41c76
@@ -1,7 +1,7 @@
a41c76
 <cpus>
a41c76
   <model name='Cascadelake-Server-noTSX'>
a41c76
     <decode host='on' guest='off'/>
a41c76
-    <signature family='6' model='85'/> 
a41c76
+    <signature family='6' model='85' stepping='5-7'/> 
a41c76
     <vendor name='Intel'/>
a41c76
     <feature name='3dnowprefetch'/>
a41c76
     <feature name='abm'/>
a41c76
diff --git a/src/cpu_map/x86_Cascadelake-Server.xml b/src/cpu_map/x86_Cascadelake-Server.xml
a41c76
index d7ec42f57e..335e9cb584 100644
a41c76
--- a/src/cpu_map/x86_Cascadelake-Server.xml
a41c76
+++ b/src/cpu_map/x86_Cascadelake-Server.xml
a41c76
@@ -1,7 +1,7 @@
a41c76
 <cpus>
a41c76
   <model name='Cascadelake-Server'>
a41c76
     <decode host='on' guest='on'/>
a41c76
-    <signature family='6' model='85'/> 
a41c76
+    <signature family='6' model='85' stepping='5-7'/> 
a41c76
     <vendor name='Intel'/>
a41c76
     <feature name='3dnowprefetch'/>
a41c76
     <feature name='abm'/>
a41c76
diff --git a/src/cpu_map/x86_Skylake-Server-IBRS.xml b/src/cpu_map/x86_Skylake-Server-IBRS.xml
a41c76
index bd6b6457ad..9fb3488809 100644
a41c76
--- a/src/cpu_map/x86_Skylake-Server-IBRS.xml
a41c76
+++ b/src/cpu_map/x86_Skylake-Server-IBRS.xml
a41c76
@@ -1,7 +1,7 @@
a41c76
 <cpus>
a41c76
   <model name='Skylake-Server-IBRS'>
a41c76
     <decode host='on' guest='on'/>
a41c76
-    <signature family='6' model='85'/> 
a41c76
+    <signature family='6' model='85' stepping='0-4'/> 
a41c76
     <vendor name='Intel'/>
a41c76
     <feature name='3dnowprefetch'/>
a41c76
     <feature name='abm'/>
a41c76
diff --git a/src/cpu_map/x86_Skylake-Server-noTSX-IBRS.xml b/src/cpu_map/x86_Skylake-Server-noTSX-IBRS.xml
a41c76
index c2b7de40e8..c162c0acc3 100644
a41c76
--- a/src/cpu_map/x86_Skylake-Server-noTSX-IBRS.xml
a41c76
+++ b/src/cpu_map/x86_Skylake-Server-noTSX-IBRS.xml
a41c76
@@ -1,7 +1,7 @@
a41c76
 <cpus>
a41c76
   <model name='Skylake-Server-noTSX-IBRS'>
a41c76
     <decode host='on' guest='off'/>
a41c76
-    <signature family='6' model='85'/> 
a41c76
+    <signature family='6' model='85' stepping='0-4'/> 
a41c76
     <vendor name='Intel'/>
a41c76
     <feature name='3dnowprefetch'/>
a41c76
     <feature name='abm'/>
a41c76
diff --git a/src/cpu_map/x86_Skylake-Server.xml b/src/cpu_map/x86_Skylake-Server.xml
a41c76
index f96875a85f..e022d94c84 100644
a41c76
--- a/src/cpu_map/x86_Skylake-Server.xml
a41c76
+++ b/src/cpu_map/x86_Skylake-Server.xml
a41c76
@@ -1,7 +1,7 @@
a41c76
 <cpus>
a41c76
   <model name='Skylake-Server'>
a41c76
     <decode host='on' guest='on'/>
a41c76
-    <signature family='6' model='85'/> 
a41c76
+    <signature family='6' model='85' stepping='0-4'/> 
a41c76
     <vendor name='Intel'/>
a41c76
     <feature name='3dnowprefetch'/>
a41c76
     <feature name='abm'/>
a41c76
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-guest.xml b/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-guest.xml
a41c76
index cfb0eb982d..c21cb1e600 100644
a41c76
--- a/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-guest.xml
a41c76
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-guest.xml
a41c76
@@ -1,5 +1,5 @@
a41c76
 <cpu mode='custom' match='exact'>
a41c76
-  <model fallback='forbid'>Cascadelake-Server</model>
a41c76
+  <model fallback='forbid'>Skylake-Server-IBRS</model>
a41c76
   <vendor>Intel</vendor>
a41c76
   <feature policy='require' name='ds'/>
a41c76
   <feature policy='require' name='acpi'/>
a41c76
@@ -20,14 +20,15 @@
a41c76
   <feature policy='require' name='osxsave'/>
a41c76
   <feature policy='require' name='tsc_adjust'/>
a41c76
   <feature policy='require' name='cmt'/>
a41c76
+  <feature policy='require' name='clflushopt'/>
a41c76
   <feature policy='require' name='intel-pt'/>
a41c76
   <feature policy='require' name='pku'/>
a41c76
   <feature policy='require' name='ospke'/>
a41c76
   <feature policy='require' name='md-clear'/>
a41c76
   <feature policy='require' name='stibp'/>
a41c76
+  <feature policy='require' name='ssbd'/>
a41c76
   <feature policy='require' name='xsaves'/>
a41c76
   <feature policy='require' name='mbm_total'/>
a41c76
   <feature policy='require' name='mbm_local'/>
a41c76
   <feature policy='require' name='invtsc'/>
a41c76
-  <feature policy='disable' name='avx512vnni'/>
a41c76
 </cpu>
a41c76
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-json.xml b/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-json.xml
a41c76
index 968d904557..b482043ccf 100644
a41c76
--- a/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-json.xml
a41c76
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-Gold-6130-json.xml
a41c76
@@ -1,16 +1,17 @@
a41c76
 <cpu mode='custom' match='exact'>
a41c76
-  <model fallback='forbid'>Cascadelake-Server</model>
a41c76
+  <model fallback='forbid'>Skylake-Server-IBRS</model>
a41c76
   <vendor>Intel</vendor>
a41c76
   <feature policy='require' name='ss'/>
a41c76
   <feature policy='require' name='vmx'/>
a41c76
   <feature policy='require' name='hypervisor'/>
a41c76
   <feature policy='require' name='tsc_adjust'/>
a41c76
+  <feature policy='require' name='clflushopt'/>
a41c76
   <feature policy='require' name='umip'/>
a41c76
   <feature policy='require' name='pku'/>
a41c76
   <feature policy='require' name='md-clear'/>
a41c76
   <feature policy='require' name='stibp'/>
a41c76
   <feature policy='require' name='arch-capabilities'/>
a41c76
+  <feature policy='require' name='ssbd'/>
a41c76
   <feature policy='require' name='xsaves'/>
a41c76
   <feature policy='require' name='skip-l1dfl-vmentry'/>
a41c76
-  <feature policy='disable' name='avx512vnni'/>
a41c76
 </cpu>
a41c76
-- 
a41c76
2.26.2
a41c76