render / rpms / libvirt

Forked from rpms/libvirt 10 months ago
Clone
c480ed
From 77ad41e460c2e91f2c2e187c80581069654b741d Mon Sep 17 00:00:00 2001
c480ed
Message-Id: <77ad41e460c2e91f2c2e187c80581069654b741d@dist-git>
c480ed
From: Jiri Denemark <Jiri.Denemark@gmail.com>
c480ed
Date: Fri, 21 Jun 2019 09:25:16 +0200
c480ed
Subject: [PATCH] cpu_map: Add more signatures for Broadwell CPU models
c480ed
MIME-Version: 1.0
c480ed
Content-Type: text/plain; charset=UTF-8
c480ed
Content-Transfer-Encoding: 8bit
c480ed
c480ed
This fixes several CPUs which were incorrectly detected as
c480ed
Skylake-Client.
c480ed
c480ed
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
c480ed
Reviewed-by: Ján Tomko <jtomko@redhat.com>
c480ed
(cherry picked from commit 4ff74a806ad42820eef3877c8ec146770914d8df)
c480ed
c480ed
https://bugzilla.redhat.com/show_bug.cgi?id=1686895
c480ed
c480ed
Conflicts:
c480ed
	src/cpu_map/x86_Broadwell-IBRS.xml
c480ed
	src/cpu_map/x86_Broadwell-noTSX-IBRS.xml
c480ed
	src/cpu_map/x86_Broadwell-noTSX.xml
c480ed
	src/cpu_map/x86_Broadwell.xml
c480ed
            - cpu_map split not backported
c480ed
c480ed
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
c480ed
Message-Id: <543534b544d2d09470c218aeb6c7d945facaf2c8.1561068591.git.jdenemar@redhat.com>
c480ed
Reviewed-by: Ján Tomko <jtomko@redhat.com>
c480ed
---
c480ed
 src/cpu/cpu_map.xml                                  | 12 ++++++++++++
c480ed
 .../x86_64-cpuid-Xeon-E5-2623-v4-guest.xml           | 11 +++++++----
c480ed
 .../x86_64-cpuid-Xeon-E5-2623-v4-json.xml            | 11 +++++++----
c480ed
 .../x86_64-cpuid-Xeon-E5-2630-v4-guest.xml           | 11 +++++++----
c480ed
 .../x86_64-cpuid-Xeon-E5-2630-v4-json.xml            | 11 +++++++----
c480ed
 .../x86_64-cpuid-Xeon-E5-2650-v4-guest.xml           | 11 +++++++----
c480ed
 .../x86_64-cpuid-Xeon-E5-2650-v4-json.xml            | 11 +++++++----
c480ed
 7 files changed, 54 insertions(+), 24 deletions(-)
c480ed
c480ed
diff --git a/src/cpu/cpu_map.xml b/src/cpu/cpu_map.xml
c480ed
index ed6006643b..04369d1eda 100644
c480ed
--- a/src/cpu/cpu_map.xml
c480ed
+++ b/src/cpu/cpu_map.xml
c480ed
@@ -1473,6 +1473,9 @@
c480ed
 
c480ed
     <model name='Broadwell-noTSX'>
c480ed
       <signature family='6' model='61'/> 
c480ed
+      <signature family='6' model='71'/> 
c480ed
+      <signature family='6' model='79'/> 
c480ed
+      <signature family='6' model='86'/> 
c480ed
       <vendor name='Intel'/>
c480ed
       <feature name='3dnowprefetch'/>
c480ed
       <feature name='adx'/>
c480ed
@@ -1530,6 +1533,9 @@
c480ed
 
c480ed
     <model name='Broadwell-noTSX-IBRS'>
c480ed
       <signature family='6' model='61'/> 
c480ed
+      <signature family='6' model='71'/> 
c480ed
+      <signature family='6' model='79'/> 
c480ed
+      <signature family='6' model='86'/> 
c480ed
       <vendor name='Intel'/>
c480ed
       <feature name='3dnowprefetch'/>
c480ed
       <feature name='adx'/>
c480ed
@@ -1588,6 +1594,9 @@
c480ed
 
c480ed
     <model name='Broadwell'>
c480ed
       <signature family='6' model='61'/> 
c480ed
+      <signature family='6' model='71'/> 
c480ed
+      <signature family='6' model='79'/> 
c480ed
+      <signature family='6' model='86'/> 
c480ed
       <vendor name='Intel'/>
c480ed
       <feature name='3dnowprefetch'/>
c480ed
       <feature name='adx'/>
c480ed
@@ -1647,6 +1656,9 @@
c480ed
 
c480ed
     <model name='Broadwell-IBRS'>
c480ed
       <signature family='6' model='61'/> 
c480ed
+      <signature family='6' model='71'/> 
c480ed
+      <signature family='6' model='79'/> 
c480ed
+      <signature family='6' model='86'/> 
c480ed
       <vendor name='Intel'/>
c480ed
       <feature name='3dnowprefetch'/>
c480ed
       <feature name='adx'/>
c480ed
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
c480ed
index 7718d7ca59..a5c6d9b471 100644
c480ed
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-guest.xml
c480ed
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-guest.xml
c480ed
@@ -1,6 +1,7 @@
c480ed
 <cpu mode='custom' match='exact'>
c480ed
-  <model fallback='forbid'>Skylake-Client-IBRS</model>
c480ed
+  <model fallback='forbid'>Broadwell-IBRS</model>
c480ed
   <vendor>Intel</vendor>
c480ed
+  <feature policy='require' name='vme'/>
c480ed
   <feature policy='require' name='ds'/>
c480ed
   <feature policy='require' name='acpi'/>
c480ed
   <feature policy='require' name='ss'/>
c480ed
@@ -18,15 +19,17 @@
c480ed
   <feature policy='require' name='pdcm'/>
c480ed
   <feature policy='require' name='dca'/>
c480ed
   <feature policy='require' name='osxsave'/>
c480ed
+  <feature policy='require' name='f16c'/>
c480ed
+  <feature policy='require' name='rdrand'/>
c480ed
+  <feature policy='require' name='arat'/>
c480ed
   <feature policy='require' name='tsc_adjust'/>
c480ed
   <feature policy='require' name='cmt'/>
c480ed
   <feature policy='require' name='intel-pt'/>
c480ed
   <feature policy='require' name='stibp'/>
c480ed
+  <feature policy='require' name='xsaveopt'/>
c480ed
   <feature policy='require' name='mbm_total'/>
c480ed
   <feature policy='require' name='mbm_local'/>
c480ed
   <feature policy='require' name='pdpe1gb'/>
c480ed
+  <feature policy='require' name='abm'/>
c480ed
   <feature policy='require' name='invtsc'/>
c480ed
-  <feature policy='disable' name='mpx'/>
c480ed
-  <feature policy='disable' name='xsavec'/>
c480ed
-  <feature policy='disable' name='xgetbv1'/>
c480ed
 </cpu>
c480ed
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
c480ed
index 167a9028ab..de082dbd93 100644
c480ed
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-json.xml
c480ed
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2623-v4-json.xml
c480ed
@@ -1,11 +1,14 @@
c480ed
 <cpu mode='custom' match='exact'>
c480ed
-  <model fallback='forbid'>Skylake-Client-IBRS</model>
c480ed
+  <model fallback='forbid'>Broadwell-IBRS</model>
c480ed
   <vendor>Intel</vendor>
c480ed
+  <feature policy='require' name='vme'/>
c480ed
   <feature policy='require' name='ss'/>
c480ed
+  <feature policy='require' name='f16c'/>
c480ed
+  <feature policy='require' name='rdrand'/>
c480ed
   <feature policy='require' name='hypervisor'/>
c480ed
+  <feature policy='require' name='arat'/>
c480ed
   <feature policy='require' name='tsc_adjust'/>
c480ed
+  <feature policy='require' name='xsaveopt'/>
c480ed
   <feature policy='require' name='pdpe1gb'/>
c480ed
-  <feature policy='disable' name='mpx'/>
c480ed
-  <feature policy='disable' name='xsavec'/>
c480ed
-  <feature policy='disable' name='xgetbv1'/>
c480ed
+  <feature policy='require' name='abm'/>
c480ed
 </cpu>
c480ed
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
c480ed
index cd7e25b52a..e2999db8e9 100644
c480ed
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-guest.xml
c480ed
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-guest.xml
c480ed
@@ -1,6 +1,7 @@
c480ed
 <cpu mode='custom' match='exact'>
c480ed
-  <model fallback='forbid'>Skylake-Client</model>
c480ed
+  <model fallback='forbid'>Broadwell</model>
c480ed
   <vendor>Intel</vendor>
c480ed
+  <feature policy='require' name='vme'/>
c480ed
   <feature policy='require' name='ds'/>
c480ed
   <feature policy='require' name='acpi'/>
c480ed
   <feature policy='require' name='ss'/>
c480ed
@@ -18,14 +19,16 @@
c480ed
   <feature policy='require' name='pdcm'/>
c480ed
   <feature policy='require' name='dca'/>
c480ed
   <feature policy='require' name='osxsave'/>
c480ed
+  <feature policy='require' name='f16c'/>
c480ed
+  <feature policy='require' name='rdrand'/>
c480ed
+  <feature policy='require' name='arat'/>
c480ed
   <feature policy='require' name='tsc_adjust'/>
c480ed
   <feature policy='require' name='cmt'/>
c480ed
   <feature policy='require' name='intel-pt'/>
c480ed
+  <feature policy='require' name='xsaveopt'/>
c480ed
   <feature policy='require' name='mbm_total'/>
c480ed
   <feature policy='require' name='mbm_local'/>
c480ed
   <feature policy='require' name='pdpe1gb'/>
