On 13/02/2026 18.50, Philippe Mathieu-Daudé wrote:
- Avoid qemu_arch_available() in hw/
- Document qemu_arch_available(),
- define QEMU_ARCH_* based on SYS_EMU_TARGET_*, allowing
(via with TargetInfo API) to compile arch_init.c as common,
- restrict qemu_arch_available() scope to system/ sub-directory.
---
v4: New approach
v3: Return directly within switch
v2: Prefer switch over array (pbo)
Philippe Mathieu-Daudé (5):
qemu: Document qemu_arch_available() method
qemu/target_info: Add target_s390x() helper
hw/scsi/disk: Prefer target_s390x() over qemu_arch_available()
system: Make qemu_arch_available() common code
system: Reduce 'arch_init.h' scope
meson.build | 2 --
include/qemu/target-info.h | 7 ++++++
include/system/arch_init.h | 30 -----------------------
system/arch_init.h | 50 ++++++++++++++++++++++++++++++++++++++
hw/scsi/scsi-disk.c | 4 +--
system/arch_init.c | 6 +++--
target-info.c | 5 ++++
system/meson.build | 2 +-
8 files changed, 69 insertions(+), 37 deletions(-)
delete mode 100644 include/system/arch_init.h
create mode 100644 system/arch_init.h
Series
Reviewed-by: Thomas Huth <[email protected]>
... and since arch_init.c only contains one single small function, I think
you could even go one step further and merge that into another file in
system/ ... but that could also be done in a later patch.
Thomas