99cbc7
From e7e1e3003d52fee05ef040e273c590a66d90e6f9 Mon Sep 17 00:00:00 2001
99cbc7
Message-Id: <e7e1e3003d52fee05ef040e273c590a66d90e6f9@dist-git>
99cbc7
From: Jiri Denemark <Jiri.Denemark@gmail.com>
99cbc7
Date: Wed, 20 Mar 2019 17:15:16 +0100
99cbc7
Subject: [PATCH] cpu_map: Add more signatures for Broadwell CPU models
99cbc7
MIME-Version: 1.0
99cbc7
Content-Type: text/plain; charset=UTF-8
99cbc7
Content-Transfer-Encoding: 8bit
99cbc7
99cbc7
This fixes several CPUs which were incorrectly detected as
99cbc7
Skylake-Client.
99cbc7
99cbc7
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
99cbc7
Reviewed-by: Ján Tomko <jtomko@redhat.com>
99cbc7
(cherry picked from commit 4ff74a806ad42820eef3877c8ec146770914d8df)
99cbc7
99cbc7
https://bugzilla.redhat.com/show_bug.cgi?id=1558558
99cbc7
https://bugzilla.redhat.com/show_bug.cgi?id=1687515
99cbc7
99cbc7
Conflicts:
99cbc7
	src/cpu_map/x86_Broadwell-IBRS.xml
99cbc7
	src/cpu_map/x86_Broadwell-noTSX-IBRS.xml
99cbc7
	src/cpu_map/x86_Broadwell-noTSX.xml
99cbc7
	src/cpu_map/x86_Broadwell.xml
99cbc7
            - cpu_map split not backported
99cbc7
99cbc7
	tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-guest.xml
99cbc7
            - context: intel-pt feature does not exist in RHEL-7.6
99cbc7
99cbc7
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
99cbc7
Reviewed-by: Ján Tomko <jtomko@redhat.com>
99cbc7
---
99cbc7
 src/cpu/cpu_map.xml                                  | 12 ++++++++++++
99cbc7
 .../x86_64-cpuid-Xeon-E5-2623-v4-guest.xml           | 11 +++++++----
99cbc7
 .../x86_64-cpuid-Xeon-E5-2623-v4-json.xml            | 11 +++++++----
99cbc7
 .../x86_64-cpuid-Xeon-E5-2630-v4-guest.xml           | 11 +++++++----
99cbc7
 .../x86_64-cpuid-Xeon-E5-2630-v4-json.xml            | 11 +++++++----
99cbc7
 .../x86_64-cpuid-Xeon-E5-2650-v4-guest.xml           | 11 +++++++----
99cbc7
 .../x86_64-cpuid-Xeon-E5-2650-v4-json.xml            | 11 +++++++----
99cbc7
 7 files changed, 54 insertions(+), 24 deletions(-)
99cbc7
99cbc7
diff --git a/src/cpu/cpu_map.xml b/src/cpu/cpu_map.xml
99cbc7
index 7c6f70113e..095ea0a839 100644
99cbc7
--- a/src/cpu/cpu_map.xml
99cbc7
+++ b/src/cpu/cpu_map.xml
99cbc7
@@ -1434,6 +1434,9 @@
99cbc7
 
99cbc7
     <model name='Broadwell-noTSX'>
99cbc7
       <signature family='6' model='61'/> 
99cbc7
+      <signature family='6' model='71'/> 
99cbc7
+      <signature family='6' model='79'/> 
99cbc7
+      <signature family='6' model='86'/> 
99cbc7
       <vendor name='Intel'/>
99cbc7
       <feature name='3dnowprefetch'/>
99cbc7
       <feature name='adx'/>
99cbc7
@@ -1491,6 +1494,9 @@
99cbc7
 
99cbc7
     <model name='Broadwell-noTSX-IBRS'>
99cbc7
       <signature family='6' model='61'/> 
99cbc7
+      <signature family='6' model='71'/> 
99cbc7
+      <signature family='6' model='79'/> 
99cbc7
+      <signature family='6' model='86'/> 
99cbc7
       <vendor name='Intel'/>
99cbc7
       <feature name='3dnowprefetch'/>
99cbc7
       <feature name='adx'/>
99cbc7
@@ -1549,6 +1555,9 @@
99cbc7
 
99cbc7
     <model name='Broadwell'>
99cbc7
       <signature family='6' model='61'/> 
99cbc7
+      <signature family='6' model='71'/> 
99cbc7
+      <signature family='6' model='79'/> 
99cbc7
+      <signature family='6' model='86'/> 
99cbc7
       <vendor name='Intel'/>
99cbc7
       <feature name='3dnowprefetch'/>
99cbc7
       <feature name='adx'/>
99cbc7
@@ -1608,6 +1617,9 @@
99cbc7
 
99cbc7
     <model name='Broadwell-IBRS'>
99cbc7
       <signature family='6' model='61'/> 
99cbc7
+      <signature family='6' model='71'/> 
99cbc7
+      <signature family='6' model='79'/> 
99cbc7
+      <signature family='6' model='86'/> 
99cbc7
       <vendor name='Intel'/>