c480ed
+  <feature policy='require' name='abm'/>
c480ed
   <feature policy='require' name='invtsc'/>
c480ed
-  <feature policy='disable' name='mpx'/>
c480ed
-  <feature policy='disable' name='xsavec'/>
c480ed
-  <feature policy='disable' name='xgetbv1'/>
c480ed
 </cpu>
c480ed
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
c480ed
index 5dfce947b2..5b8891093a 100644
c480ed
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-json.xml
c480ed
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2630-v4-json.xml
c480ed
@@ -1,11 +1,14 @@
c480ed
 <cpu mode='custom' match='exact'>
c480ed
-  <model fallback='forbid'>Skylake-Client</model>
c480ed
+  <model fallback='forbid'>Broadwell</model>
c480ed
   <vendor>Intel</vendor>
c480ed
+  <feature policy='require' name='vme'/>
c480ed
   <feature policy='require' name='ss'/>
c480ed
+  <feature policy='require' name='f16c'/>
c480ed
+  <feature policy='require' name='rdrand'/>
c480ed
   <feature policy='require' name='hypervisor'/>
c480ed
+  <feature policy='require' name='arat'/>
c480ed
   <feature policy='require' name='tsc_adjust'/>
c480ed
+  <feature policy='require' name='xsaveopt'/>
c480ed
   <feature policy='require' name='pdpe1gb'/>
c480ed
-  <feature policy='disable' name='mpx'/>
c480ed
-  <feature policy='disable' name='xsavec'/>
c480ed
-  <feature policy='disable' name='xgetbv1'/>
c480ed
+  <feature policy='require' name='abm'/>
c480ed
 </cpu>
c480ed
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
c480ed
index cd7e25b52a..e2999db8e9 100644
c480ed
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-guest.xml
c480ed
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-guest.xml
c480ed
@@ -1,6 +1,7 @@
c480ed
 <cpu mode='custom' match='exact'>
c480ed
-  <model fallback='forbid'>Skylake-Client</model>
c480ed
+  <model fallback='forbid'>Broadwell</model>
c480ed
   <vendor>Intel</vendor>
c480ed
+  <feature policy='require' name='vme'/>
c480ed
   <feature policy='require' name='ds'/>
c480ed
   <feature policy='require' name='acpi'/>
c480ed
   <feature policy='require' name='ss'/>
c480ed
@@ -18,14 +19,16 @@
c480ed
   <feature policy='require' name='pdcm'/>
c480ed
   <feature policy='require' name='dca'/>
c480ed
   <feature policy='require' name='osxsave'/>
c480ed
+  <feature policy='require' name='f16c'/>
c480ed
+  <feature policy='require' name='rdrand'/>
c480ed
+  <feature policy='require' name='arat'/>
c480ed
   <feature policy='require' name='tsc_adjust'/>
c480ed
   <feature policy='require' name='cmt'/>
c480ed
   <feature policy='require' name='intel-pt'/>
c480ed
+  <feature policy='require' name='xsaveopt'/>
c480ed
   <feature policy='require' name='mbm_total'/>
c480ed
   <feature policy='require' name='mbm_local'/>
c480ed
   <feature policy='require' name='pdpe1gb'/>
c480ed
+  <feature policy='require' name='abm'/>
c480ed
   <feature policy='require' name='invtsc'/>
c480ed
-  <feature policy='disable' name='mpx'/>
c480ed
-  <feature policy='disable' name='xsavec'/>
c480ed
-  <feature policy='disable' name='xgetbv1'/>
c480ed
 </cpu>
c480ed
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
c480ed
index 5dfce947b2..5b8891093a 100644
c480ed
--- a/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-json.xml
c480ed
+++ b/tests/cputestdata/x86_64-cpuid-Xeon-E5-2650-v4-json.xml
c480ed
@@ -1,11 +1,14 @@
c480ed
 <cpu mode='custom' match='exact'>
c480ed
-  <model fallback='forbid'>Skylake-Client</model>
c480ed
+  <model fallback='forbid'>Broadwell</model>
c480ed
   <vendor>Intel</vendor>
c480ed
+  <feature policy='require' name='vme'/>
c480ed
   <feature policy='require' name='ss'/>
c480ed
+  <feature policy='require' name='f16c'/>
c480ed
+  <feature policy='require' name='rdrand'/>
c480ed
   <feature policy='require' name='hypervisor'/>
c480ed
+  <feature policy='require' name='arat'/>
c480ed
   <feature policy='require' name='tsc_adjust'/>
c480ed
+  <feature policy='require' name='xsaveopt'/>
c480ed
   <feature policy='require' name='pdpe1gb'/>
c480ed
-  <feature policy='disable' name='mpx'/>
c480ed
-  <feature policy='disable' name='xsavec'/>
c480ed
-  <feature policy='disable' name='xgetbv1'/>
c480ed
+  <feature policy='require' name='abm'/>
c480ed
 </cpu>
c480ed
-- 
c480ed
2.22.0
c480ed