* [PATCH v19 0/8] Specifying cache topology on ARM
@ 2026-03-11 16:06 Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 1/8] target/arm/tcg: increase cache level for cpu=max Alireza Sanaee via qemu development
` (7 more replies)
0 siblings, 8 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
Specifying the cache layout in virtual machines is useful for applications and
operating systems to fetch accurate information about the cache structure and
make appropriate adjustments. Enforcing correct sharing information can lead to
better optimizations. Patches that allow for an interface to express caches was
landed in the prior cycles. This patchset uses the interface as a foundation.
Thus, the device tree and ACPI/PPTT table, and device tree are populated based
on user-provided information and CPU topology.
Example:
+----------------+ +----------------+
| Socket 0 | | Socket 1 |
| (L3 Cache) | | (L3 Cache) |
+--------+-------+ +--------+-------+
| |
+--------+--------+ +--------+--------+
| Cluster 0 | | Cluster 0 |
| (L2 Cache) | | (L2 Cache) |
+--------+--------+ +--------+--------+
| |
+--------+--------+ +--------+--------+ +--------+--------+ +--------+----+
| Core 0 | | Core 1 | | Core 0 | | Core 1 |
| (L1i, L1d) | | (L1i, L1d) | | (L1i, L1d) | | (L1i, L1d)|
+--------+--------+ +--------+--------+ +--------+--------+ +--------+----+
| | | |
+--------+ +--------+ +--------+ +--------+
|Thread 0| |Thread 1| |Thread 1| |Thread 0|
+--------+ +--------+ +--------+ +--------+
|Thread 1| |Thread 0| |Thread 0| |Thread 1|
+--------+ +--------+ +--------+ +--------+
The following command will represent the system relying on **ACPI PPTT tables**.
./qemu-system-aarch64 \
-machine virt,smp-cache.0.cache=l1i,smp-cache.0.topology=core,smp-cache.1.cache=l1d,smp-cache.1.topology=core,smp-cache.2.cache=l2,smp-cache.2.topology=cluster,smp-cache.3.cache=l3,smp-cache.3.topology=socket \
-cpu max \
-m 2048 \
-smp sockets=2,clusters=1,cores=2,threads=2 \
-kernel ./Image.gz \
-append "console=ttyAMA0 root=/dev/ram rdinit=/init acpi=force" \
-initrd rootfs.cpio.gz \
-bios ./edk2-aarch64-code.fd \
-nographic
The following command will represent the system relying on **the device tree**.
./qemu-system-aarch64 \
-machine virt,acpi=off,smp-cache.0.cache=l1i,smp-cache.0.topology=core,smp-cache.1.cache=l1d,smp-cache.1.topology=core,smp-cache.2.cache=l2,smp-cache.2.topology=cluster,smp-cache.3.cache=l3,smp-cache.3.topology=socket \
-cpu max \
-m 2048 \
-smp sockets=2,clusters=1,cores=2,threads=2 \
-kernel ./Image.gz \
-append "console=ttyAMA0 root=/dev/ram rdinit=/init acpi=off" \
-initrd rootfs.cpio.gz \
-nographic
Failure cases:
1) There are scenarios where caches exist in systems' registers but left
unspecified by users. In this case qemu returns failure.
2) SMT threads cannot share caches which is not very common. More
discussions here [1].
Currently only three levels of caches are supported to be specified from the
command line. However, increasing the value does not require significant
changes. Further, this patch assumes l2 and l3 unified caches and does not
allow l(2/3)(i/d). The level terminology is thread/core/cluster/socket.
Hierarchy assumed in this patch: Socket level = Cluster level + 1 = Core level
+ 2 = Thread level + 3;
Possible future enhancements:
1) Separated data and instruction cache at L2 and L3.
2) Additional cache controls. e.g. size of L3 may not want to just match the
underlying system, because only some of the associated host CPUs may be bound
to this VM.
[1] https://lore.kernel.org/devicetree-spec/20250203120527.3534-1-alireza.sanaee@huawei.com/
Change Log:
v18->v19:
* Rebased to 1fd5ff9d76d23ab23a68419cbc76d5ee33e8b455.
* Addressed Gustavo's comments. Thanks for the review.
* v18: https://lore.kernel.org/qemu-devel/20260220141128.789-1-alireza.sanaee@huawei.com/
v17->v18:
* Rebased to 07f97d5da04a9f97e273de85c76f5017d8135a6e.
* Addressed Mst's comments regarding documenting build_cache_node function. Thanks for the review.
* v17: https://lore.kernel.org/all/20260106155828.643-1-alireza.sanaee@huawei.com/
v16->v17:
* Rebased to 0fc482b73d8e085d1375b4e17b0647fd2e6fe8f0.
* Addressed Gustavo's comments. Thanks for the review.
* v16: https://lore.kernel.org/all/20250827142152.206-1-alireza.sanaee@huawei.com/
v15->v16:
* Rebase to e771ba98de25c9f43959f79fc7099cf7fbba44cce.
* v15: https://lore.kernel.org/qemu-devel/20250812122829.204-1-alireza.sanaee@huawei.com/
v14->v15:
* Introduced a separate patch for loongarch64 build_pptt function.
* Made sure loongarch64 tests pass.
* Downgraded to V2 for ACPI PPTT. Removed PPTT IDs as was not necessary.
* Removed dependency as it's been merged in the recent cycle.
-- 20250604115233.1234-1-alireza.sanaee@huawei.com
* Fixed styling issues and removed irrelevant changes.
* Moved cache headers to core/cpu.h to be used in both acpi and virt.
* v14: https://lore.kernel.org/qemu-devel/20250707121908.155-1-alireza.sanaee@huawei.com/
# Thanks to Jonathan and Zhao for their comments.
v13->v14:
* Rebased on latest staging.
* Made some naming changes to machine-smp.c, addd docs added to the
same file.
v12->v13:
* Applied comments from Zhao.
* Introduced a new patch for machine specific cache topology functions.
* Base: bc98ffdc7577e55ab8373c579c28fe24d600c40f.
v11->v12:
* Patch #4 couldn't not merge properly as the main file diverged. Now it is fixed (hopefully).
* Loonarch build_pptt function updated.
* Rebased on 09be8a511a2e278b45729d7b065d30c68dd699d0.
v10->v11:
* Fix some coding style issues.
* Rename some variables.
v9->v10:
* PPTT rev down to 2.
v8->v9:
* rebase to 10
* Fixed a bug in device-tree generation related to a scenario when
caches are shared at core in higher levels than 1.
v7->v8:
* rebase: Merge tag 'pull-nbd-2024-08-26' of https://repo.or.cz/qemu/ericb into staging
* I mis-included a file in patch #4 and I removed it in this one.
v6->v7:
* Intel stuff got pulled up, so rebase.
* added some discussions on device tree.
v5->v6:
* Minor bug fix.
* rebase based on new Intel patchset.
- https://lore.kernel.org/qemu-devel/20250110145115.1574345-1-zhao1.liu@intel.com/
v4->v5:
* Added Reviewed-by tags.
* Applied some comments.
v3->v4:
* Device tree added.
Alireza Sanaee (8):
target/arm/tcg: increase cache level for cpu=max
hw/core/machine: topology functions capabilities added
hw/arm/virt: add cache hierarchy to device tree
bios-tables-test: prepare to change ARM ACPI virt PPTT
acpi: Add parameters to pass cache descriptions to ACPI build_pptt()
hw/acpi: add cache hierarchy to pptt table
tests/qtest/bios-table-test: testing new ARM ACPI PPTT topology
Update the ACPI tables based on new aml-build.c
hw/acpi/aml-build.c | 203 ++++++++++++-
hw/arm/virt-acpi-build.c | 8 +-
hw/arm/virt.c | 335 ++++++++++++++++++++-
hw/core/machine-smp.c | 52 ++++
hw/loongarch/virt-acpi-build.c | 4 +-
include/hw/acpi/aml-build.h | 4 +-
include/hw/acpi/cpu.h | 10 +
include/hw/arm/virt.h | 17 +-
include/hw/core/boards.h | 5 +
include/hw/core/cpu.h | 12 +
target/arm/tcg/cpu64.c | 10 +
tests/data/acpi/aarch64/virt/PPTT.topology | Bin 356 -> 516 bytes
tests/qtest/bios-tables-test.c | 4 +
13 files changed, 649 insertions(+), 15 deletions(-)
base-commit: 1fd5ff9d76d23ab23a68419cbc76d5ee33e8b455
--
2.43.0
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v19 1/8] target/arm/tcg: increase cache level for cpu=max
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 2/8] hw/core/machine: topology functions capabilities added Alireza Sanaee via qemu development
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
This patch addresses cache description in the `aarch64_max_tcg_initfn`
function for cpu=max. It introduces three levels of caches and modifies
the cache description registers accordingly.
Reviewed-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Change Log:
v18->v19:
No changes to this patch.
target/arm/tcg/cpu64.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/target/arm/tcg/cpu64.c b/target/arm/tcg/cpu64.c
index 84857fb706..649d854a65 100644
--- a/target/arm/tcg/cpu64.c
+++ b/target/arm/tcg/cpu64.c
@@ -1167,6 +1167,16 @@ void aarch64_max_tcg_initfn(Object *obj)
uint64_t t;
uint32_t u;
+ SET_IDREG(isar, CLIDR, 0x8200123);
+ /* 64KB L1 dcache */
+ cpu->ccsidr[0] = make_ccsidr(CCSIDR_FORMAT_LEGACY, 4, 64, 64 * KiB, 7);
+ /* 64KB L1 icache */
+ cpu->ccsidr[1] = make_ccsidr(CCSIDR_FORMAT_LEGACY, 4, 64, 64 * KiB, 2);
+ /* 1MB L2 unified cache */
+ cpu->ccsidr[2] = make_ccsidr(CCSIDR_FORMAT_LEGACY, 8, 64, 1 * MiB, 7);
+ /* 2MB L3 unified cache */
+ cpu->ccsidr[4] = make_ccsidr(CCSIDR_FORMAT_LEGACY, 8, 64, 2 * MiB, 7);
+
/*
* Unset ARM_FEATURE_BACKCOMPAT_CNTFRQ, which we would otherwise default
* to because we started with aarch64_a57_initfn(). A 'max' CPU might
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v19 2/8] hw/core/machine: topology functions capabilities added
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 1/8] target/arm/tcg: increase cache level for cpu=max Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 3/8] hw/arm/virt: add cache hierarchy to device tree Alireza Sanaee via qemu development
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
Add two functions one of which finds the lowest cache level defined in
the cache description input, and the other checks if a given cache
topology is defined at a particular cache level
Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Thanks for the review Gustavo.
Change log:
v18->v19:
Fixing the naming of the parameters.
hw/core/machine-smp.c | 52 ++++++++++++++++++++++++++++++++++++++++
include/hw/core/boards.h | 5 ++++
2 files changed, 57 insertions(+)
diff --git a/hw/core/machine-smp.c b/hw/core/machine-smp.c
index 189c70015f..bef04aa2d7 100644
--- a/hw/core/machine-smp.c
+++ b/hw/core/machine-smp.c
@@ -406,3 +406,55 @@ bool machine_check_smp_cache(const MachineState *ms, Error **errp)
return true;
}
+
+/*
+ * This function assumes L3 and L2 have unified cache and L1 is split L1d and
+ * L1i.
+ */
+bool machine_find_lowest_level_cache_at_topo_level(const MachineState *ms,
+ int *lowest_cache_level,
+ CpuTopologyLevel topo_level)
+{
+ enum CacheLevelAndType cache_level;
+ enum CpuTopologyLevel t;
+
+ for (cache_level = CACHE_LEVEL_AND_TYPE_L1D;
+ cache_level < CACHE_LEVEL_AND_TYPE__MAX; cache_level++) {
+ t = machine_get_cache_topo_level(ms, cache_level);
+ if (t == topo_level) {
+ /* Assume L1 is split into L1d and L1i caches. */
+ if (cache_level == CACHE_LEVEL_AND_TYPE_L1D ||
+ cache_level == CACHE_LEVEL_AND_TYPE_L1I) {
+ *lowest_cache_level = 1; /* L1 */
+ } else {
+ /* Assume the other caches are unified. */
+ *lowest_cache_level = cache_level;
+ }
+
+ return true;
+ }
+ }
+
+ return false;
+}
+
+/*
+ * Check if there are caches defined at a particular level. It supports only
+ * L1, L2 and L3 caches, but this can be extended to more levels as needed.
+ *
+ * Return True on success, False otherwise.
+ */
+bool machine_defines_cache_at_topo_level(const MachineState *ms,
+ CpuTopologyLevel topology)
+{
+ enum CacheLevelAndType cache_level;
+
+ for (cache_level = CACHE_LEVEL_AND_TYPE_L1D;
+ cache_level < CACHE_LEVEL_AND_TYPE__MAX; cache_level++) {
+ if (machine_get_cache_topo_level(ms, cache_level) == topology) {
+ return true;
+ }
+ }
+
+ return false;
+}
diff --git a/include/hw/core/boards.h b/include/hw/core/boards.h
index f85f31bd90..eaf78e1886 100644
--- a/include/hw/core/boards.h
+++ b/include/hw/core/boards.h
@@ -60,6 +60,11 @@ void machine_set_cache_topo_level(MachineState *ms, CacheLevelAndType cache,
CpuTopologyLevel level);
bool machine_check_smp_cache(const MachineState *ms, Error **errp);
void machine_memory_devices_init(MachineState *ms, hwaddr base, uint64_t size);
+bool machine_defines_cache_at_topo_level(const MachineState *ms,
+ CpuTopologyLevel topology);
+bool machine_find_lowest_level_cache_at_topo_level(const MachineState *ms,
+ int *lowest_cache_level,
+ CpuTopologyLevel topo_level);
/**
* machine_class_allow_dynamic_sysbus_dev: Add type to list of valid devices
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v19 3/8] hw/arm/virt: add cache hierarchy to device tree
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 1/8] target/arm/tcg: increase cache level for cpu=max Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 2/8] hw/core/machine: topology functions capabilities added Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 4/8] bios-tables-test: prepare to change ARM ACPI virt PPTT Alireza Sanaee via qemu development
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
Specify which level (core/cluster/socket) caches found at in the CPU
topology. Updating cache topology to device tree (spec v0.4).
Example:
For example, 2 sockets (packages), and 2 clusters, 4 cores and 2 threads
created, in aggregate 2*2*4*2 logical cores. In the smp-cache object,
cores will have L1d and L1i. However, extending this is not difficult.
The clusters will share a unified L2 level cache, and finally sockets
will share L3. In this patch, threads will share L1 caches by default,
but this can be adjusted if case required.
Only three levels of caches are supported. The patch does not
allow partial declaration of caches. In other words, the topology level
of every cache must be specified if that of any level is.
./qemu-system-aarch64 \
-machine virt,\
smp-cache.0.cache=l1i,smp-cache.0.topology=core,\
smp-cache.1.cache=l1d,smp-cache.1.topology=core,\
smp-cache.2.cache=l2,smp-cache.2.topology=cluster,\
smp-cache.3.cache=l3,smp-cache.3.topology=socket\
-cpu max \
-m 2048 \
-smp sockets=2,clusters=2,cores=4,threads=1 \
-kernel ./Image.gz \
-append "console=ttyAMA0 root=/dev/ram rdinit=/init acpi=force" \
-initrd rootfs.cpio.gz \
-bios ./edk2-aarch64-code.fd \
-nographic
For instance, following device tree will be generated for a scenario
where we have 2 sockets, 2 clusters, 2 cores and 2 threads, in total 16
PEs. L1i and L1d are private to each thread, and L2 and L3 are shared at
socket level as an example.
Limitation: SMT cores cannot share L1 cache for now. This
problem does not exist in PPTT tables.
Co-developed-by: Jonathan Cameron <jonathan.cameron@huawei.com>
Signed-off-by: Jonathan Cameron <jonathan.cameron@huawei.com>
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Thanks for the review Gustavo.
Change log:
v18->v19:
- Fix comments. Adding more details to the commit message.
- Removed the extra check on existence of cache at the topology level
in add_cpu_cache_hierarchy function since it is already checked
in fdt_add_cpu_nodes function.
- Fixed typos.
hw/arm/virt.c | 335 +++++++++++++++++++++++++++++++++++++++++-
include/hw/arm/virt.h | 17 ++-
include/hw/core/cpu.h | 12 ++
3 files changed, 362 insertions(+), 2 deletions(-)
diff --git a/hw/arm/virt.c b/hw/arm/virt.c
index 7456614d05..87fb31ec6b 100644
--- a/hw/arm/virt.c
+++ b/hw/arm/virt.c
@@ -91,6 +91,7 @@
#include "hw/virtio/virtio-md-pci.h"
#include "hw/virtio/virtio-iommu.h"
#include "hw/char/pl011.h"
+#include "hw/core/cpu.h"
#include "hw/cxl/cxl.h"
#include "hw/cxl/cxl_host.h"
#include "qemu/guest-random.h"
@@ -281,6 +282,86 @@ static bool ns_el2_virt_timer_present(void)
arm_feature(env, ARM_FEATURE_EL2) && cpu_isar_feature(aa64_vh, cpu);
}
+void set_cpu_cache(CPUCoreCaches *cpu_cache, enum CacheType cache_type,
+ int cache_level, bool is_i_cache0)
+{
+ int bank_index = ((cache_level - 1) * 2) | is_i_cache0;
+ ARMCPU *armcpu = ARM_CPU(qemu_get_cpu(0));
+ bool ccidx = cpu_isar_feature(any_ccidx, armcpu);
+
+ if (ccidx) {
+ *cpu_cache = (CPUCoreCaches){
+ .linesize = 1 << (FIELD_EX64(armcpu->ccsidr[bank_index], CCSIDR_EL1,
+ CCIDX_LINESIZE) + 4),
+ .associativity = FIELD_EX64(armcpu->ccsidr[bank_index], CCSIDR_EL1,
+ CCIDX_ASSOCIATIVITY) + 1,
+ .sets = FIELD_EX64(armcpu->ccsidr[bank_index], CCSIDR_EL1,
+ CCIDX_NUMSETS) + 1,
+ };
+ } else {
+ *cpu_cache = (CPUCoreCaches){
+ .linesize = 1 << (FIELD_EX64(armcpu->ccsidr[bank_index], CCSIDR_EL1,
+ LINESIZE) + 4),
+ .associativity = FIELD_EX64(armcpu->ccsidr[bank_index], CCSIDR_EL1,
+ ASSOCIATIVITY) + 1,
+ .sets =
+ FIELD_EX64(armcpu->ccsidr[bank_index], CCSIDR_EL1, NUMSETS) + 1,
+ };
+ }
+ cpu_cache->type = cache_type;
+ cpu_cache->level = cache_level;
+ cpu_cache->size = cpu_cache->associativity *
+ cpu_cache->sets *
+ cpu_cache->linesize;
+
+ return;
+}
+
+unsigned int virt_get_caches(const VirtMachineState *vms, CPUCoreCaches *caches)
+{
+ int num_cache = 0;
+ ARMCPU *armcpu = ARM_CPU(qemu_get_cpu(0)); /* assume homogeneous CPUs */
+ ARMISARegisters *isar = &armcpu->isar;
+ uint32_t clidr = GET_IDREG(isar, CLIDR);
+
+ for (int cache_level = 1; cache_level <= CLIDR_CTYPE_MAX_CACHE_LEVEL;
+ cache_level++) {
+ uint8_t ctype =
+ (clidr >> (3 * (cache_level - 1))) & CLIDR_CTYPE_MAX_CACHE_LEVEL;
+
+ if (ctype == CLIDR_CTYPE_NO_CACHE) {
+ /*
+ * If a "No cache" cache type is found it means no manageable caches
+ * exist at further-out levels of hierarchy, so ignore them.
+ */
+ break;
+ } else if (ctype == CLIDR_CTYPE_SEPARATE_I_D_CACHES) {
+ /*
+ * Create separate D and I caches. D-cache is stored first.
+ */
+ enum CacheType cache_type;
+ for (cache_type = DATA_CACHE; cache_type <= INSTRUCTION_CACHE;
+ cache_type++) {
+ set_cpu_cache(&caches[num_cache++], cache_type, cache_level,
+ cache_type == INSTRUCTION_CACHE ? true : false);
+ }
+ } else if (ctype == CLIDR_CTYPE_UNIFIED_CACHE) {
+ set_cpu_cache(&caches[num_cache++], UNIFIED_CACHE, cache_level,
+ false);
+ } else if (ctype == CLIDR_CTYPE_D_CACHE) {
+ set_cpu_cache(&caches[num_cache++], DATA_CACHE, cache_level, false);
+ } else if (ctype == CLIDR_CTYPE_I_CACHE) {
+ set_cpu_cache(&caches[num_cache++], INSTRUCTION_CACHE, cache_level,
+ true);
+ } else {
+ error_setg(&error_abort, "Unrecognized cache type");
+ return 0;
+ }
+ }
+
+ return num_cache;
+}
+
static void create_fdt(VirtMachineState *vms)
{
MachineState *ms = MACHINE(vms);
@@ -431,13 +512,124 @@ static void fdt_add_timer_nodes(const VirtMachineState *vms)
}
}
+static void add_cache_node(void *fdt, char *nodepath, CPUCoreCaches cache,
+ uint32_t *next_level)
+{
+ /* Assume L2/3 are unified caches. */
+
+ uint32_t phandle;
+
+ qemu_fdt_add_path(fdt, nodepath);
+ phandle = qemu_fdt_alloc_phandle(fdt);
+ qemu_fdt_setprop_cell(fdt, nodepath, "phandle", phandle);
+ qemu_fdt_setprop_cell(fdt, nodepath, "cache-level", cache.level);
+ qemu_fdt_setprop_cell(fdt, nodepath, "cache-size", cache.size);
+ qemu_fdt_setprop_cell(fdt, nodepath, "cache-block-size", cache.linesize);
+ qemu_fdt_setprop_cell(fdt, nodepath, "cache-sets", cache.sets);
+ qemu_fdt_setprop(fdt, nodepath, "cache-unified", NULL, 0);
+ qemu_fdt_setprop_string(fdt, nodepath, "compatible", "cache");
+ if (cache.level != 3) {
+ /* top level cache doesn't have next-level-cache property */
+ qemu_fdt_setprop_cell(fdt, nodepath, "next-level-cache", *next_level);
+ }
+
+ *next_level = phandle;
+}
+
+static bool add_cpu_cache_hierarchy(void *fdt, CPUCoreCaches* cache,
+ uint32_t cache_cnt,
+ uint32_t top_level,
+ uint32_t bottom_level,
+ uint32_t cpu_id,
+ uint32_t *next_level) {
+ bool found_cache = false;
+
+ for (int level = top_level; level >= bottom_level; level--) {
+ for (int i = 0; i < cache_cnt; i++) {
+ char *nodepath;
+
+ if (i != level) {
+ continue;
+ }
+
+ nodepath = g_strdup_printf("/cpus/cpu@%d/l%d-cache",
+ cpu_id, level);
+ add_cache_node(fdt, nodepath, cache[i], next_level);
+ found_cache = true;
+ g_free(nodepath);
+
+ }
+ }
+
+ return found_cache;
+}
+
+static void set_cache_properties(void *fdt, const char *nodename,
+ const char *prefix, CPUCoreCaches cache)
+{
+ char prop_name[64];
+
+ snprintf(prop_name, sizeof(prop_name), "%s-block-size", prefix);
+ qemu_fdt_setprop_cell(fdt, nodename, prop_name, cache.linesize);
+
+ snprintf(prop_name, sizeof(prop_name), "%s-size", prefix);
+ qemu_fdt_setprop_cell(fdt, nodename, prop_name, cache.size);
+
+ snprintf(prop_name, sizeof(prop_name), "%s-sets", prefix);
+ qemu_fdt_setprop_cell(fdt, nodename, prop_name, cache.sets);
+}
+
+static bool partial_cache_description(const MachineState *ms, int num_caches)
+{
+ assert(num_caches - 1 < CACHE_LEVEL_AND_TYPE__MAX);
+ enum CpuTopologyLevel topo_level;
+ enum CacheLevelAndType cache_level;
+
+ for (cache_level = 0; cache_level < num_caches; cache_level++) {
+ topo_level = machine_get_cache_topo_level(ms, cache_level);
+ if (topo_level == CPU_TOPOLOGY_LEVEL_DEFAULT) {
+ /* No topology level described for this cache level. */
+ return true;
+ }
+ }
+
+ return false;
+}
+
static void fdt_add_cpu_nodes(const VirtMachineState *vms)
{
int cpu;
int addr_cells = 1;
const MachineState *ms = MACHINE(vms);
+ const MachineClass *mc = MACHINE_GET_CLASS(ms);
const VirtMachineClass *vmc = VIRT_MACHINE_GET_CLASS(vms);
int smp_cpus = ms->smp.cpus;
+ int socket_id, cluster_id, core_id;
+ uint32_t next_level = 0;
+ uint32_t socket_offset = 0;
+ uint32_t cluster_offset = 0;
+ uint32_t core_offset = 0;
+ int last_socket = -1;
+ int last_cluster = -1;
+ int last_core = -1;
+ int top_node = 3;
+ int top_cluster = 3;
+ int top_core = 3;
+ int bottom_node = 3;
+ int bottom_cluster = 3;
+ int bottom_core = 3;
+ unsigned int num_cache;
+ CPUCoreCaches caches[CPU_MAX_CACHES];
+ bool cache_created = false;
+ bool cache_at_topo_level;
+
+ num_cache = virt_get_caches(vms, caches);
+
+ if (mc->smp_props.has_caches &&
+ partial_cache_description(ms, num_cache)) {
+ error_setg(&error_fatal, "Missing cache description");
+ return;
+ }
/*
* See Linux Documentation/devicetree/bindings/arm/cpus.yaml
@@ -466,9 +658,14 @@ static void fdt_add_cpu_nodes(const VirtMachineState *vms)
qemu_fdt_setprop_cell(ms->fdt, "/cpus", "#size-cells", 0x0);
for (cpu = smp_cpus - 1; cpu >= 0; cpu--) {
+ socket_id = cpu / (ms->smp.clusters * ms->smp.cores * ms->smp.threads);
+ cluster_id = cpu / (ms->smp.cores * ms->smp.threads) % ms->smp.clusters;
+ core_id = cpu / ms->smp.threads % ms->smp.cores;
+
char *nodename = g_strdup_printf("/cpus/cpu@%d", cpu);
ARMCPU *armcpu = ARM_CPU(qemu_get_cpu(cpu));
CPUState *cs = CPU(armcpu);
+ const char *prefix = NULL;
qemu_fdt_add_subnode(ms->fdt, nodename);
qemu_fdt_setprop_string(ms->fdt, nodename, "device_type", "cpu");
@@ -498,6 +695,136 @@ static void fdt_add_cpu_nodes(const VirtMachineState *vms)
qemu_fdt_alloc_phandle(ms->fdt));
}
+ if (!vmc->no_cpu_topology && num_cache) {
+ for (uint8_t i = 0; i < num_cache; i++) {
+ /* Only level 1 in the CPU entry. */
+ if (caches[i].level > 1) {
+ continue;
+ }
+
+ if (caches[i].type == INSTRUCTION_CACHE) {
+ prefix = "i-cache";
+ } else if (caches[i].type == DATA_CACHE) {
+ prefix = "d-cache";
+ } else if (caches[i].type == UNIFIED_CACHE) {
+ error_setg(&error_fatal,
+ "Unified type is not implemented at level %d",
+ caches[i].level);
+ return;
+ } else {
+ error_setg(&error_fatal, "Undefined cache type");
+ return;
+ }
+
+ set_cache_properties(ms->fdt, nodename, prefix, caches[i]);
+ }
+ }
+
+ if (socket_id != last_socket) {
+ bottom_node = top_node;
+ /* This assumes socket as the highest topological level. */
+ socket_offset = 0;
+ cluster_offset = 0;
+ cache_at_topo_level =
+ machine_find_lowest_level_cache_at_topo_level(ms,
+ &bottom_node,
+ CPU_TOPOLOGY_LEVEL_SOCKET);
+ if (cache_at_topo_level) {
+ if (bottom_node == 1 && !virt_is_acpi_enabled(vms))
+ error_setg(
+ &error_fatal,
+ "Cannot share L1 at socket_id %d."
+ "DT limitation on sharing at cache level = 1",
+ socket_id);
+
+ cache_created = add_cpu_cache_hierarchy(ms->fdt, caches,
+ num_cache,
+ top_node,
+ bottom_node, cpu,
+ &socket_offset);
+
+ if (!cache_created) {
+ error_setg(&error_fatal,
+ "Socket: No caches at levels %d-%d",
+ top_node, bottom_node);
+ return;
+ }
+
+ top_cluster = bottom_node - 1;
+ }
+
+ last_socket = socket_id;
+ }
+
+ if (cluster_id != last_cluster) {
+ bottom_cluster = top_cluster;
+ cluster_offset = socket_offset;
+ core_offset = 0;
+ cache_at_topo_level =
+ machine_find_lowest_level_cache_at_topo_level(ms,
+ &bottom_cluster,
+ CPU_TOPOLOGY_LEVEL_CLUSTER);
+ if (cache_at_topo_level) {
+ cache_created = add_cpu_cache_hierarchy(ms->fdt, caches,
+ num_cache,
+ top_cluster,
+ bottom_cluster, cpu,
+ &cluster_offset);
+ if (bottom_cluster == 1 && !virt_is_acpi_enabled(vms)) {
+ error_setg(&error_fatal,
+ "Cannot share L1 at socket_id %d, cluster_id %d. "
+ "DT limitation on sharing at cache level = 1.",
+ socket_id, cluster_id);
+ }
+
+ if (!cache_created) {
+ error_setg(&error_fatal,
+ "Cluster: No caches at levels %d-%d.",
+ top_cluster, bottom_cluster);
+ return;
+ }
+
+ top_core = bottom_cluster - 1;
+ } else if (top_cluster == bottom_node - 1) {
+ top_core = bottom_node - 1;
+ }
+
+ last_cluster = cluster_id;
+ }
+
+ if (core_id != last_core) {
+ bottom_core = top_core;
+ core_offset = cluster_offset;
+ cache_at_topo_level =
+ machine_find_lowest_level_cache_at_topo_level(ms,
+ &bottom_core,
+ CPU_TOPOLOGY_LEVEL_CORE);
+ if (cache_at_topo_level) {
+ if (bottom_core == 1 && top_core > 1) {
+ bottom_core++;
+ cache_created = add_cpu_cache_hierarchy(ms->fdt,
+ caches,
+ num_cache,
+ top_core,
+ bottom_core, cpu,
+ &core_offset);
+
+ if (!cache_created) {
+ error_setg(&error_fatal,
+ "Core: No caches at levels %d-%d",
+ top_core, bottom_core);
+ return;
+ }
+ }
+ }
+
+ last_core = core_id;
+ }
+
+ next_level = core_offset;
+ qemu_fdt_setprop_cell(ms->fdt, nodename, "next-level-cache",
+ next_level);
+
g_free(nodename);
}
@@ -2912,7 +3239,7 @@ static void virt_set_oem_table_id(Object *obj, const char *value,
}
-bool virt_is_acpi_enabled(VirtMachineState *vms)
+bool virt_is_acpi_enabled(const VirtMachineState *vms)
{
if (vms->acpi == ON_OFF_AUTO_OFF) {
return false;
@@ -3496,6 +3823,12 @@ static void virt_machine_class_init(ObjectClass *oc, const void *data)
hc->unplug = virt_machine_device_unplug_cb;
mc->nvdimm_supported = true;
mc->smp_props.clusters_supported = true;
+
+ /* Supported caches */
+ mc->smp_props.cache_supported[CACHE_LEVEL_AND_TYPE_L1D] = true;
+ mc->smp_props.cache_supported[CACHE_LEVEL_AND_TYPE_L1I] = true;
+ mc->smp_props.cache_supported[CACHE_LEVEL_AND_TYPE_L2] = true;
+ mc->smp_props.cache_supported[CACHE_LEVEL_AND_TYPE_L3] = true;
mc->auto_enable_numa_with_memhp = true;
mc->auto_enable_numa_with_memdev = true;
/* platform instead of architectural choice */
diff --git a/include/hw/arm/virt.h b/include/hw/arm/virt.h
index dba8ac7f2f..b22868c210 100644
--- a/include/hw/arm/virt.h
+++ b/include/hw/arm/virt.h
@@ -41,6 +41,7 @@
#include "system/kvm.h"
#include "hw/intc/arm_gicv3_common.h"
#include "qom/object.h"
+#include "hw/core/cpu.h"
#define NUM_GICV2M_SPIS 64
#define NUM_VIRTIO_TRANSPORTS 32
@@ -52,6 +53,8 @@
/* GPIO pins */
#define GPIO_PIN_POWER_BUTTON 3
+#define CPU_MAX_CACHES 16
+
enum {
VIRT_FLASH,
VIRT_MEM,
@@ -195,7 +198,19 @@ struct VirtMachineState {
OBJECT_DECLARE_TYPE(VirtMachineState, VirtMachineClass, VIRT_MACHINE)
void virt_acpi_setup(VirtMachineState *vms);
-bool virt_is_acpi_enabled(VirtMachineState *vms);
+bool virt_is_acpi_enabled(const VirtMachineState *vms);
+
+#define CLIDR_CTYPE_NO_CACHE 0x00
+#define CLIDR_CTYPE_I_CACHE 0x01
+#define CLIDR_CTYPE_D_CACHE 0x02
+#define CLIDR_CTYPE_SEPARATE_I_D_CACHES 0x03
+#define CLIDR_CTYPE_UNIFIED_CACHE 0x04
+#define CLIDR_CTYPE_MAX_CACHE_LEVEL 7
+
+unsigned int virt_get_caches(const VirtMachineState *vms,
+ CPUCoreCaches *caches);
+void set_cpu_cache(CPUCoreCaches *cpu_cache, enum CacheType cache_type,
+ int cache_level, bool is_i_cache);
/* Return number of redistributors that fit in the specified region */
static uint32_t virt_redist_capacity(VirtMachineState *vms, int region)
diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h
index 7d2f4459d2..08f8df3586 100644
--- a/include/hw/core/cpu.h
+++ b/include/hw/core/cpu.h
@@ -1218,4 +1218,16 @@ enum CacheType {
UNIFIED_CACHE
};
+struct CPUCoreCaches {
+ enum CacheType type;
+ uint32_t sets;
+ uint32_t size;
+ uint32_t level;
+ uint16_t linesize;
+ uint8_t attributes; /* write policy: 0x0 write back, 0x1 write through */
+ uint8_t associativity;
+};
+
+typedef struct CPUCoreCaches CPUCoreCaches;
+
#endif
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v19 4/8] bios-tables-test: prepare to change ARM ACPI virt PPTT
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
` (2 preceding siblings ...)
2026-03-11 16:06 ` [PATCH v19 3/8] hw/arm/virt: add cache hierarchy to device tree Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 5/8] acpi: Add parameters to pass cache descriptions to ACPI build_pptt() Alireza Sanaee via qemu development
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
Prepare to update `build_pptt` function to add cache description
functionalities, thus add binaries in this patch.
Reviewed-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Thanks for the tag Gustavo.
Change log:
v18->v19:
No changes to this patch.
tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..e84d6c6955 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1 +1,4 @@
/* List of comma-separated changed AML files to ignore */
+"tests/data/acpi/aarch64/virt/PPTT",
+"tests/data/acpi/aarch64/virt/PPTT.acpihmatvirt",
+"tests/data/acpi/aarch64/virt/PPTT.topology",
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v19 5/8] acpi: Add parameters to pass cache descriptions to ACPI build_pptt()
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
` (3 preceding siblings ...)
2026-03-11 16:06 ` [PATCH v19 4/8] bios-tables-test: prepare to change ARM ACPI virt PPTT Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 6/8] hw/acpi: add cache hierarchy to pptt table Alireza Sanaee via qemu development
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
Add optional parameters to pass cache descriptions to build_pptt().
Update ARM and Loongarch callers to pass none for now.
Reviewed-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Thanks for the review Gustavo.
Change log:
v18->v19:
No changes to this patch.
hw/acpi/aml-build.c | 3 ++-
hw/arm/virt-acpi-build.c | 2 +-
hw/loongarch/virt-acpi-build.c | 4 ++--
include/hw/acpi/aml-build.h | 4 +++-
4 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
index 4b37405088..b0ea8a5d5d 100644
--- a/hw/acpi/aml-build.c
+++ b/hw/acpi/aml-build.c
@@ -2155,7 +2155,8 @@ void build_spcr(GArray *table_data, BIOSLinker *linker,
* 5.2.29 Processor Properties Topology Table (PPTT)
*/
void build_pptt(GArray *table_data, BIOSLinker *linker, MachineState *ms,
- const char *oem_id, const char *oem_table_id)
+ const char *oem_id, const char *oem_table_id,
+ int num_caches, CPUCoreCaches *caches)
{
MachineClass *mc = MACHINE_GET_CLASS(ms);
CPUArchIdList *cpus = ms->possible_cpus;
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
index 719d2f994e..3fff78bcca 100644
--- a/hw/arm/virt-acpi-build.c
+++ b/hw/arm/virt-acpi-build.c
@@ -1277,7 +1277,7 @@ void virt_acpi_build(VirtMachineState *vms, AcpiBuildTables *tables)
if (!vmc->no_cpu_topology) {
acpi_add_table(table_offsets, tables_blob);
build_pptt(tables_blob, tables->linker, ms,
- vms->oem_id, vms->oem_table_id);
+ vms->oem_id, vms->oem_table_id, 0, NULL);
}
acpi_add_table(table_offsets, tables_blob);
diff --git a/hw/loongarch/virt-acpi-build.c b/hw/loongarch/virt-acpi-build.c
index 3e34bedcd6..a0b445f297 100644
--- a/hw/loongarch/virt-acpi-build.c
+++ b/hw/loongarch/virt-acpi-build.c
@@ -538,8 +538,8 @@ static void acpi_build(AcpiBuildTables *tables, MachineState *machine)
build_madt(tables_blob, tables->linker, lvms);
acpi_add_table(table_offsets, tables_blob);
- build_pptt(tables_blob, tables->linker, machine,
- lvms->oem_id, lvms->oem_table_id);
+ build_pptt(tables_blob, tables->linker, machine, lvms->oem_id,
+ lvms->oem_table_id, 0, NULL);
acpi_add_table(table_offsets, tables_blob);
build_srat(tables_blob, tables->linker, machine);
diff --git a/include/hw/acpi/aml-build.h b/include/hw/acpi/aml-build.h
index f38e129719..e70e0643b1 100644
--- a/include/hw/acpi/aml-build.h
+++ b/include/hw/acpi/aml-build.h
@@ -3,6 +3,7 @@
#include "hw/acpi/acpi-defs.h"
#include "hw/acpi/bios-linker-loader.h"
+#include "hw/core/cpu.h"
#define ACPI_BUILD_APPNAME6 "BOCHS "
#define ACPI_BUILD_APPNAME8 "BXPC "
@@ -499,7 +500,8 @@ void build_slit(GArray *table_data, BIOSLinker *linker, MachineState *ms,
const char *oem_id, const char *oem_table_id);
void build_pptt(GArray *table_data, BIOSLinker *linker, MachineState *ms,
- const char *oem_id, const char *oem_table_id);
+ const char *oem_id, const char *oem_table_id,
+ int num_caches, CPUCoreCaches *caches);
void build_fadt(GArray *tbl, BIOSLinker *linker, const AcpiFadtData *f,
const char *oem_id, const char *oem_table_id);
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v19 6/8] hw/acpi: add cache hierarchy to pptt table
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
` (4 preceding siblings ...)
2026-03-11 16:06 ` [PATCH v19 5/8] acpi: Add parameters to pass cache descriptions to ACPI build_pptt() Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 7/8] tests/qtest/bios-table-test: testing new ARM ACPI PPTT topology Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 8/8] Update the ACPI tables based on new aml-build.c Alireza Sanaee via qemu development
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
Add cache topology to PPTT table.
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Thanks for the review Gustavo.
Change log:
v18->v19:
- Fixed some comments.
- Removed the extra check on existence of cache at the topology level
in build_pptt function since it is already checked in
machine_find_lowest_level_cache_at_topo_level function.
hw/acpi/aml-build.c | 200 +++++++++++++++++++++++++++++++++++++--
hw/arm/virt-acpi-build.c | 8 +-
include/hw/acpi/cpu.h | 10 ++
3 files changed, 209 insertions(+), 9 deletions(-)
diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
index b0ea8a5d5d..7edc8aed42 100644
--- a/hw/acpi/aml-build.c
+++ b/hw/acpi/aml-build.c
@@ -34,6 +34,7 @@
#include "hw/pci/pci_bridge.h"
#include "hw/acpi/acpi_aml_interface.h"
#include "qemu/cutils.h"
+#include "hw/core/cpu.h"
static GArray *build_alloc_array(void)
{
@@ -2150,6 +2151,108 @@ void build_spcr(GArray *table_data, BIOSLinker *linker,
}
acpi_table_end(linker, &table);
}
+
+/*
+ * ACPI spec, Revision 6.3
+ * 5.2.29.2 Cache Type Structure (Type 1)
+ */
+static void build_cache_nodes(GArray *tbl, CPUCoreCaches *cache,
+ uint32_t next_offset)
+{
+ const uint8_t node_length = 24;
+ int start_len = tbl->len;
+ int val;
+
+ build_append_byte(tbl, 1); /* Type 1 - cache */
+ build_append_byte(tbl, node_length); /* Length */
+ build_append_int_noprefix(tbl, 0, 2); /* Reserved */
+ build_append_int_noprefix(tbl, 0x7f, 4); /* Flags */
+ build_append_int_noprefix(tbl, next_offset, 4); /* Next Level of Cache */
+ build_append_int_noprefix(tbl, cache->size, 4); /* Size */
+ build_append_int_noprefix(tbl, cache->sets, 4); /* Number of sets */
+ build_append_byte(tbl, cache->associativity); /* Associativity */
+ val = 0x3;
+ switch (cache->type) {
+ case INSTRUCTION_CACHE:
+ val |= (1 << 2); /* Instruction Cache */
+ break;
+ case DATA_CACHE:
+ val |= (0 << 2); /* Data Cache */
+ break;
+ case UNIFIED_CACHE:
+ val |= (3 << 2); /* Unified */
+ break;
+ }
+ build_append_byte(tbl, val); /* Attributes */
+ build_append_int_noprefix(tbl, cache->linesize, 2); /* Line size */
+ g_assert(tbl->len == start_len + node_length);
+}
+
+/*
+ * Build PPTT Cache Type structures (Type 1) from cache level `level_high`
+ * down to `level_low` (both inclusive), appending them to the PPTT table.
+ *
+ * On output, `data_offset` and `instr_offset` hold the PPTT offsets of the
+ * lowest-level data and instruction cache nodes respectively. These offsets
+ * are referenced as private resources in the Processor Hierarchy Node (Type 0)
+ * that owns the caches.
+ */
+static bool build_caches(GArray *table_data, uint32_t pptt_start,
+ int num_caches, CPUCoreCaches *caches,
+ uint8_t level_high, /* Inclusive */
+ uint8_t level_low, /* Inclusive */
+ uint32_t *data_offset,
+ uint32_t *instr_offset)
+{
+ uint32_t next_level_offset_data = 0, next_level_offset_instruction = 0;
+ uint32_t this_offset, next_offset = 0;
+ int c, level;
+ bool found_cache = false;
+
+ /* Walk caches from top to bottom */
+ for (level = level_high; level >= level_low; level--) {
+ for (c = 0; c < num_caches; c++) {
+ if (caches[c].level != level) {
+ continue;
+ }
+
+ /* Assume only unified above l1 for now */
+ this_offset = table_data->len - pptt_start;
+ switch (caches[c].type) {
+ case INSTRUCTION_CACHE:
+ next_offset = next_level_offset_instruction;
+ break;
+ case DATA_CACHE:
+ next_offset = next_level_offset_data;
+ break;
+ case UNIFIED_CACHE:
+ /* Either is fine here */
+ next_offset = next_level_offset_instruction;
+ break;
+ }
+ build_cache_nodes(table_data, &caches[c], next_offset);
+ switch (caches[c].type) {
+ case INSTRUCTION_CACHE:
+ next_level_offset_instruction = this_offset;
+ break;
+ case DATA_CACHE:
+ next_level_offset_data = this_offset;
+ break;
+ case UNIFIED_CACHE:
+ next_level_offset_instruction = this_offset;
+ next_level_offset_data = this_offset;
+ break;
+ }
+ *data_offset = next_level_offset_data;
+ *instr_offset = next_level_offset_instruction;
+
+ found_cache = true;
+ }
+ }
+
+ return found_cache;
+}
+
/*
* ACPI spec, Revision 6.3
* 5.2.29 Processor Properties Topology Table (PPTT)
@@ -2160,11 +2263,31 @@ void build_pptt(GArray *table_data, BIOSLinker *linker, MachineState *ms,
{
MachineClass *mc = MACHINE_GET_CLASS(ms);
CPUArchIdList *cpus = ms->possible_cpus;
- int64_t socket_id = -1, cluster_id = -1, core_id = -1;
- uint32_t socket_offset = 0, cluster_offset = 0, core_offset = 0;
+ uint32_t core_data_offset = 0;
+ uint32_t core_instr_offset = 0;
+ uint32_t cluster_instr_offset = 0;
+ uint32_t cluster_data_offset = 0;
+ uint32_t node_data_offset = 0;
+ uint32_t node_instr_offset = 0;
+ int top_node = 3;
+ int top_cluster = 3;
+ int top_core = 3;
+ int bottom_node = 3;
+ int bottom_cluster = 3;
+ int bottom_core = 3;
+ int64_t socket_id = -1;
+ int64_t cluster_id = -1;
+ int64_t core_id = -1;
+ uint32_t socket_offset = 0;
+ uint32_t cluster_offset = 0;
+ uint32_t core_offset = 0;
uint32_t pptt_start = table_data->len;
uint32_t root_offset;
int n;
+ uint32_t priv_rsrc[2];
+ uint32_t num_priv = 0;
+ bool cache_at_topo_level;
+
AcpiTable table = { .sig = "PPTT", .rev = 2,
.oem_id = oem_id, .oem_table_id = oem_table_id };
@@ -2194,11 +2317,29 @@ void build_pptt(GArray *table_data, BIOSLinker *linker, MachineState *ms,
socket_id = cpus->cpus[n].props.socket_id;
cluster_id = -1;
core_id = -1;
+ bottom_node = top_node;
+ num_priv = 0;
+ cache_at_topo_level =
+ machine_find_lowest_level_cache_at_topo_level(
+ ms, &bottom_node, CPU_TOPOLOGY_LEVEL_SOCKET);
+ if (cache_at_topo_level) {
+ build_caches(table_data, pptt_start, num_caches, caches,
+ top_node, bottom_node, &node_data_offset,
+ &node_instr_offset);
+ priv_rsrc[0] = node_instr_offset;
+ priv_rsrc[1] = node_data_offset;
+ if (node_instr_offset || node_data_offset) {
+ num_priv = node_instr_offset == node_data_offset ? 1 : 2;
+ }
+
+ top_cluster = bottom_node - 1;
+ }
+
socket_offset = table_data->len - pptt_start;
build_processor_hierarchy_node(table_data,
(1 << 0) | /* Physical package */
(1 << 4), /* Identical Implementation */
- root_offset, socket_id, NULL, 0);
+ root_offset, socket_id, priv_rsrc, num_priv);
}
if (mc->smp_props.clusters_supported && mc->smp_props.has_clusters) {
@@ -2206,21 +2347,66 @@ void build_pptt(GArray *table_data, BIOSLinker *linker, MachineState *ms,
assert(cpus->cpus[n].props.cluster_id > cluster_id);
cluster_id = cpus->cpus[n].props.cluster_id;
core_id = -1;
+ bottom_cluster = top_cluster;
+ num_priv = 0;
+ cache_at_topo_level =
+ machine_find_lowest_level_cache_at_topo_level(
+ ms, &bottom_cluster, CPU_TOPOLOGY_LEVEL_CLUSTER);
+
+ if (cache_at_topo_level) {
+ build_caches(table_data, pptt_start, num_caches, caches,
+ top_cluster, bottom_cluster,
+ &cluster_data_offset, &cluster_instr_offset);
+ priv_rsrc[0] = cluster_instr_offset;
+ priv_rsrc[1] = cluster_data_offset;
+ if (cluster_instr_offset || cluster_data_offset) {
+ num_priv =
+ cluster_instr_offset == cluster_data_offset ? 1 : 2;
+ }
+ top_core = bottom_cluster - 1;
+ } else if (top_cluster == bottom_node - 1) {
+ /* socket cache but no cluster cache */
+ top_core = bottom_node - 1;
+ }
+
cluster_offset = table_data->len - pptt_start;
build_processor_hierarchy_node(table_data,
(0 << 0) | /* Not a physical package */
(1 << 4), /* Identical Implementation */
- socket_offset, cluster_id, NULL, 0);
+ socket_offset, cluster_id, priv_rsrc, num_priv);
}
} else {
+ if (machine_defines_cache_at_topo_level(
+ ms, CPU_TOPOLOGY_LEVEL_CLUSTER)) {
+ error_setg(&error_fatal, "Not clusters found for the cache");
+ return;
+ }
+
cluster_offset = socket_offset;
+ top_core = bottom_node - 1; /* there is no cluster */
+ }
+
+ if (cpus->cpus[n].props.core_id != core_id) {
+ bottom_core = top_core;
+ num_priv = 0;
+ cache_at_topo_level =
+ machine_find_lowest_level_cache_at_topo_level(
+ ms, &bottom_core, CPU_TOPOLOGY_LEVEL_CORE);
+ if (cache_at_topo_level) {
+ build_caches(table_data, pptt_start, num_caches, caches,
+ top_core, bottom_core, &core_data_offset,
+ &core_instr_offset);
+ priv_rsrc[0] = core_instr_offset;
+ priv_rsrc[1] = core_data_offset;
+ num_priv = core_instr_offset == core_data_offset ? 1 : 2;
+ }
}
if (ms->smp.threads == 1) {
build_processor_hierarchy_node(table_data,
(1 << 1) | /* ACPI Processor ID valid */
- (1 << 3), /* Node is a Leaf */
- cluster_offset, n, NULL, 0);
+ (1 << 3), /* Node is a Leaf */
+ cluster_offset, n, priv_rsrc, num_priv);
} else {
if (cpus->cpus[n].props.core_id != core_id) {
assert(cpus->cpus[n].props.core_id > core_id);
@@ -2229,7 +2415,7 @@ void build_pptt(GArray *table_data, BIOSLinker *linker, MachineState *ms,
build_processor_hierarchy_node(table_data,
(0 << 0) | /* Not a physical package */
(1 << 4), /* Identical Implementation */
- cluster_offset, core_id, NULL, 0);
+ cluster_offset, core_id, priv_rsrc, num_priv);
}
build_processor_hierarchy_node(table_data,
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
index 3fff78bcca..b230ea2c8e 100644
--- a/hw/arm/virt-acpi-build.c
+++ b/hw/arm/virt-acpi-build.c
@@ -1255,6 +1255,10 @@ void virt_acpi_build(VirtMachineState *vms, AcpiBuildTables *tables)
unsigned dsdt, xsdt;
GArray *tables_blob = tables->table_data;
MachineState *ms = MACHINE(vms);
+ CPUCoreCaches caches[CPU_MAX_CACHES];
+ unsigned int num_caches;
+
+ num_caches = virt_get_caches(vms, caches);
table_offsets = g_array_new(false, true /* clear */,
sizeof(uint32_t));
@@ -1276,8 +1280,8 @@ void virt_acpi_build(VirtMachineState *vms, AcpiBuildTables *tables)
if (!vmc->no_cpu_topology) {
acpi_add_table(table_offsets, tables_blob);
- build_pptt(tables_blob, tables->linker, ms,
- vms->oem_id, vms->oem_table_id, 0, NULL);
+ build_pptt(tables_blob, tables->linker, ms, vms->oem_id,
+ vms->oem_table_id, num_caches, caches);
}
acpi_add_table(table_offsets, tables_blob);
diff --git a/include/hw/acpi/cpu.h b/include/hw/acpi/cpu.h
index 2809dd8a91..04c821d2b9 100644
--- a/include/hw/acpi/cpu.h
+++ b/include/hw/acpi/cpu.h
@@ -69,6 +69,16 @@ void build_cpus_aml(Aml *table, MachineState *machine, CPUHotplugFeatures opts,
void acpi_cpu_ospm_status(CPUHotplugState *cpu_st, ACPIOSTInfoList ***list);
+struct CPUPPTTCaches {
+ enum CacheType type;
+ uint32_t sets;
+ uint32_t size;
+ uint32_t level;
+ uint16_t linesize;
+ uint8_t attributes; /* write policy: 0x0 write back, 0x1 write through */
+ uint8_t associativity;
+};
+
extern const VMStateDescription vmstate_cpu_hotplug;
#define VMSTATE_CPU_HOTPLUG(cpuhp, state) \
VMSTATE_STRUCT(cpuhp, state, 1, \
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v19 7/8] tests/qtest/bios-table-test: testing new ARM ACPI PPTT topology
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
` (5 preceding siblings ...)
2026-03-11 16:06 ` [PATCH v19 6/8] hw/acpi: add cache hierarchy to pptt table Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 8/8] Update the ACPI tables based on new aml-build.c Alireza Sanaee via qemu development
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
Test new PPTT topolopy with cache representation.
Reviewed-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Thanks for the tag Gustavo.
Change log:
v18->v19:
No changes to this patch.
tests/qtest/bios-tables-test.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index a5a5b8807b..510751799e 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest/bios-tables-test.c
@@ -2229,6 +2229,10 @@ static void test_acpi_aarch64_virt_tcg_topology(void)
};
test_acpi_one("-cpu cortex-a57 "
+ "-M virt,smp-cache.0.cache=l1i,smp-cache.0.topology=cluster,"
+ "smp-cache.1.cache=l1d,smp-cache.1.topology=cluster,"
+ "smp-cache.2.cache=l2,smp-cache.2.topology=cluster,"
+ "smp-cache.3.cache=l3,smp-cache.3.topology=cluster "
"-smp sockets=1,clusters=2,cores=2,threads=2", &data);
free_test_data(&data);
}
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v19 8/8] Update the ACPI tables based on new aml-build.c
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
` (6 preceding siblings ...)
2026-03-11 16:06 ` [PATCH v19 7/8] tests/qtest/bios-table-test: testing new ARM ACPI PPTT topology Alireza Sanaee via qemu development
@ 2026-03-11 16:06 ` Alireza Sanaee via qemu development
7 siblings, 0 replies; 9+ messages in thread
From: Alireza Sanaee via qemu development @ 2026-03-11 16:06 UTC (permalink / raw)
To: qemu-devel, gustavo.romero, mst
Cc: anisinha, armbru, berrange, dapeng1.mi, eric.auger, farman,
imammedo, jiangkunkun, jonathan.cameron, linuxarm, maobibo,
mtosatti, peter.maydell, philmd, qemu-arm, richard.henderson,
shannon.zhaosl, zhao1.liu
The disassembled differences between actual and expected PPTT based on
the following cache topology representation:
- l1d and l1i shared at cluster level
- l2 shared at cluster level
- l3 shared at cluster level
/*
* Intel ACPI Component Architecture
* AML/ASL+ Disassembler version 20230628 (64-bit version)
* Copyright (c) 2000 - 2023 Intel Corporation
*
- * Disassembly of tests/data/acpi/aarch64/virt/PPTT.topology, Fri Aug 8 16:50:38 2025
+ * Disassembly of /tmp/aml-JGBZA3, Fri Aug 8 16:50:38 2025
*
* ACPI Data Table [PPTT]
*
* Format: [HexOffset DecimalOffset ByteLength] FieldName : FieldValue (in hex)
*/
[000h 0000 004h] Signature : "PPTT" [Processor Properties Topology Table]
-[004h 0004 004h] Table Length : 00000164
+[004h 0004 004h] Table Length : 00000204
[008h 0008 001h] Revision : 02
-[009h 0009 001h] Checksum : 97
+[009h 0009 001h] Checksum : B8
[00Ah 0010 006h] Oem ID : "BOCHS "
[010h 0016 008h] Oem Table ID : "BXPC "
[018h 0024 004h] Oem Revision : 00000001
[01Ch 0028 004h] Asl Compiler ID : "BXPC"
[020h 0032 004h] Asl Compiler Revision : 00000001
[024h 0036 001h] Subtable Type : 00 [Processor Hierarchy Node]
[025h 0037 001h] Length : 14
[026h 0038 002h] Reserved : 0000
[028h 0040 004h] Flags (decoded below) : 00000011
Physical package : 1
ACPI Processor ID valid : 0
Processor is a thread : 0
Node is a leaf : 0
Identical Implementation : 1
@@ -34,223 +34,369 @@
[030h 0048 004h] ACPI Processor ID : 00000000
[034h 0052 004h] Private Resource Number : 00000000
[038h 0056 001h] Subtable Type : 00 [Processor Hierarchy Node]
[039h 0057 001h] Length : 14
[03Ah 0058 002h] Reserved : 0000
[03Ch 0060 004h] Flags (decoded below) : 00000011
Physical package : 1
ACPI Processor ID valid : 0
Processor is a thread : 0
Node is a leaf : 0
Identical Implementation : 1
[040h 0064 004h] Parent : 00000024
[044h 0068 004h] ACPI Processor ID : 00000000
[048h 0072 004h] Private Resource Number : 00000000
-[04Ch 0076 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[04Dh 0077 001h] Length : 14
+[04Ch 0076 001h] Subtable Type : 01 [Cache Type]
+[04Dh 0077 001h] Length : 18
[04Eh 0078 002h] Reserved : 0000
-[050h 0080 004h] Flags (decoded below) : 00000010
+[050h 0080 004h] Flags (decoded below) : 0000007F
+ Size valid : 1
+ Number of Sets valid : 1
+ Associativity valid : 1
+ Allocation Type valid : 1
+ Cache Type valid : 1
+ Write Policy valid : 1
+ Line Size valid : 1
+ Cache ID valid : 0
+[054h 0084 004h] Next Level of Cache : 00000000
+[058h 0088 004h] Size : 00200000
+[05Ch 0092 004h] Number of Sets : 00000800
+[060h 0096 001h] Associativity : 10
+[061h 0097 001h] Attributes : 0F
+ Allocation Type : 3
+ Cache Type : 3
+ Write Policy : 0
+[062h 0098 002h] Line Size : 0040
+
+[064h 0100 001h] Subtable Type : 01 [Cache Type]
+[065h 0101 001h] Length : 18
+[066h 0102 002h] Reserved : 0000
+[068h 0104 004h] Flags (decoded below) : 0000007F
+ Size valid : 1
+ Number of Sets valid : 1
+ Associativity valid : 1
+ Allocation Type valid : 1
+ Cache Type valid : 1
+ Write Policy valid : 1
+ Line Size valid : 1
+ Cache ID valid : 0
+[06Ch 0108 004h] Next Level of Cache : 0000004C
+[070h 0112 004h] Size : 00008000
+[074h 0116 004h] Number of Sets : 00000080
+[078h 0120 001h] Associativity : 04
+[079h 0121 001h] Attributes : 03
+ Allocation Type : 3
+ Cache Type : 0
+ Write Policy : 0
+[07Ah 0122 002h] Line Size : 0040
+
+[07Ch 0124 001h] Subtable Type : 01 [Cache Type]
+[07Dh 0125 001h] Length : 18
+[07Eh 0126 002h] Reserved : 0000
+[080h 0128 004h] Flags (decoded below) : 0000007F
+ Size valid : 1
+ Number of Sets valid : 1
+ Associativity valid : 1
+ Allocation Type valid : 1
+ Cache Type valid : 1
+ Write Policy valid : 1
+ Line Size valid : 1
+ Cache ID valid : 0
+[084h 0132 004h] Next Level of Cache : 0000004C
+[088h 0136 004h] Size : 0000C000
+[08Ch 0140 004h] Number of Sets : 00000100
+[090h 0144 001h] Associativity : 03
+[091h 0145 001h] Attributes : 07
+ Allocation Type : 3
+ Cache Type : 1
+ Write Policy : 0
+[092h 0146 002h] Line Size : 0040
+
+[094h 0148 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[095h 0149 001h] Length : 1C
+[096h 0150 002h] Reserved : 0000
+[098h 0152 004h] Flags (decoded below) : 00000010
Physical package : 0
ACPI Processor ID valid : 0
Processor is a thread : 0
Node is a leaf : 0
Identical Implementation : 1
-[054h 0084 004h] Parent : 00000038
-[058h 0088 004h] ACPI Processor ID : 00000000
-[05Ch 0092 004h] Private Resource Number : 00000000
-
-[060h 0096 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[061h 0097 001h] Length : 14
-[062h 0098 002h] Reserved : 0000
-[064h 0100 004h] Flags (decoded below) : 00000010
- Physical package : 0
- ACPI Processor ID valid : 0
- Processor is a thread : 0
- Node is a leaf : 0
- Identical Implementation : 1
-[068h 0104 004h] Parent : 0000004C
-[06Ch 0108 004h] ACPI Processor ID : 00000000
-[070h 0112 004h] Private Resource Number : 00000000
-
-[074h 0116 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[075h 0117 001h] Length : 14
-[076h 0118 002h] Reserved : 0000
-[078h 0120 004h] Flags (decoded below) : 0000000E
- Physical package : 0
- ACPI Processor ID valid : 1
- Processor is a thread : 1
- Node is a leaf : 1
- Identical Implementation : 0
-[07Ch 0124 004h] Parent : 00000060
-[080h 0128 004h] ACPI Processor ID : 00000000
-[084h 0132 004h] Private Resource Number : 00000000
-
-[088h 0136 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[089h 0137 001h] Length : 14
-[08Ah 0138 002h] Reserved : 0000
-[08Ch 0140 004h] Flags (decoded below) : 0000000E
- Physical package : 0
- ACPI Processor ID valid : 1
- Processor is a thread : 1
- Node is a leaf : 1
- Identical Implementation : 0
-[090h 0144 004h] Parent : 00000060
-[094h 0148 004h] ACPI Processor ID : 00000001
-[098h 0152 004h] Private Resource Number : 00000000
-
-[09Ch 0156 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[09Dh 0157 001h] Length : 14
-[09Eh 0158 002h] Reserved : 0000
-[0A0h 0160 004h] Flags (decoded below) : 00000010
- Physical package : 0
- ACPI Processor ID valid : 0
- Processor is a thread : 0
- Node is a leaf : 0
- Identical Implementation : 1
-[0A4h 0164 004h] Parent : 0000004C
-[0A8h 0168 004h] ACPI Processor ID : 00000001
-[0ACh 0172 004h] Private Resource Number : 00000000
+[09Ch 0156 004h] Parent : 00000038
+[0A0h 0160 004h] ACPI Processor ID : 00000000
+[0A4h 0164 004h] Private Resource Number : 00000002
+[0A8h 0168 004h] Private Resource : 0000007C
+[0ACh 0172 004h] Private Resource : 00000064
[0B0h 0176 001h] Subtable Type : 00 [Processor Hierarchy Node]
[0B1h 0177 001h] Length : 14
[0B2h 0178 002h] Reserved : 0000
-[0B4h 0180 004h] Flags (decoded below) : 0000000E
+[0B4h 0180 004h] Flags (decoded below) : 00000010
Physical package : 0
- ACPI Processor ID valid : 1
- Processor is a thread : 1
- Node is a leaf : 1
- Identical Implementation : 0
-[0B8h 0184 004h] Parent : 0000009C
-[0BCh 0188 004h] ACPI Processor ID : 00000002
+ ACPI Processor ID valid : 0
+ Processor is a thread : 0
+ Node is a leaf : 0
+ Identical Implementation : 1
+[0B8h 0184 004h] Parent : 00000094
+[0BCh 0188 004h] ACPI Processor ID : 00000000
[0C0h 0192 004h] Private Resource Number : 00000000
[0C4h 0196 001h] Subtable Type : 00 [Processor Hierarchy Node]
[0C5h 0197 001h] Length : 14
[0C6h 0198 002h] Reserved : 0000
[0C8h 0200 004h] Flags (decoded below) : 0000000E
Physical package : 0
ACPI Processor ID valid : 1
Processor is a thread : 1
Node is a leaf : 1
Identical Implementation : 0
-[0CCh 0204 004h] Parent : 0000009C
-[0D0h 0208 004h] ACPI Processor ID : 00000003
+[0CCh 0204 004h] Parent : 000000B0
+[0D0h 0208 004h] ACPI Processor ID : 00000000
[0D4h 0212 004h] Private Resource Number : 00000000
[0D8h 0216 001h] Subtable Type : 00 [Processor Hierarchy Node]
[0D9h 0217 001h] Length : 14
[0DAh 0218 002h] Reserved : 0000
-[0DCh 0220 004h] Flags (decoded below) : 00000010
+[0DCh 0220 004h] Flags (decoded below) : 0000000E
Physical package : 0
- ACPI Processor ID valid : 0
- Processor is a thread : 0
- Node is a leaf : 0
- Identical Implementation : 1
-[0E0h 0224 004h] Parent : 00000038
+ ACPI Processor ID valid : 1
+ Processor is a thread : 1
+ Node is a leaf : 1
+ Identical Implementation : 0
+[0E0h 0224 004h] Parent : 000000B0
[0E4h 0228 004h] ACPI Processor ID : 00000001
[0E8h 0232 004h] Private Resource Number : 00000000
[0ECh 0236 001h] Subtable Type : 00 [Processor Hierarchy Node]
[0EDh 0237 001h] Length : 14
[0EEh 0238 002h] Reserved : 0000
[0F0h 0240 004h] Flags (decoded below) : 00000010
Physical package : 0
ACPI Processor ID valid : 0
Processor is a thread : 0
Node is a leaf : 0
Identical Implementation : 1
-[0F4h 0244 004h] Parent : 000000D8
-[0F8h 0248 004h] ACPI Processor ID : 00000000
+[0F4h 0244 004h] Parent : 00000094
+[0F8h 0248 004h] ACPI Processor ID : 00000001
[0FCh 0252 004h] Private Resource Number : 00000000
[100h 0256 001h] Subtable Type : 00 [Processor Hierarchy Node]
[101h 0257 001h] Length : 14
[102h 0258 002h] Reserved : 0000
[104h 0260 004h] Flags (decoded below) : 0000000E
Physical package : 0
ACPI Processor ID valid : 1
Processor is a thread : 1
Node is a leaf : 1
Identical Implementation : 0
[108h 0264 004h] Parent : 000000EC
-[10Ch 0268 004h] ACPI Processor ID : 00000004
+[10Ch 0268 004h] ACPI Processor ID : 00000002
[110h 0272 004h] Private Resource Number : 00000000
[114h 0276 001h] Subtable Type : 00 [Processor Hierarchy Node]
[115h 0277 001h] Length : 14
[116h 0278 002h] Reserved : 0000
[118h 0280 004h] Flags (decoded below) : 0000000E
Physical package : 0
ACPI Processor ID valid : 1
Processor is a thread : 1
Node is a leaf : 1
Identical Implementation : 0
[11Ch 0284 004h] Parent : 000000EC
-[120h 0288 004h] ACPI Processor ID : 00000005
+[120h 0288 004h] ACPI Processor ID : 00000003
[124h 0292 004h] Private Resource Number : 00000000
-[128h 0296 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[129h 0297 001h] Length : 14
+[128h 0296 001h] Subtable Type : 01 [Cache Type]
+[129h 0297 001h] Length : 18
[12Ah 0298 002h] Reserved : 0000
-[12Ch 0300 004h] Flags (decoded below) : 00000010
+[12Ch 0300 004h] Flags (decoded below) : 0000007F
+ Size valid : 1
+ Number of Sets valid : 1
+ Associativity valid : 1
+ Allocation Type valid : 1
+ Cache Type valid : 1
+ Write Policy valid : 1
+ Line Size valid : 1
+ Cache ID valid : 0
+[130h 0304 004h] Next Level of Cache : 00000000
+[134h 0308 004h] Size : 00200000
+[138h 0312 004h] Number of Sets : 00000800
+[13Ch 0316 001h] Associativity : 10
+[13Dh 0317 001h] Attributes : 0F
+ Allocation Type : 3
+ Cache Type : 3
+ Write Policy : 0
+[13Eh 0318 002h] Line Size : 0040
+
+[140h 0320 001h] Subtable Type : 01 [Cache Type]
+[141h 0321 001h] Length : 18
+[142h 0322 002h] Reserved : 0000
+[144h 0324 004h] Flags (decoded below) : 0000007F
+ Size valid : 1
+ Number of Sets valid : 1
+ Associativity valid : 1
+ Allocation Type valid : 1
+ Cache Type valid : 1
+ Write Policy valid : 1
+ Line Size valid : 1
+ Cache ID valid : 0
+[148h 0328 004h] Next Level of Cache : 00000128
+[14Ch 0332 004h] Size : 00008000
+[150h 0336 004h] Number of Sets : 00000080
+[154h 0340 001h] Associativity : 04
+[155h 0341 001h] Attributes : 03
+ Allocation Type : 3
+ Cache Type : 0
+ Write Policy : 0
+[156h 0342 002h] Line Size : 0040
+
+[158h 0344 001h] Subtable Type : 01 [Cache Type]
+[159h 0345 001h] Length : 18
+[15Ah 0346 002h] Reserved : 0000
+[15Ch 0348 004h] Flags (decoded below) : 0000007F
+ Size valid : 1
+ Number of Sets valid : 1
+ Associativity valid : 1
+ Allocation Type valid : 1
+ Cache Type valid : 1
+ Write Policy valid : 1
+ Line Size valid : 1
+ Cache ID valid : 0
+[160h 0352 004h] Next Level of Cache : 00000128
+[164h 0356 004h] Size : 0000C000
+[168h 0360 004h] Number of Sets : 00000100
+[16Ch 0364 001h] Associativity : 03
+[16Dh 0365 001h] Attributes : 07
+ Allocation Type : 3
+ Cache Type : 1
+ Write Policy : 0
+[16Eh 0366 002h] Line Size : 0040
+
+[170h 0368 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[171h 0369 001h] Length : 1C
+[172h 0370 002h] Reserved : 0000
+[174h 0372 004h] Flags (decoded below) : 00000010
+ Physical package : 0
+ ACPI Processor ID valid : 0
+ Processor is a thread : 0
+ Node is a leaf : 0
+ Identical Implementation : 1
+[178h 0376 004h] Parent : 00000038
+[17Ch 0380 004h] ACPI Processor ID : 00000001
+[180h 0384 004h] Private Resource Number : 00000002
+[184h 0388 004h] Private Resource : 00000158
+[188h 0392 004h] Private Resource : 00000140
+
+[18Ch 0396 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[18Dh 0397 001h] Length : 14
+[18Eh 0398 002h] Reserved : 0000
+[190h 0400 004h] Flags (decoded below) : 00000010
+ Physical package : 0
+ ACPI Processor ID valid : 0
+ Processor is a thread : 0
+ Node is a leaf : 0
+ Identical Implementation : 1
+[194h 0404 004h] Parent : 00000170
+[198h 0408 004h] ACPI Processor ID : 00000000
+[19Ch 0412 004h] Private Resource Number : 00000000
+
+[1A0h 0416 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[1A1h 0417 001h] Length : 14
+[1A2h 0418 002h] Reserved : 0000
+[1A4h 0420 004h] Flags (decoded below) : 0000000E
+ Physical package : 0
+ ACPI Processor ID valid : 1
+ Processor is a thread : 1
+ Node is a leaf : 1
+ Identical Implementation : 0
+[1A8h 0424 004h] Parent : 0000018C
+[1ACh 0428 004h] ACPI Processor ID : 00000004
+[1B0h 0432 004h] Private Resource Number : 00000000
+
+[1B4h 0436 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[1B5h 0437 001h] Length : 14
+[1B6h 0438 002h] Reserved : 0000
+[1B8h 0440 004h] Flags (decoded below) : 0000000E
+ Physical package : 0
+ ACPI Processor ID valid : 1
+ Processor is a thread : 1
+ Node is a leaf : 1
+ Identical Implementation : 0
+[1BCh 0444 004h] Parent : 0000018C
+[1C0h 0448 004h] ACPI Processor ID : 00000005
+[1C4h 0452 004h] Private Resource Number : 00000000
+
+[1C8h 0456 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[1C9h 0457 001h] Length : 14
+[1CAh 0458 002h] Reserved : 0000
+[1CCh 0460 004h] Flags (decoded below) : 00000010
Physical package : 0
ACPI Processor ID valid : 0
Processor is a thread : 0
Node is a leaf : 0
Identical Implementation : 1
-[130h 0304 004h] Parent : 000000D8
-[134h 0308 004h] ACPI Processor ID : 00000001
-[138h 0312 004h] Private Resource Number : 00000000
-
-[13Ch 0316 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[13Dh 0317 001h] Length : 14
-[13Eh 0318 002h] Reserved : 0000
-[140h 0320 004h] Flags (decoded below) : 0000000E
+[1D0h 0464 004h] Parent : 00000170
+[1D4h 0468 004h] ACPI Processor ID : 00000001
+[1D8h 0472 004h] Private Resource Number : 00000000
+
+[1DCh 0476 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[1DDh 0477 001h] Length : 14
+[1DEh 0478 002h] Reserved : 0000
+[1E0h 0480 004h] Flags (decoded below) : 0000000E
Physical package : 0
ACPI Processor ID valid : 1
Processor is a thread : 1
Node is a leaf : 1
Identical Implementation : 0
-[144h 0324 004h] Parent : 00000128
-[148h 0328 004h] ACPI Processor ID : 00000006
-[14Ch 0332 004h] Private Resource Number : 00000000
-
-[150h 0336 001h] Subtable Type : 00 [Processor Hierarchy Node]
-[151h 0337 001h] Length : 14
-[152h 0338 002h] Reserved : 0000
-[154h 0340 004h] Flags (decoded below) : 0000000E
+[1E4h 0484 004h] Parent : 000001C8
+[1E8h 0488 004h] ACPI Processor ID : 00000006
+[1ECh 0492 004h] Private Resource Number : 00000000
+
+[1F0h 0496 001h] Subtable Type : 00 [Processor Hierarchy Node]
+[1F1h 0497 001h] Length : 14
+[1F2h 0498 002h] Reserved : 0000
+[1F4h 0500 004h] Flags (decoded below) : 0000000E
Physical package : 0
ACPI Processor ID valid : 1
Processor is a thread : 1
Node is a leaf : 1
Identical Implementation : 0
-[158h 0344 004h] Parent : 00000128
-[15Ch 0348 004h] ACPI Processor ID : 00000007
-[160h 0352 004h] Private Resource Number : 00000000
+[1F8h 0504 004h] Parent : 000001C8
+[1FCh 0508 004h] ACPI Processor ID : 00000007
+[200h 0512 004h] Private Resource Number : 00000000
-Raw Table Data: Length 356 (0x164)
+Raw Table Data: Length 516 (0x204)
- 0000: 50 50 54 54 64 01 00 00 02 97 42 4F 43 48 53 20 // PPTTd.....BOCHS
+ 0000: 50 50 54 54 04 02 00 00 02 B8 42 4F 43 48 53 20 // PPTT......BOCHS
0010: 42 58 50 43 20 20 20 20 01 00 00 00 42 58 50 43 // BXPC ....BXPC
0020: 01 00 00 00 00 14 00 00 11 00 00 00 00 00 00 00 // ................
0030: 00 00 00 00 00 00 00 00 00 14 00 00 11 00 00 00 // ................
- 0040: 24 00 00 00 00 00 00 00 00 00 00 00 00 14 00 00 // $...............
- 0050: 10 00 00 00 38 00 00 00 00 00 00 00 00 00 00 00 // ....8...........
- 0060: 00 14 00 00 10 00 00 00 4C 00 00 00 00 00 00 00 // ........L.......
- 0070: 00 00 00 00 00 14 00 00 0E 00 00 00 60 00 00 00 // ............`...
- 0080: 00 00 00 00 00 00 00 00 00 14 00 00 0E 00 00 00 // ................
- 0090: 60 00 00 00 01 00 00 00 00 00 00 00 00 14 00 00 // `...............
- 00A0: 10 00 00 00 4C 00 00 00 01 00 00 00 00 00 00 00 // ....L...........
- 00B0: 00 14 00 00 0E 00 00 00 9C 00 00 00 02 00 00 00 // ................
- 00C0: 00 00 00 00 00 14 00 00 0E 00 00 00 9C 00 00 00 // ................
- 00D0: 03 00 00 00 00 00 00 00 00 14 00 00 10 00 00 00 // ................
- 00E0: 38 00 00 00 01 00 00 00 00 00 00 00 00 14 00 00 // 8...............
- 00F0: 10 00 00 00 D8 00 00 00 00 00 00 00 00 00 00 00 // ................
- 0100: 00 14 00 00 0E 00 00 00 EC 00 00 00 04 00 00 00 // ................
+ 0040: 24 00 00 00 00 00 00 00 00 00 00 00 01 18 00 00 // $...............
+ 0050: 7F 00 00 00 00 00 00 00 00 00 20 00 00 08 00 00 // .......... .....
+ 0060: 10 0F 40 00 01 18 00 00 7F 00 00 00 4C 00 00 00 // ..@.........L...
+ 0070: 00 80 00 00 80 00 00 00 04 03 40 00 01 18 00 00 // ..........@.....
+ 0080: 7F 00 00 00 4C 00 00 00 00 C0 00 00 00 01 00 00 // ....L...........
+ 0090: 03 07 40 00 00 1C 00 00 10 00 00 00 38 00 00 00 // ..@.........8...
+ 00A0: 00 00 00 00 02 00 00 00 7C 00 00 00 64 00 00 00 // ........|...d...
+ 00B0: 00 14 00 00 10 00 00 00 94 00 00 00 00 00 00 00 // ................
+ 00C0: 00 00 00 00 00 14 00 00 0E 00 00 00 B0 00 00 00 // ................
+ 00D0: 00 00 00 00 00 00 00 00 00 14 00 00 0E 00 00 00 // ................
+ 00E0: B0 00 00 00 01 00 00 00 00 00 00 00 00 14 00 00 // ................
+ 00F0: 10 00 00 00 94 00 00 00 01 00 00 00 00 00 00 00 // ................
+ 0100: 00 14 00 00 0E 00 00 00 EC 00 00 00 02 00 00 00 // ................
0110: 00 00 00 00 00 14 00 00 0E 00 00 00 EC 00 00 00 // ................
- 0120: 05 00 00 00 00 00 00 00 00 14 00 00 10 00 00 00 // ................
- 0130: D8 00 00 00 01 00 00 00 00 00 00 00 00 14 00 00 // ................
- 0140: 0E 00 00 00 28 01 00 00 06 00 00 00 00 00 00 00 // ....(...........
- 0150: 00 14 00 00 0E 00 00 00 28 01 00 00 07 00 00 00 // ........(.......
- 0160: 00 00 00 00 // ....
+ 0120: 03 00 00 00 00 00 00 00 01 18 00 00 7F 00 00 00 // ................
+ 0130: 00 00 00 00 00 00 20 00 00 08 00 00 10 0F 40 00 // ...... .......@.
+ 0140: 01 18 00 00 7F 00 00 00 28 01 00 00 00 80 00 00 // ........(.......
+ 0150: 80 00 00 00 04 03 40 00 01 18 00 00 7F 00 00 00 // ......@.........
+ 0160: 28 01 00 00 00 C0 00 00 00 01 00 00 03 07 40 00 // (.............@.
+ 0170: 00 1C 00 00 10 00 00 00 38 00 00 00 01 00 00 00 // ........8.......
+ 0180: 02 00 00 00 58 01 00 00 40 01 00 00 00 14 00 00 // ....X...@.......
+ 0190: 10 00 00 00 70 01 00 00 00 00 00 00 00 00 00 00 // ....p...........
+ 01A0: 00 14 00 00 0E 00 00 00 8C 01 00 00 04 00 00 00 // ................
+ 01B0: 00 00 00 00 00 14 00 00 0E 00 00 00 8C 01 00 00 // ................
+ 01C0: 05 00 00 00 00 00 00 00 00 14 00 00 10 00 00 00 // ................
+ 01D0: 70 01 00 00 01 00 00 00 00 00 00 00 00 14 00 00 // p...............
+ 01E0: 0E 00 00 00 C8 01 00 00 06 00 00 00 00 00 00 00 // ................
+ 01F0: 00 14 00 00 0E 00 00 00 C8 01 00 00 07 00 00 00 // ................
+ 0200: 00 00 00 00 // ....
Reviewed-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
---
Thanks for the tag Gustavo.
Change log:
v18 -> v19:
- No change in this patch.
tests/data/acpi/aarch64/virt/PPTT.topology | Bin 356 -> 516 bytes
tests/qtest/bios-tables-test-allowed-diff.h | 3 ---
2 files changed, 3 deletions(-)
diff --git a/tests/data/acpi/aarch64/virt/PPTT.topology b/tests/data/acpi/aarch64/virt/PPTT.topology
index 6b864f035c9f48845e9a3beb482c5171074864a5..4f9472c5f728f3068d1054d5042b85190bdb88da 100644
GIT binary patch
literal 516
zcmZvXy$!-Z4255QAXNNF6ciL!P%r{zlr$7bL?T57U;qX{A_Gt|2qk4ohG7Wa3wP0p
z#END6^S#(Ein5GDAbe%Ve19@oRpf>i08p-oC9qKR&9aThf)#M<Y6DDw`7DLw2leXq
zLmd6_hCL38k`!1|$8txPaXnn=XBC{Q-b1-FvMKYYs}()g-e8&2`b^pnU2|HqTCvC?
zcf+qVz1z0>Vcoy2<qdlSw@IRz6_Zp2=W4%;a%XmzJ6SxyMjmt8PHwetg0c5b_lhN!
FeE|+Z9RUCU
literal 356
zcmWFt2nk7HWME*L?&R<65v<@85#X!<1VAAM5F11@h%hh+f@ov_6;nYI69Dopu!#Af
ziSYsX2{^>Sc7o)9c7V(S=|vU;>74__Oh60<Ky@%NW+X9~TafjF#BRXUfM}@RH$Wx}
cOdLs!6-f-H7uh_Jy&6CPHY9a0F?OgJ00?&w0RR91
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index e84d6c6955..dfb8523c8b 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1,4 +1 @@
/* List of comma-separated changed AML files to ignore */
-"tests/data/acpi/aarch64/virt/PPTT",
-"tests/data/acpi/aarch64/virt/PPTT.acpihmatvirt",
-"tests/data/acpi/aarch64/virt/PPTT.topology",
--
2.43.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2026-03-11 16:14 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-11 16:06 [PATCH v19 0/8] Specifying cache topology on ARM Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 1/8] target/arm/tcg: increase cache level for cpu=max Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 2/8] hw/core/machine: topology functions capabilities added Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 3/8] hw/arm/virt: add cache hierarchy to device tree Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 4/8] bios-tables-test: prepare to change ARM ACPI virt PPTT Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 5/8] acpi: Add parameters to pass cache descriptions to ACPI build_pptt() Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 6/8] hw/acpi: add cache hierarchy to pptt table Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 7/8] tests/qtest/bios-table-test: testing new ARM ACPI PPTT topology Alireza Sanaee via qemu development
2026-03-11 16:06 ` [PATCH v19 8/8] Update the ACPI tables based on new aml-build.c Alireza Sanaee via qemu development
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox