@samitouri / QOSamiQemu / commits / ba0b4e84ae

hw/riscv, target/riscv: move pmu fdt function to fdt-common.c

riscv_pmu_generate_fdt_node() can be moved to fdt_common.c since it has no PMU TCG internals. Signed-off-by: Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com> Reviewed-by: Alistair Francis <alistair.francis@wdc.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com> Message-ID: <20260703180538.3346781-4-daniel.barboza@oss.qualcomm.com> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>

Daniel Henrique Barboza committed Jul 3, 2026 at 15:05 UTC ba0b4e84aee186b5805f37bbd0abfee51193de79
5 files changed +53 -54
hw/riscv/fdt-common.c
+52
@@ -230,3 +230,55 @@ void create_fdt_plic(void *fdt, hwaddr addr, uint64_t size,
230 }
231 qemu_fdt_setprop_cell(fdt, nodename, "phandle", plic_phandle);
232 }
233 +
234 +/*
235 + * To keep it simple, any event can be mapped to any programmable counters in
236 + * QEMU. The generic cycle & instruction count events can also be monitored
237 + * using programmable counters. In that case, mcycle & minstret must continue
238 + * to provide the correct value as well. Heterogeneous PMU per hart is not
239 + * supported yet. Thus, number of counters are same across all harts.
240 + */
241 +void riscv_pmu_generate_fdt_node(void *fdt, uint32_t cmask, char *pmu_name)
242 +{
243 + uint32_t fdt_event_ctr_map[15] = {};
244 +
245 + /*
246 + * The event encoding is specified in the SBI specification
247 + * Event idx is a 20bits wide number encoded as follows:
248 + * event_idx[19:16] = type
249 + * event_idx[15:0] = code
250 + * The code field in cache events are encoded as follows:
251 + * event_idx.code[15:3] = cache_id
252 + * event_idx.code[2:1] = op_id
253 + * event_idx.code[0:0] = result_id
254 + */
255 +
256 + /* SBI_PMU_HW_CPU_CYCLES: 0x01 : type(0x00) */
257 + fdt_event_ctr_map[0] = cpu_to_be32(0x00000001);
258 + fdt_event_ctr_map[1] = cpu_to_be32(0x00000001);
259 + fdt_event_ctr_map[2] = cpu_to_be32(cmask | 1 << 0);
260 +
261 + /* SBI_PMU_HW_INSTRUCTIONS: 0x02 : type(0x00) */
262 + fdt_event_ctr_map[3] = cpu_to_be32(0x00000002);
263 + fdt_event_ctr_map[4] = cpu_to_be32(0x00000002);
264 + fdt_event_ctr_map[5] = cpu_to_be32(cmask | 1 << 2);
265 +
266 + /* SBI_PMU_HW_CACHE_DTLB : 0x03 READ : 0x00 MISS : 0x00 type(0x01) */
267 + fdt_event_ctr_map[6] = cpu_to_be32(0x00010019);
268 + fdt_event_ctr_map[7] = cpu_to_be32(0x00010019);
269 + fdt_event_ctr_map[8] = cpu_to_be32(cmask);
270 +
271 + /* SBI_PMU_HW_CACHE_DTLB : 0x03 WRITE : 0x01 MISS : 0x00 type(0x01) */
272 + fdt_event_ctr_map[9] = cpu_to_be32(0x0001001B);
273 + fdt_event_ctr_map[10] = cpu_to_be32(0x0001001B);
274 + fdt_event_ctr_map[11] = cpu_to_be32(cmask);
275 +
276 + /* SBI_PMU_HW_CACHE_ITLB : 0x04 READ : 0x00 MISS : 0x00 type(0x01) */
277 + fdt_event_ctr_map[12] = cpu_to_be32(0x00010021);
278 + fdt_event_ctr_map[13] = cpu_to_be32(0x00010021);
279 + fdt_event_ctr_map[14] = cpu_to_be32(cmask);
280 +
281 + /* This a OpenSBI specific DT property documented in OpenSBI docs */
282 + qemu_fdt_setprop(fdt, pmu_name, "riscv,event-to-mhpmcounters",
283 + fdt_event_ctr_map, sizeof(fdt_event_ctr_map));
284 +}
hw/riscv/tt_atlantis.c
-1
@@ -17,7 +17,6 @@
17 #include "hw/core/sysbus.h"
18
19 #include "target/riscv/cpu.h"
20 -#include "target/riscv/pmu.h"
20
21 #include "hw/riscv/boot.h"
22 #include "hw/riscv/fdt-common.h"
hw/riscv/virt.c
-1
@@ -30,7 +30,6 @@
30 #include "hw/char/serial-mm.h"
31 #include "target/riscv/cpu.h"
32 #include "hw/core/sysbus-fdt.h"
33 -#include "target/riscv/pmu.h"
33 #include "hw/riscv/riscv_hart.h"
34 #include "hw/riscv/iommu.h"
35 #include "hw/riscv/riscv-iommu-bits.h"
include/hw/riscv/fdt-common.h
+1
@@ -35,4 +35,5 @@ void create_fdt_plic(void *fdt, hwaddr addr, uint64_t size,
35 uint32_t addr_cells, uint32_t *plic_cells,
36 uint32_t cells_size, uint32_t ndev_sources,
37 bool numa_enabled, int socket);
38 +void riscv_pmu_generate_fdt_node(void *fdt, uint32_t cmask, char *pmu_name);
39 #endif
target/riscv/pmu.c
-52
@@ -27,58 +27,6 @@
27
28 #define RISCV_TIMEBASE_FREQ 1000000000 /* 1Ghz */
29
30 -/*
31 - * To keep it simple, any event can be mapped to any programmable counters in
32 - * QEMU. The generic cycle & instruction count events can also be monitored
33 - * using programmable counters. In that case, mcycle & minstret must continue
34 - * to provide the correct value as well. Heterogeneous PMU per hart is not
35 - * supported yet. Thus, number of counters are same across all harts.
36 - */
37 -void riscv_pmu_generate_fdt_node(void *fdt, uint32_t cmask, char *pmu_name)
38 -{
39 - uint32_t fdt_event_ctr_map[15] = {};
40 -
41 - /*
42 - * The event encoding is specified in the SBI specification
43 - * Event idx is a 20bits wide number encoded as follows:
44 - * event_idx[19:16] = type
45 - * event_idx[15:0] = code
46 - * The code field in cache events are encoded as follows:
47 - * event_idx.code[15:3] = cache_id
48 - * event_idx.code[2:1] = op_id
49 - * event_idx.code[0:0] = result_id
50 - */
51 -
52 - /* SBI_PMU_HW_CPU_CYCLES: 0x01 : type(0x00) */
53 - fdt_event_ctr_map[0] = cpu_to_be32(0x00000001);
54 - fdt_event_ctr_map[1] = cpu_to_be32(0x00000001);
55 - fdt_event_ctr_map[2] = cpu_to_be32(cmask | 1 << 0);
56 -
57 - /* SBI_PMU_HW_INSTRUCTIONS: 0x02 : type(0x00) */
58 - fdt_event_ctr_map[3] = cpu_to_be32(0x00000002);
59 - fdt_event_ctr_map[4] = cpu_to_be32(0x00000002);
60 - fdt_event_ctr_map[5] = cpu_to_be32(cmask | 1 << 2);
61 -
62 - /* SBI_PMU_HW_CACHE_DTLB : 0x03 READ : 0x00 MISS : 0x00 type(0x01) */
63 - fdt_event_ctr_map[6] = cpu_to_be32(0x00010019);
64 - fdt_event_ctr_map[7] = cpu_to_be32(0x00010019);
65 - fdt_event_ctr_map[8] = cpu_to_be32(cmask);
66 -
67 - /* SBI_PMU_HW_CACHE_DTLB : 0x03 WRITE : 0x01 MISS : 0x00 type(0x01) */
68 - fdt_event_ctr_map[9] = cpu_to_be32(0x0001001B);
69 - fdt_event_ctr_map[10] = cpu_to_be32(0x0001001B);
70 - fdt_event_ctr_map[11] = cpu_to_be32(cmask);
71 -
72 - /* SBI_PMU_HW_CACHE_ITLB : 0x04 READ : 0x00 MISS : 0x00 type(0x01) */
73 - fdt_event_ctr_map[12] = cpu_to_be32(0x00010021);
74 - fdt_event_ctr_map[13] = cpu_to_be32(0x00010021);
75 - fdt_event_ctr_map[14] = cpu_to_be32(cmask);
76 -
77 - /* This a OpenSBI specific DT property documented in OpenSBI docs */
78 - qemu_fdt_setprop(fdt, pmu_name, "riscv,event-to-mhpmcounters",
79 - fdt_event_ctr_map, sizeof(fdt_event_ctr_map));
80 -}
81 -
30 static bool riscv_pmu_counter_valid(RISCVCPU *cpu, uint32_t ctr_idx)
31 {
32 if (ctr_idx < 3 || ctr_idx >= RV_MAX_MHPMCOUNTERS ||