Commit d8c0ef548c for qemu.org
commit d8c0ef548c8cac5623fdb800bd7f9735a0cf0e66
Author: Zhao Liu <zhao1.liu@intel.com>
Date: Tue Mar 10 22:08:17 2026 +0800
i386/cpu: Remove unnecessary cache_info fields from builtin CPU model
In x86_cpu_get_versioned_cache_info(), higher version CPU models will
inherit cache_info from lower versions if the cache_info field is not
specified, so there is no need to repeatedly set the same cache_info.
Therefore, remove the repeated cache models from SapphireRapids-v6
(SapphireRapids-v4 has set the cache model), GraniteRapids-v5
(GraniteRapids-v3 has set) and SierraForest-v5 (SierraForest-v3 has
set).
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
Link: https://lore.kernel.org/r/20260310140819.1563084-4-zhao1.liu@intel.com
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index 08450a9142..05121c5b32 100644
--- a/target/i386/cpu.c
+++ b/target/i386/cpu.c
@@ -5264,7 +5264,6 @@ static const X86CPUDefinition builtin_x86_defs[] = {
{
.version = 6,
.note = "with cet-ss, cet-ibt, its-no",
- .cache_info = &xeon_spr_cache_info,
.props = (PropValue[]) {
{ "its-no", "on" },
{ /* end of list */ },
@@ -5446,7 +5445,6 @@ static const X86CPUDefinition builtin_x86_defs[] = {
{
.version = 5,
.note = "with cet-ss, cet-ibt, its-no",
- .cache_info = &xeon_gnr_cache_info,
.props = (PropValue[]) {
{ "its-no", "on" },
{ /* end of list */ },
@@ -5812,7 +5810,6 @@ static const X86CPUDefinition builtin_x86_defs[] = {
{
.version = 5,
.note = "with ITS_NO",
- .cache_info = &xeon_srf_cache_info,
.props = (PropValue[]) {
{ "its-no", "on" },
{ /* end of list */ },