99cbc7
       <feature name='3dnowprefetch'/>
99cbc7
       <feature name='adx'/>
99cbc7
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-guest.xml b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-guest.xml
99cbc7
index d4f303ecce..e2ad344497 100644
99cbc7
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-guest.xml
99cbc7
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-guest.xml
99cbc7
@@ -1,6 +1,7 @@
99cbc7
 <cpu mode='custom' match='exact'>
99cbc7
-  <model fallback='forbid'>Skylake-Client-IBRS</model>
99cbc7
+  <model fallback='forbid'>Broadwell-IBRS</model>
99cbc7
   <vendor>Intel</vendor>
99cbc7
+  <feature policy='require' name='vme'/>
99cbc7
   <feature policy='require' name='ds'/>
99cbc7
   <feature policy='require' name='acpi'/>
99cbc7
   <feature policy='require' name='ss'/>
99cbc7
@@ -18,14 +19,16 @@
99cbc7
   <feature policy='require' name='pdcm'/>
99cbc7
   <feature policy='require' name='dca'/>
99cbc7
   <feature policy='require' name='osxsave'/>
99cbc7
+  <feature policy='require' name='f16c'/>
99cbc7
+  <feature policy='require' name='rdrand'/>
99cbc7
+  <feature policy='require' name='arat'/>
99cbc7
   <feature policy='require' name='tsc_adjust'/>
99cbc7
   <feature policy='require' name='cmt'/>
99cbc7
   <feature policy='require' name='stibp'/>
99cbc7
+  <feature policy='require' name='xsaveopt'/>
99cbc7
   <feature policy='require' name='mbm_total'/>
99cbc7
   <feature policy='require' name='mbm_local'/>
99cbc7
   <feature policy='require' name='pdpe1gb'/>
99cbc7
+  <feature policy='require' name='abm'/>
99cbc7
   <feature policy='require' name='invtsc'/>
99cbc7
-  <feature policy='disable' name='mpx'/>
99cbc7
-  <feature policy='disable' name='xsavec'/>
99cbc7
-  <feature policy='disable' name='xgetbv1'/>
99cbc7
 </cpu>
99cbc7
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-json.xml b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-json.xml
99cbc7
index 167a9028ab..de082dbd93 100644
99cbc7
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-json.xml
99cbc7
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-json.xml
99cbc7
@@ -1,11 +1,14 @@
99cbc7
 <cpu mode='custom' match='exact'>
99cbc7
-  <model fallback='forbid'>Skylake-Client-IBRS</model>
99cbc7
+  <model fallback='forbid'>Broadwell-IBRS</model>
99cbc7
   <vendor>Intel</vendor>
99cbc7
+  <feature policy='require' name='vme'/>
99cbc7
   <feature policy='require' name='ss'/>
99cbc7
+  <feature policy='require' name='f16c'/>
99cbc7
+  <feature policy='require' name='rdrand'/>
99cbc7
   <feature policy='require' name='hypervisor'/>
99cbc7
+  <feature policy='require' name='arat'/>
99cbc7
   <feature policy='require' name='tsc_adjust'/>
99cbc7
+  <feature policy='require' name='xsaveopt'/>
99cbc7
   <feature policy='require' name='pdpe1gb'/>
99cbc7
-  <feature policy='disable' name='mpx'/>
99cbc7
-  <feature policy='disable' name='xsavec'/>
99cbc7
-  <feature policy='disable' name='xgetbv1'/>
99cbc7
+  <feature policy='require' name='abm'/>
99cbc7
 </cpu>
99cbc7
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-guest.xml b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-guest.xml
99cbc7
index 2fac54355c..c670b1f4df 100644
99cbc7
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-guest.xml
99cbc7
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-guest.xml
99cbc7
@@ -1,6 +1,7 @@
99cbc7
 <cpu mode='custom' match='exact'>
99cbc7
-  <model fallback='forbid'>Skylake-Client</model>
99cbc7
+  <model fallback='forbid'>Broadwell</model>
99cbc7
   <vendor>Intel</vendor>
99cbc7
+  <feature policy='require' name='vme'/>
99cbc7
   <feature policy='require' name='ds'/>
99cbc7
   <feature policy='require' name='acpi'/>
99cbc7
   <feature policy='require' name='ss'/>
99cbc7
@@ -18,13 +19,15 @@
99cbc7
   <feature policy='require' name='pdcm'/>
99cbc7
   <feature policy='require' name='dca'/>
99cbc7
   <feature policy='require' name='osxsave'/>
99cbc7
+  <feature policy='require' name='f16c'/>
99cbc7
+  <feature policy='require' name='rdrand'/>
99cbc7
+  <feature policy='require' name='arat'/>
99cbc7
   <feature policy='require' name='tsc_adjust'/>
99cbc7
   <feature policy='require' name='cmt'/>
99cbc7
+  <feature policy='require' name='xsaveopt'/>
99cbc7
   <feature policy='require' name='mbm_total'/>
99cbc7
   <feature policy='require' name='mbm_local'/>
99cbc7
   <feature policy='require' name='pdpe1gb'/>
