At present, all cases using the cache model (CPUID 0x2, 0x4, 0x80000005,
0x80000006 and 0x8000001D leaves) have been verified to be able to
select either cache_info_intel or cache_info_amd based on the vendor.
Therefore, further merge cache_info_intel and cache_info_amd into a
unified cache_info in X86CPUState, and during its initialization, set
different legacy cache models based on the vendor.
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
---
target/i386/cpu.c | 150 ++++++++--------------------------------------
target/i386/cpu.h | 5 +-
2 files changed, 27 insertions(+), 128 deletions(-)
diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index 4e9ac37850c0..c1d1289ee9de 100644
--- a/target/i386/cpu.c
+++ b/target/i386/cpu.c
@@ -7484,27 +7484,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
} else if (env->enable_legacy_vendor_cache) {
caches = &legacy_intel_cache_info;
} else {
- /*
- * FIXME: Temporarily select cache info model here based on
- * vendor, and merge these 2 cache info models later.
- *
- * This condition covers the following cases (with
- * enable_legacy_vendor_cache=false):
- * - When CPU model has its own cache model and doesn't use legacy
- * cache model (legacy_model=off). Then cache_info_amd and
- * cache_info_cpuid4 are the same.
- *
- * - For v10.1 and newer machines, when CPU model uses legacy cache
- * model. Non-AMD CPUs use cache_info_cpuid4 like before and AMD
- * CPU will use cache_info_amd. But this doesn't matter for AMD
- * CPU, because this leaf encodes all-0 for AMD whatever its cache
- * model is.
- */
- if (IS_AMD_CPU(env)) {
- caches = &env->cache_info_amd;
- } else {
- caches = &env->cache_info_cpuid4;
- }
+ caches = &env->cache_info;
}
if (cpu->cache_info_passthrough) {
@@ -7523,27 +7503,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
if (env->enable_legacy_vendor_cache) {
caches = &legacy_intel_cache_info;
} else {
- /*
- * FIXME: Temporarily select cache info model here based on
- * vendor, and merge these 2 cache info models later.
- *
- * This condition covers the following cases (with
- * enable_legacy_vendor_cache=false):
- * - When CPU model has its own cache model and doesn't use legacy
- * cache model (legacy_model=off). Then cache_info_amd and
- * cache_info_cpuid4 are the same.
- *
- * - For v10.1 and newer machines, when CPU model uses legacy cache
- * model. Non-AMD CPUs use cache_info_cpuid4 like before and AMD
- * CPU will use cache_info_amd. But this doesn't matter for AMD
- * CPU, because this leaf encodes all-0 for AMD whatever its cache
- * model is.
- */
- if (IS_AMD_CPU(env)) {
- caches = &env->cache_info_amd;
- } else {
- caches = &env->cache_info_cpuid4;
- }
+ caches = &env->cache_info;
}
/* cache info: needed for Core compatibility */
@@ -7951,27 +7911,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
if (env->enable_legacy_vendor_cache) {
caches = &legacy_amd_cache_info;
} else {
- /*
- * FIXME: Temporarily select cache info model here based on
- * vendor, and merge these 2 cache info models later.
- *
- * This condition covers the following cases (with
- * enable_legacy_vendor_cache=false):
- * - When CPU model has its own cache model and doesn't uses legacy
- * cache model (legacy_model=off). Then cache_info_amd and
- * cache_info_cpuid4 are the same.
- *
- * - For v10.1 and newer machines, when CPU model uses legacy cache
- * model. AMD CPUs use cache_info_amd like before and non-AMD
- * CPU will use cache_info_cpuid4. But this doesn't matter,
- * because for Intel CPU, it will get all-0 leaf, and Zhaoxin CPU
- * will get correct cache info. Both are expected.
- */
- if (IS_AMD_CPU(env)) {
- caches = &env->cache_info_amd;
- } else {
- caches = &env->cache_info_cpuid4;
- }
+ caches = &env->cache_info;
}
if (cpu->cache_info_passthrough) {
@@ -7998,25 +7938,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
if (env->enable_legacy_vendor_cache) {
caches = &legacy_amd_cache_info;
} else {
- /*
- * FIXME: Temporarily select cache info model here based on
- * vendor, and merge these 2 cache info models later.
- *
- * This condition covers the following cases (with
- * enable_legacy_vendor_cache=false):
- * - When CPU model has its own cache model and doesn't uses legacy
- * cache model (legacy_model=off). Then cache_info_amd and
- * cache_info_cpuid4 are the same.
- *
- * - For v10.1 and newer machines, when CPU model uses legacy cache
- * model. AMD CPUs use cache_info_amd like before and non-AMD
- * CPU (Intel & Zhaoxin) will use cache_info_cpuid4 as expected.
- */
- if (IS_AMD_CPU(env)) {
- caches = &env->cache_info_amd;
- } else {
- caches = &env->cache_info_cpuid4;
- }
+ caches = &env->cache_info;
}
if (cpu->cache_info_passthrough) {
@@ -8089,22 +8011,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
*edx = 0;
}
break;
- case 0x8000001D: {
- const CPUCaches *caches;
-
- /*
- * FIXME: Temporarily select cache info model here based on
- * vendor, and merge these 2 cache info models later.
- *
- * Intel doesn't support this leaf so that Intel Guests don't
- * have this leaf. This change is harmless to Intel CPUs.
- */
- if (IS_AMD_CPU(env)) {
- caches = &env->cache_info_amd;
- } else {
- caches = &env->cache_info_cpuid4;
- }
-
+ case 0x8000001D:
*eax = 0;
if (cpu->cache_info_passthrough) {
x86_cpu_get_cache_cpuid(index, count, eax, ebx, ecx, edx);
@@ -8112,19 +8019,19 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
}
switch (count) {
case 0: /* L1 dcache info */
- encode_cache_cpuid8000001d(caches->l1d_cache,
+ encode_cache_cpuid8000001d(env->cache_info.l1d_cache,
topo_info, eax, ebx, ecx, edx);
break;
case 1: /* L1 icache info */
- encode_cache_cpuid8000001d(caches->l1i_cache,
+ encode_cache_cpuid8000001d(env->cache_info.l1i_cache,
topo_info, eax, ebx, ecx, edx);
break;
case 2: /* L2 cache info */
- encode_cache_cpuid8000001d(caches->l2_cache,
+ encode_cache_cpuid8000001d(env->cache_info.l2_cache,
topo_info, eax, ebx, ecx, edx);
break;
case 3: /* L3 cache info */
- encode_cache_cpuid8000001d(caches->l3_cache,
+ encode_cache_cpuid8000001d(env->cache_info.l3_cache,
topo_info, eax, ebx, ecx, edx);
break;
default: /* end of info */
@@ -8135,7 +8042,6 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
*edx &= CACHE_NO_INVD_SHARING | CACHE_INCLUSIVE;
}
break;
- }
case 0x8000001E:
if (cpu->core_id <= 255) {
encode_topo_cpuid8000001e(cpu, topo_info, eax, ebx, ecx, edx);
@@ -8825,46 +8731,34 @@ static bool x86_cpu_update_smp_cache_topo(MachineState *ms, X86CPU *cpu,
level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1D);
if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
- env->cache_info_cpuid4.l1d_cache->share_level = level;
- env->cache_info_amd.l1d_cache->share_level = level;
+ env->cache_info.l1d_cache->share_level = level;
} else {
machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1D,
- env->cache_info_cpuid4.l1d_cache->share_level);
- machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1D,
- env->cache_info_amd.l1d_cache->share_level);
+ env->cache_info.l1d_cache->share_level);
}
level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1I);
if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
- env->cache_info_cpuid4.l1i_cache->share_level = level;
- env->cache_info_amd.l1i_cache->share_level = level;
+ env->cache_info.l1i_cache->share_level = level;
} else {
machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1I,
- env->cache_info_cpuid4.l1i_cache->share_level);
- machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1I,
- env->cache_info_amd.l1i_cache->share_level);
+ env->cache_info.l1i_cache->share_level);
}
level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L2);
if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
- env->cache_info_cpuid4.l2_cache->share_level = level;
- env->cache_info_amd.l2_cache->share_level = level;
+ env->cache_info.l2_cache->share_level = level;
} else {
machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L2,
- env->cache_info_cpuid4.l2_cache->share_level);
- machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L2,
- env->cache_info_amd.l2_cache->share_level);
+ env->cache_info.l2_cache->share_level);
}
level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L3);
if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
- env->cache_info_cpuid4.l3_cache->share_level = level;
- env->cache_info_amd.l3_cache->share_level = level;
+ env->cache_info.l3_cache->share_level = level;
} else {
machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L3,
- env->cache_info_cpuid4.l3_cache->share_level);
- machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L3,
- env->cache_info_amd.l3_cache->share_level);
+ env->cache_info.l3_cache->share_level);
}
if (!machine_check_smp_cache(ms, errp)) {
@@ -9091,7 +8985,7 @@ static void x86_cpu_realizefn(DeviceState *dev, Error **errp)
"CPU model '%s' doesn't support legacy-cache=off", name);
return;
}
- env->cache_info_cpuid4 = env->cache_info_amd = *cache_info;
+ env->cache_info = *cache_info;
} else {
/* Build legacy cache information */
if (!cpu->consistent_cache) {
@@ -9101,8 +8995,12 @@ static void x86_cpu_realizefn(DeviceState *dev, Error **errp)
if (!cpu->vendor_cpuid_only_v2) {
env->enable_legacy_vendor_cache = true;
}
- env->cache_info_cpuid4 = legacy_intel_cache_info;
- env->cache_info_amd = legacy_amd_cache_info;
+
+ if (IS_AMD_CPU(env)) {
+ env->cache_info = legacy_amd_cache_info;
+ } else {
+ env->cache_info = legacy_intel_cache_info;
+ }
}
#ifndef CONFIG_USER_ONLY
diff --git a/target/i386/cpu.h b/target/i386/cpu.h
index 243383efd602..3f79db679888 100644
--- a/target/i386/cpu.h
+++ b/target/i386/cpu.h
@@ -2072,11 +2072,12 @@ typedef struct CPUArchState {
/* Features that were explicitly enabled/disabled */
FeatureWordArray user_features;
uint32_t cpuid_model[12];
- /* Cache information for CPUID. When legacy-cache=on, the cache data
+ /*
+ * Cache information for CPUID. When legacy-cache=on, the cache data
* on each CPUID leaf will be different, because we keep compatibility
* with old QEMU versions.
*/
- CPUCaches cache_info_cpuid4, cache_info_amd;
+ CPUCaches cache_info;
bool enable_legacy_cpuid2_cache;
bool enable_legacy_vendor_cache;
--
2.34.1
On 6/20/2025 5:27 PM, Zhao Liu wrote:
> At present, all cases using the cache model (CPUID 0x2, 0x4, 0x80000005,
> 0x80000006 and 0x8000001D leaves) have been verified to be able to
> select either cache_info_intel or cache_info_amd based on the vendor.
>
> Therefore, further merge cache_info_intel and cache_info_amd into a
> unified cache_info in X86CPUState, and during its initialization, set
> different legacy cache models based on the vendor.
>
> Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
> ---
> target/i386/cpu.c | 150 ++++++++--------------------------------------
> target/i386/cpu.h | 5 +-
> 2 files changed, 27 insertions(+), 128 deletions(-)
>
> diff --git a/target/i386/cpu.c b/target/i386/cpu.c
> index 4e9ac37850c0..c1d1289ee9de 100644
> --- a/target/i386/cpu.c
> +++ b/target/i386/cpu.c
> @@ -7484,27 +7484,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
> } else if (env->enable_legacy_vendor_cache) {
> caches = &legacy_intel_cache_info;
> } else {
> - /*
> - * FIXME: Temporarily select cache info model here based on
> - * vendor, and merge these 2 cache info models later.
> - *
> - * This condition covers the following cases (with
> - * enable_legacy_vendor_cache=false):
> - * - When CPU model has its own cache model and doesn't use legacy
> - * cache model (legacy_model=off). Then cache_info_amd and
> - * cache_info_cpuid4 are the same.
> - *
> - * - For v10.1 and newer machines, when CPU model uses legacy cache
> - * model. Non-AMD CPUs use cache_info_cpuid4 like before and AMD
> - * CPU will use cache_info_amd. But this doesn't matter for AMD
> - * CPU, because this leaf encodes all-0 for AMD whatever its cache
> - * model is.
> - */
> - if (IS_AMD_CPU(env)) {
> - caches = &env->cache_info_amd;
> - } else {
> - caches = &env->cache_info_cpuid4;
> - }
> + caches = &env->cache_info;
> }
>
> if (cpu->cache_info_passthrough) {
> @@ -7523,27 +7503,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
> if (env->enable_legacy_vendor_cache) {
> caches = &legacy_intel_cache_info;
> } else {
> - /*
> - * FIXME: Temporarily select cache info model here based on
> - * vendor, and merge these 2 cache info models later.
> - *
> - * This condition covers the following cases (with
> - * enable_legacy_vendor_cache=false):
> - * - When CPU model has its own cache model and doesn't use legacy
> - * cache model (legacy_model=off). Then cache_info_amd and
> - * cache_info_cpuid4 are the same.
> - *
> - * - For v10.1 and newer machines, when CPU model uses legacy cache
> - * model. Non-AMD CPUs use cache_info_cpuid4 like before and AMD
> - * CPU will use cache_info_amd. But this doesn't matter for AMD
> - * CPU, because this leaf encodes all-0 for AMD whatever its cache
> - * model is.
> - */
> - if (IS_AMD_CPU(env)) {
> - caches = &env->cache_info_amd;
> - } else {
> - caches = &env->cache_info_cpuid4;
> - }
> + caches = &env->cache_info;
> }
>
> /* cache info: needed for Core compatibility */
> @@ -7951,27 +7911,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
> if (env->enable_legacy_vendor_cache) {
> caches = &legacy_amd_cache_info;
> } else {
> - /*
> - * FIXME: Temporarily select cache info model here based on
> - * vendor, and merge these 2 cache info models later.
> - *
> - * This condition covers the following cases (with
> - * enable_legacy_vendor_cache=false):
> - * - When CPU model has its own cache model and doesn't uses legacy
> - * cache model (legacy_model=off). Then cache_info_amd and
> - * cache_info_cpuid4 are the same.
> - *
> - * - For v10.1 and newer machines, when CPU model uses legacy cache
> - * model. AMD CPUs use cache_info_amd like before and non-AMD
> - * CPU will use cache_info_cpuid4. But this doesn't matter,
> - * because for Intel CPU, it will get all-0 leaf, and Zhaoxin CPU
> - * will get correct cache info. Both are expected.
> - */
> - if (IS_AMD_CPU(env)) {
> - caches = &env->cache_info_amd;
> - } else {
> - caches = &env->cache_info_cpuid4;
> - }
> + caches = &env->cache_info;
> }
>
> if (cpu->cache_info_passthrough) {
> @@ -7998,25 +7938,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
> if (env->enable_legacy_vendor_cache) {
> caches = &legacy_amd_cache_info;
> } else {
> - /*
> - * FIXME: Temporarily select cache info model here based on
> - * vendor, and merge these 2 cache info models later.
> - *
> - * This condition covers the following cases (with
> - * enable_legacy_vendor_cache=false):
> - * - When CPU model has its own cache model and doesn't uses legacy
> - * cache model (legacy_model=off). Then cache_info_amd and
> - * cache_info_cpuid4 are the same.
> - *
> - * - For v10.1 and newer machines, when CPU model uses legacy cache
> - * model. AMD CPUs use cache_info_amd like before and non-AMD
> - * CPU (Intel & Zhaoxin) will use cache_info_cpuid4 as expected.
> - */
> - if (IS_AMD_CPU(env)) {
> - caches = &env->cache_info_amd;
> - } else {
> - caches = &env->cache_info_cpuid4;
> - }
> + caches = &env->cache_info;
> }
>
> if (cpu->cache_info_passthrough) {
> @@ -8089,22 +8011,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
> *edx = 0;
> }
> break;
> - case 0x8000001D: {
> - const CPUCaches *caches;
> -
> - /*
> - * FIXME: Temporarily select cache info model here based on
> - * vendor, and merge these 2 cache info models later.
> - *
> - * Intel doesn't support this leaf so that Intel Guests don't
> - * have this leaf. This change is harmless to Intel CPUs.
> - */
> - if (IS_AMD_CPU(env)) {
> - caches = &env->cache_info_amd;
> - } else {
> - caches = &env->cache_info_cpuid4;
> - }
> -
> + case 0x8000001D:
> *eax = 0;
> if (cpu->cache_info_passthrough) {
> x86_cpu_get_cache_cpuid(index, count, eax, ebx, ecx, edx);
> @@ -8112,19 +8019,19 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
> }
> switch (count) {
> case 0: /* L1 dcache info */
> - encode_cache_cpuid8000001d(caches->l1d_cache,
> + encode_cache_cpuid8000001d(env->cache_info.l1d_cache,
> topo_info, eax, ebx, ecx, edx);
> break;
> case 1: /* L1 icache info */
> - encode_cache_cpuid8000001d(caches->l1i_cache,
> + encode_cache_cpuid8000001d(env->cache_info.l1i_cache,
> topo_info, eax, ebx, ecx, edx);
> break;
> case 2: /* L2 cache info */
> - encode_cache_cpuid8000001d(caches->l2_cache,
> + encode_cache_cpuid8000001d(env->cache_info.l2_cache,
> topo_info, eax, ebx, ecx, edx);
> break;
> case 3: /* L3 cache info */
> - encode_cache_cpuid8000001d(caches->l3_cache,
> + encode_cache_cpuid8000001d(env->cache_info.l3_cache,
> topo_info, eax, ebx, ecx, edx);
> break;
> default: /* end of info */
> @@ -8135,7 +8042,6 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
> *edx &= CACHE_NO_INVD_SHARING | CACHE_INCLUSIVE;
> }
> break;
> - }
> case 0x8000001E:
> if (cpu->core_id <= 255) {
> encode_topo_cpuid8000001e(cpu, topo_info, eax, ebx, ecx, edx);
> @@ -8825,46 +8731,34 @@ static bool x86_cpu_update_smp_cache_topo(MachineState *ms, X86CPU *cpu,
>
> level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1D);
> if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
> - env->cache_info_cpuid4.l1d_cache->share_level = level;
> - env->cache_info_amd.l1d_cache->share_level = level;
> + env->cache_info.l1d_cache->share_level = level;
> } else {
> machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1D,
> - env->cache_info_cpuid4.l1d_cache->share_level);
> - machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1D,
> - env->cache_info_amd.l1d_cache->share_level);
> + env->cache_info.l1d_cache->share_level);
> }
>
> level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1I);
> if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
> - env->cache_info_cpuid4.l1i_cache->share_level = level;
> - env->cache_info_amd.l1i_cache->share_level = level;
> + env->cache_info.l1i_cache->share_level = level;
> } else {
> machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1I,
> - env->cache_info_cpuid4.l1i_cache->share_level);
> - machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L1I,
> - env->cache_info_amd.l1i_cache->share_level);
> + env->cache_info.l1i_cache->share_level);
> }
>
> level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L2);
> if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
> - env->cache_info_cpuid4.l2_cache->share_level = level;
> - env->cache_info_amd.l2_cache->share_level = level;
> + env->cache_info.l2_cache->share_level = level;
> } else {
> machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L2,
> - env->cache_info_cpuid4.l2_cache->share_level);
> - machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L2,
> - env->cache_info_amd.l2_cache->share_level);
> + env->cache_info.l2_cache->share_level);
> }
>
> level = machine_get_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L3);
> if (level != CPU_TOPOLOGY_LEVEL_DEFAULT) {
> - env->cache_info_cpuid4.l3_cache->share_level = level;
> - env->cache_info_amd.l3_cache->share_level = level;
> + env->cache_info.l3_cache->share_level = level;
> } else {
> machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L3,
> - env->cache_info_cpuid4.l3_cache->share_level);
> - machine_set_cache_topo_level(ms, CACHE_LEVEL_AND_TYPE_L3,
> - env->cache_info_amd.l3_cache->share_level);
> + env->cache_info.l3_cache->share_level);
> }
>
> if (!machine_check_smp_cache(ms, errp)) {
> @@ -9091,7 +8985,7 @@ static void x86_cpu_realizefn(DeviceState *dev, Error **errp)
> "CPU model '%s' doesn't support legacy-cache=off", name);
> return;
> }
> - env->cache_info_cpuid4 = env->cache_info_amd = *cache_info;
> + env->cache_info = *cache_info;
> } else {
> /* Build legacy cache information */
> if (!cpu->consistent_cache) {
> @@ -9101,8 +8995,12 @@ static void x86_cpu_realizefn(DeviceState *dev, Error **errp)
> if (!cpu->vendor_cpuid_only_v2) {
> env->enable_legacy_vendor_cache = true;
> }
> - env->cache_info_cpuid4 = legacy_intel_cache_info;
> - env->cache_info_amd = legacy_amd_cache_info;
> +
> + if (IS_AMD_CPU(env)) {
> + env->cache_info = legacy_amd_cache_info;
> + } else {
> + env->cache_info = legacy_intel_cache_info;
> + }
> }
>
> #ifndef CONFIG_USER_ONLY
> diff --git a/target/i386/cpu.h b/target/i386/cpu.h
> index 243383efd602..3f79db679888 100644
> --- a/target/i386/cpu.h
> +++ b/target/i386/cpu.h
> @@ -2072,11 +2072,12 @@ typedef struct CPUArchState {
> /* Features that were explicitly enabled/disabled */
> FeatureWordArray user_features;
> uint32_t cpuid_model[12];
> - /* Cache information for CPUID. When legacy-cache=on, the cache data
> + /*
> + * Cache information for CPUID. When legacy-cache=on, the cache data
> * on each CPUID leaf will be different, because we keep compatibility
> * with old QEMU versions.
> */
> - CPUCaches cache_info_cpuid4, cache_info_amd;
> + CPUCaches cache_info;
> bool enable_legacy_cpuid2_cache;
> bool enable_legacy_vendor_cache;
>
Nice clean-up patch series. Thanks.
Reviewed-by: Dapeng Mi <dapeng1.mi@linux.intel.com>
© 2016 - 2025 Red Hat, Inc.