99cbc7
+  <feature policy='require' name='abm'/>
99cbc7
   <feature policy='require' name='invtsc'/>
99cbc7
-  <feature policy='disable' name='mpx'/>
99cbc7
-  <feature policy='disable' name='xsavec'/>
99cbc7
-  <feature policy='disable' name='xgetbv1'/>
99cbc7
 </cpu>
99cbc7
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-json.xml b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-json.xml
99cbc7
index 5dfce947b2..5b8891093a 100644
99cbc7
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-json.xml
99cbc7
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-json.xml
99cbc7
@@ -1,11 +1,14 @@
99cbc7
 <cpu mode='custom' match='exact'>
99cbc7
-  <model fallback='forbid'>Skylake-Client</model>
99cbc7
+  <model fallback='forbid'>Broadwell</model>
99cbc7
   <vendor>Intel</vendor>
99cbc7
+  <feature policy='require' name='vme'/>
99cbc7
   <feature policy='require' name='ss'/>
99cbc7
+  <feature policy='require' name='f16c'/>
99cbc7
+  <feature policy='require' name='rdrand'/>
99cbc7
   <feature policy='require' name='hypervisor'/>
99cbc7
+  <feature policy='require' name='arat'/>
99cbc7
   <feature policy='require' name='tsc_adjust'/>
99cbc7
+  <feature policy='require' name='xsaveopt'/>
99cbc7
   <feature policy='require' name='pdpe1gb'/>
99cbc7
-  <feature policy='disable' name='mpx'/>
99cbc7
-  <feature policy='disable' name='xsavec'/>
99cbc7
-  <feature policy='disable' name='xgetbv1'/>
99cbc7
+  <feature policy='require' name='abm'/>
99cbc7
 </cpu>
99cbc7
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-guest.xml b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-guest.xml
99cbc7
index 2fac54355c..c670b1f4df 100644
99cbc7
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-guest.xml
99cbc7
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-guest.xml
99cbc7
@@ -1,6 +1,7 @@
99cbc7
 <cpu mode='custom' match='exact'>
99cbc7
-  <model fallback='forbid'>Skylake-Client</model>
99cbc7
+  <model fallback='forbid'>Broadwell</model>
99cbc7
   <vendor>Intel</vendor>
99cbc7
+  <feature policy='require' name='vme'/>
99cbc7
   <feature policy='require' name='ds'/>
99cbc7
   <feature policy='require' name='acpi'/>
99cbc7
   <feature policy='require' name='ss'/>
99cbc7
@@ -18,13 +19,15 @@
99cbc7
   <feature policy='require' name='pdcm'/>
99cbc7
   <feature policy='require' name='dca'/>
99cbc7
   <feature policy='require' name='osxsave'/>
99cbc7
+  <feature policy='require' name='f16c'/>
99cbc7
+  <feature policy='require' name='rdrand'/>
99cbc7
+  <feature policy='require' name='arat'/>
99cbc7
   <feature policy='require' name='tsc_adjust'/>
99cbc7
   <feature policy='require' name='cmt'/>
99cbc7
+  <feature policy='require' name='xsaveopt'/>
99cbc7
   <feature policy='require' name='mbm_total'/>
99cbc7
   <feature policy='require' name='mbm_local'/>
99cbc7
   <feature policy='require' name='pdpe1gb'/>
99cbc7
+  <feature policy='require' name='abm'/>
99cbc7
   <feature policy='require' name='invtsc'/>
99cbc7
-  <feature policy='disable' name='mpx'/>
99cbc7
-  <feature policy='disable' name='xsavec'/>
99cbc7
-  <feature policy='disable' name='xgetbv1'/>
99cbc7
 </cpu>
99cbc7
diff --git a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-json.xml b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-json.xml
99cbc7
index 5dfce947b2..5b8891093a 100644
99cbc7
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-json.xml
99cbc7
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-json.xml
99cbc7
@@ -1,11 +1,14 @@
99cbc7
 <cpu mode='custom' match='exact'>
99cbc7
-  <model fallback='forbid'>Skylake-Client</model>
99cbc7
+  <model fallback='forbid'>Broadwell</model>
99cbc7
   <vendor>Intel</vendor>
99cbc7
+  <feature policy='require' name='vme'/>
99cbc7
   <feature policy='require' name='ss'/>
99cbc7
+  <feature policy='require' name='f16c'/>
99cbc7
+  <feature policy='require' name='rdrand'/>
99cbc7
   <feature policy='require' name='hypervisor'/>
99cbc7
+  <feature policy='require' name='arat'/>
99cbc7
   <feature policy='require' name='tsc_adjust'/>
99cbc7
+  <feature policy='require' name='xsaveopt'/>
99cbc7
   <feature policy='require' name='pdpe1gb'/>
99cbc7
-  <feature policy='disable' name='mpx'/>
99cbc7
-  <feature policy='disable' name='xsavec'/>
99cbc7
-  <feature policy='disable' name='xgetbv1'/>
99cbc7
+  <feature policy='require' name='abm'/>
99cbc7
 </cpu>
99cbc7
-- 
99cbc7
2.21.0
99cbc7