On 5/17/21 13:11, Philippe Mathieu-Daudé wrote: > Guard declarations within hwaddr.h against inclusion > from user-mode emulation. > > To make it clearer this header is sysemu specific, > move it to the sysemu/ directory.
Hi Philippe, do we need include/exec/sysemu/... .h as opposed to just use the existing include/sysemu/ ? Thanks, Claudio > > Patch created mechanically using: > > $ sed -i s,exec/hwaddr.h,exec/sysemu/hwaddr.h, $(git grep -l exec/hwaddr.h) > > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > hw/audio/lm4549.h | 2 +- > hw/net/can/can_sja1000.h | 2 +- > hw/net/can/ctucan_core.h | 2 +- > hw/net/net_tx_pkt.h | 2 +- > include/disas/disas.h | 4 +++- > include/exec/cpu-all.h | 2 +- > include/exec/cpu-common.h | 2 +- > include/exec/cpu-defs.h | 2 +- > include/exec/memory.h | 2 +- > include/exec/{ => sysemu}/hwaddr.h | 7 +++++-- > include/hw/arm/sharpsl.h | 2 +- > include/hw/arm/soc_dma.h | 2 +- > include/hw/arm/sysbus-fdt.h | 2 +- > include/hw/arm/virt.h | 2 +- > include/hw/block/block.h | 2 +- > include/hw/block/fdc.h | 2 +- > include/hw/block/flash.h | 2 +- > include/hw/core/cpu.h | 4 +++- > include/hw/cris/etraxfs_dma.h | 2 +- > include/hw/display/vga.h | 2 +- > include/hw/i386/microvm.h | 2 +- > include/hw/i386/x86.h | 2 +- > include/hw/input/lasips2.h | 2 +- > include/hw/loader-fit.h | 2 +- > include/hw/misc/allwinner-h3-dramc.h | 2 +- > include/hw/misc/empty_slot.h | 2 +- > include/hw/nvram/fw_cfg.h | 2 +- > include/hw/pci-host/gpex.h | 2 +- > include/hw/remote/memory.h | 2 +- > include/hw/remote/mpqemu-link.h | 2 +- > include/hw/rtc/m48t59.h | 2 +- > include/hw/rtc/sun4v-rtc.h | 2 +- > include/hw/timer/tmu012.h | 2 +- > include/hw/virtio/virtio-access.h | 2 +- > include/monitor/monitor.h | 2 +- > include/qemu/accel.h | 4 +++- > include/qemu/iova-tree.h | 2 +- > include/qemu/userfaultfd.h | 2 +- > dump/dump.c | 2 +- > dump/win_dump.c | 2 +- > hw/arm/sbsa-ref.c | 2 +- > hw/input/lasips2.c | 2 +- > hw/m68k/next-cube.c | 2 +- > hw/ppc/pnv_homer.c | 2 +- > tests/qtest/microbit-test.c | 2 +- > MAINTAINERS | 1 + > scripts/codeconverter/codeconverter/test_regexps.py | 4 ++-- > 47 files changed, 58 insertions(+), 48 deletions(-) > rename include/exec/{ => sysemu}/hwaddr.h (81%) > > diff --git a/hw/audio/lm4549.h b/hw/audio/lm4549.h > index aba9bb5b077..5d53c2f2179 100644 > --- a/hw/audio/lm4549.h > +++ b/hw/audio/lm4549.h > @@ -13,7 +13,7 @@ > #define HW_LM4549_H > > #include "audio/audio.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > typedef void (*lm4549_callback)(void *opaque); > > diff --git a/hw/net/can/can_sja1000.h b/hw/net/can/can_sja1000.h > index 7ca9cd681ed..57e6d4d34e4 100644 > --- a/hw/net/can/can_sja1000.h > +++ b/hw/net/can/can_sja1000.h > @@ -27,7 +27,7 @@ > #ifndef HW_CAN_SJA1000_H > #define HW_CAN_SJA1000_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "net/can_emu.h" > > #define CAN_SJA_MEM_SIZE 128 > diff --git a/hw/net/can/ctucan_core.h b/hw/net/can/ctucan_core.h > index bbc09ae0678..c0e4beafba2 100644 > --- a/hw/net/can/ctucan_core.h > +++ b/hw/net/can/ctucan_core.h > @@ -28,7 +28,7 @@ > #ifndef HW_CAN_CTUCAN_CORE_H > #define HW_CAN_CTUCAN_CORE_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "net/can_emu.h" > > #ifndef HOST_WORDS_BIGENDIAN > diff --git a/hw/net/net_tx_pkt.h b/hw/net/net_tx_pkt.h > index 4ec8bbe9bd9..86548b4f613 100644 > --- a/hw/net/net_tx_pkt.h > +++ b/hw/net/net_tx_pkt.h > @@ -19,7 +19,7 @@ > #define NET_TX_PKT_H > > #include "net/eth.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > /* define to enable packet dump functions */ > /*#define NET_TX_PKT_DEBUG*/ > diff --git a/include/disas/disas.h b/include/disas/disas.h > index d363e95edeb..1b85d121a7a 100644 > --- a/include/disas/disas.h > +++ b/include/disas/disas.h > @@ -1,7 +1,9 @@ > #ifndef QEMU_DISAS_H > #define QEMU_DISAS_H > > -#include "exec/hwaddr.h" > +#ifndef CONFIG_USER_ONLY > +#include "exec/sysemu/hwaddr.h" > +#endif > > #ifdef NEED_CPU_H > #include "cpu.h" > diff --git a/include/exec/cpu-all.h b/include/exec/cpu-all.h > index 32cfb634c6a..3e7edddead5 100644 > --- a/include/exec/cpu-all.h > +++ b/include/exec/cpu-all.h > @@ -175,7 +175,7 @@ extern unsigned long reserved_va; > > #else > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > #define SUFFIX > #define ARG1 as > diff --git a/include/exec/cpu-common.h b/include/exec/cpu-common.h > index ccabed4003a..1a64b0b5ac6 100644 > --- a/include/exec/cpu-common.h > +++ b/include/exec/cpu-common.h > @@ -4,7 +4,7 @@ > /* CPU interfaces that are target independent. */ > > #ifndef CONFIG_USER_ONLY > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #endif > > /* The CPU list lock nests outside page_(un)lock or mmap_(un)lock */ > diff --git a/include/exec/cpu-defs.h b/include/exec/cpu-defs.h > index ba3cd32a1ec..52a2b93493d 100644 > --- a/include/exec/cpu-defs.h > +++ b/include/exec/cpu-defs.h > @@ -26,7 +26,7 @@ > #include "qemu/host-utils.h" > #include "qemu/thread.h" > #ifndef CONFIG_USER_ONLY > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #endif > #include "exec/memattrs.h" > #include "hw/core/cpu.h" > diff --git a/include/exec/memory.h b/include/exec/memory.h > index c8b90889241..52a2659b396 100644 > --- a/include/exec/memory.h > +++ b/include/exec/memory.h > @@ -17,7 +17,7 @@ > #ifndef CONFIG_USER_ONLY > > #include "exec/cpu-common.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "exec/memattrs.h" > #include "exec/memop.h" > #include "exec/ramlist.h" > diff --git a/include/exec/hwaddr.h b/include/exec/sysemu/hwaddr.h > similarity index 81% > rename from include/exec/hwaddr.h > rename to include/exec/sysemu/hwaddr.h > index 8f16d179a88..9693cd516b4 100644 > --- a/include/exec/hwaddr.h > +++ b/include/exec/sysemu/hwaddr.h > @@ -1,8 +1,9 @@ > /* Define hwaddr if it exists. */ > > -#ifndef HWADDR_H > -#define HWADDR_H > +#ifndef EXEC_SYSEMU_HWADDR_H > +#define EXEC_SYSEMU_HWADDR_H > > +#ifndef CONFIG_USER_ONLY > > #define HWADDR_BITS 64 > /* hwaddr is the type of a physical address (its size can > @@ -23,4 +24,6 @@ typedef struct MemMapEntry { > hwaddr size; > } MemMapEntry; > > +#endif /* !CONFIG_USER_ONLY */ > + > #endif > diff --git a/include/hw/arm/sharpsl.h b/include/hw/arm/sharpsl.h > index e986b28c527..36ed1df72a7 100644 > --- a/include/hw/arm/sharpsl.h > +++ b/include/hw/arm/sharpsl.h > @@ -7,7 +7,7 @@ > #ifndef QEMU_SHARPSL_H > #define QEMU_SHARPSL_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > /* zaurus.c */ > > diff --git a/include/hw/arm/soc_dma.h b/include/hw/arm/soc_dma.h > index e93a7499a80..14b802c2330 100644 > --- a/include/hw/arm/soc_dma.h > +++ b/include/hw/arm/soc_dma.h > @@ -21,7 +21,7 @@ > #ifndef HW_SOC_DMA_H > #define HW_SOC_DMA_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > struct soc_dma_s; > struct soc_dma_ch_s; > diff --git a/include/hw/arm/sysbus-fdt.h b/include/hw/arm/sysbus-fdt.h > index 340c382cdde..7a8ace37b4c 100644 > --- a/include/hw/arm/sysbus-fdt.h > +++ b/include/hw/arm/sysbus-fdt.h > @@ -24,7 +24,7 @@ > #ifndef HW_ARM_SYSBUS_FDT_H > #define HW_ARM_SYSBUS_FDT_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > /** > * platform_bus_add_all_fdt_nodes - create all the platform bus nodes > diff --git a/include/hw/arm/virt.h b/include/hw/arm/virt.h > index 921416f918b..504b0bb6e57 100644 > --- a/include/hw/arm/virt.h > +++ b/include/hw/arm/virt.h > @@ -30,7 +30,7 @@ > #ifndef QEMU_ARM_VIRT_H > #define QEMU_ARM_VIRT_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "qemu/notify.h" > #include "hw/boards.h" > #include "hw/arm/boot.h" > diff --git a/include/hw/block/block.h b/include/hw/block/block.h > index c172cbe65f1..0f36b1f0277 100644 > --- a/include/hw/block/block.h > +++ b/include/hw/block/block.h > @@ -11,7 +11,7 @@ > #ifndef HW_BLOCK_H > #define HW_BLOCK_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "qapi/qapi-types-block-core.h" > #include "hw/qdev-properties-system.h" > > diff --git a/include/hw/block/fdc.h b/include/hw/block/fdc.h > index 1ecca7cac7f..7f0fb3d3986 100644 > --- a/include/hw/block/fdc.h > +++ b/include/hw/block/fdc.h > @@ -1,7 +1,7 @@ > #ifndef HW_FDC_H > #define HW_FDC_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "qapi/qapi-types-block.h" > > /* fdc.c */ > diff --git a/include/hw/block/flash.h b/include/hw/block/flash.h > index 86d8363bb09..e184df092eb 100644 > --- a/include/hw/block/flash.h > +++ b/include/hw/block/flash.h > @@ -3,7 +3,7 @@ > > /* NOR flash devices */ > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "qom/object.h" > > /* pflash_cfi01.c */ > diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h > index e4328de8d41..7aa5c82fa20 100644 > --- a/include/hw/core/cpu.h > +++ b/include/hw/core/cpu.h > @@ -22,7 +22,9 @@ > > #include "hw/qdev-core.h" > #include "disas/dis-asm.h" > -#include "exec/hwaddr.h" > +#if !defined(CONFIG_USER_ONLY) > +#include "exec/sysemu/hwaddr.h" > +#endif > #include "exec/memattrs.h" > #include "qapi/qapi-types-run-state.h" > #include "qemu/bitmap.h" > diff --git a/include/hw/cris/etraxfs_dma.h b/include/hw/cris/etraxfs_dma.h > index 095d76b9560..f0a4038c8ae 100644 > --- a/include/hw/cris/etraxfs_dma.h > +++ b/include/hw/cris/etraxfs_dma.h > @@ -1,7 +1,7 @@ > #ifndef HW_ETRAXFS_DMA_H > #define HW_ETRAXFS_DMA_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > struct dma_context_metadata { > /* data descriptor md */ > diff --git a/include/hw/display/vga.h b/include/hw/display/vga.h > index 5f7825e0e36..d63e0bb9c94 100644 > --- a/include/hw/display/vga.h > +++ b/include/hw/display/vga.h > @@ -9,7 +9,7 @@ > #ifndef QEMU_HW_DISPLAY_VGA_H > #define QEMU_HW_DISPLAY_VGA_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > /* > * modules can reference this symbol to avoid being loaded > diff --git a/include/hw/i386/microvm.h b/include/hw/i386/microvm.h > index f25f8374413..e6931917ed1 100644 > --- a/include/hw/i386/microvm.h > +++ b/include/hw/i386/microvm.h > @@ -19,7 +19,7 @@ > #define HW_I386_MICROVM_H > > #include "qemu-common.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "qemu/notify.h" > > #include "hw/boards.h" > diff --git a/include/hw/i386/x86.h b/include/hw/i386/x86.h > index c09b648dff2..b594f829cdb 100644 > --- a/include/hw/i386/x86.h > +++ b/include/hw/i386/x86.h > @@ -18,7 +18,7 @@ > #define HW_I386_X86_H > > #include "qemu-common.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "qemu/notify.h" > > #include "hw/i386/topology.h" > diff --git a/include/hw/input/lasips2.h b/include/hw/input/lasips2.h > index 0cd7b59064a..f4c1e787d2b 100644 > --- a/include/hw/input/lasips2.h > +++ b/include/hw/input/lasips2.h > @@ -7,7 +7,7 @@ > #ifndef HW_INPUT_LASIPS2_H > #define HW_INPUT_LASIPS2_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > #define TYPE_LASIPS2 "lasips2" > > diff --git a/include/hw/loader-fit.h b/include/hw/loader-fit.h > index 0832e379dc9..138da77d664 100644 > --- a/include/hw/loader-fit.h > +++ b/include/hw/loader-fit.h > @@ -20,7 +20,7 @@ > #ifndef HW_LOADER_FIT_H > #define HW_LOADER_FIT_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > struct fit_loader_match { > const char *compatible; > diff --git a/include/hw/misc/allwinner-h3-dramc.h > b/include/hw/misc/allwinner-h3-dramc.h > index 0b6c877ef74..93e640b23af 100644 > --- a/include/hw/misc/allwinner-h3-dramc.h > +++ b/include/hw/misc/allwinner-h3-dramc.h > @@ -22,7 +22,7 @@ > > #include "qom/object.h" > #include "hw/sysbus.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > /** > * Constants > diff --git a/include/hw/misc/empty_slot.h b/include/hw/misc/empty_slot.h > index dec56e56ae4..d942aebd3e7 100644 > --- a/include/hw/misc/empty_slot.h > +++ b/include/hw/misc/empty_slot.h > @@ -12,7 +12,7 @@ > #ifndef HW_EMPTY_SLOT_H > #define HW_EMPTY_SLOT_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > void empty_slot_init(const char *name, hwaddr addr, uint64_t slot_size); > > diff --git a/include/hw/nvram/fw_cfg.h b/include/hw/nvram/fw_cfg.h > index 0e7a8bc7af2..af1f8df9cf0 100644 > --- a/include/hw/nvram/fw_cfg.h > +++ b/include/hw/nvram/fw_cfg.h > @@ -1,7 +1,7 @@ > #ifndef FW_CFG_H > #define FW_CFG_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "standard-headers/linux/qemu_fw_cfg.h" > #include "hw/sysbus.h" > #include "sysemu/dma.h" > diff --git a/include/hw/pci-host/gpex.h b/include/hw/pci-host/gpex.h > index fcf8b638200..52da437057e 100644 > --- a/include/hw/pci-host/gpex.h > +++ b/include/hw/pci-host/gpex.h > @@ -20,7 +20,7 @@ > #ifndef HW_GPEX_H > #define HW_GPEX_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "hw/sysbus.h" > #include "hw/pci/pci.h" > #include "hw/pci/pcie_host.h" > diff --git a/include/hw/remote/memory.h b/include/hw/remote/memory.h > index bc2e30945f5..200fc982a00 100644 > --- a/include/hw/remote/memory.h > +++ b/include/hw/remote/memory.h > @@ -11,7 +11,7 @@ > #ifndef REMOTE_MEMORY_H > #define REMOTE_MEMORY_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "hw/remote/mpqemu-link.h" > > void remote_sysmem_reconfig(MPQemuMsg *msg, Error **errp); > diff --git a/include/hw/remote/mpqemu-link.h b/include/hw/remote/mpqemu-link.h > index 4ec09158851..9b56b30a311 100644 > --- a/include/hw/remote/mpqemu-link.h > +++ b/include/hw/remote/mpqemu-link.h > @@ -14,7 +14,7 @@ > #include "qom/object.h" > #include "qemu/thread.h" > #include "io/channel.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "io/channel-socket.h" > #include "hw/remote/proxy.h" > > diff --git a/include/hw/rtc/m48t59.h b/include/hw/rtc/m48t59.h > index d9b45eb1612..dda5ef4a20a 100644 > --- a/include/hw/rtc/m48t59.h > +++ b/include/hw/rtc/m48t59.h > @@ -26,7 +26,7 @@ > #ifndef HW_RTC_M48T59_H > #define HW_RTC_M48T59_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "qom/object.h" > > #define TYPE_NVRAM "nvram" > diff --git a/include/hw/rtc/sun4v-rtc.h b/include/hw/rtc/sun4v-rtc.h > index fd868f6ed2f..b2e27447121 100644 > --- a/include/hw/rtc/sun4v-rtc.h > +++ b/include/hw/rtc/sun4v-rtc.h > @@ -12,7 +12,7 @@ > #ifndef HW_RTC_SUN4V > #define HW_RTC_SUN4V > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > void sun4v_rtc_init(hwaddr addr); > > diff --git a/include/hw/timer/tmu012.h b/include/hw/timer/tmu012.h > index 808ed8de1d7..ea98ff0b3f1 100644 > --- a/include/hw/timer/tmu012.h > +++ b/include/hw/timer/tmu012.h > @@ -9,7 +9,7 @@ > #ifndef HW_TIMER_TMU012_H > #define HW_TIMER_TMU012_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > #define TMU012_FEAT_TOCR (1 << 0) > #define TMU012_FEAT_3CHAN (1 << 1) > diff --git a/include/hw/virtio/virtio-access.h > b/include/hw/virtio/virtio-access.h > index 6818a23a2d3..b18e0109d9f 100644 > --- a/include/hw/virtio/virtio-access.h > +++ b/include/hw/virtio/virtio-access.h > @@ -16,7 +16,7 @@ > #ifndef QEMU_VIRTIO_ACCESS_H > #define QEMU_VIRTIO_ACCESS_H > > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "hw/virtio/virtio.h" > #include "hw/virtio/virtio-bus.h" > > diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h > index af3887bb71d..b70fb361e3a 100644 > --- a/include/monitor/monitor.h > +++ b/include/monitor/monitor.h > @@ -4,7 +4,7 @@ > #include "block/block.h" > #include "qapi/qapi-types-misc.h" > #include "qemu/readline.h" > -#include "include/exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > typedef struct MonitorHMP MonitorHMP; > typedef struct MonitorOptions MonitorOptions; > diff --git a/include/qemu/accel.h b/include/qemu/accel.h > index 4f4c283f6fc..2fc5feddff2 100644 > --- a/include/qemu/accel.h > +++ b/include/qemu/accel.h > @@ -24,7 +24,9 @@ > #define QEMU_ACCEL_H > > #include "qom/object.h" > -#include "exec/hwaddr.h" > +#ifndef CONFIG_USER_ONLY > +#include "exec/sysemu/hwaddr.h" > +#endif > > typedef struct AccelState { > /*< private >*/ > diff --git a/include/qemu/iova-tree.h b/include/qemu/iova-tree.h > index b66cf93c4bc..1530cf0e95e 100644 > --- a/include/qemu/iova-tree.h > +++ b/include/qemu/iova-tree.h > @@ -24,7 +24,7 @@ > */ > > #include "exec/memory.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > #define IOVA_OK (0) > #define IOVA_ERR_INVALID (-1) /* Invalid parameters */ > diff --git a/include/qemu/userfaultfd.h b/include/qemu/userfaultfd.h > index 6b74f92792d..2cdce404edf 100644 > --- a/include/qemu/userfaultfd.h > +++ b/include/qemu/userfaultfd.h > @@ -14,7 +14,7 @@ > #define USERFAULTFD_H > > #include "qemu/osdep.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include <linux/userfaultfd.h> > > int uffd_query_features(uint64_t *features); > diff --git a/dump/dump.c b/dump/dump.c > index ab625909f30..544553d3579 100644 > --- a/dump/dump.c > +++ b/dump/dump.c > @@ -15,7 +15,7 @@ > #include "qemu-common.h" > #include "qemu/cutils.h" > #include "elf.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "monitor/monitor.h" > #include "sysemu/kvm.h" > #include "sysemu/dump.h" > diff --git a/dump/win_dump.c b/dump/win_dump.c > index c5eb5a9aacd..6c30c98fb09 100644 > --- a/dump/win_dump.c > +++ b/dump/win_dump.c > @@ -12,7 +12,7 @@ > #include "qemu-common.h" > #include "qemu/cutils.h" > #include "elf.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "monitor/monitor.h" > #include "sysemu/kvm.h" > #include "sysemu/dump.h" > diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c > index 43c19b49234..d01d93eb8f9 100644 > --- a/hw/arm/sbsa-ref.c > +++ b/hw/arm/sbsa-ref.c > @@ -27,7 +27,7 @@ > #include "sysemu/numa.h" > #include "sysemu/runstate.h" > #include "sysemu/sysemu.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "kvm_arm.h" > #include "hw/arm/boot.h" > #include "hw/block/flash.h" > diff --git a/hw/input/lasips2.c b/hw/input/lasips2.c > index e7faf24058b..f84f37411fd 100644 > --- a/hw/input/lasips2.c > +++ b/hw/input/lasips2.c > @@ -26,7 +26,7 @@ > #include "hw/qdev-properties.h" > #include "hw/input/ps2.h" > #include "hw/input/lasips2.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "trace.h" > #include "exec/address-spaces.h" > #include "migration/vmstate.h" > diff --git a/hw/m68k/next-cube.c b/hw/m68k/next-cube.c > index de951ffe5d3..84c346e7420 100644 > --- a/hw/m68k/next-cube.c > +++ b/hw/m68k/next-cube.c > @@ -10,7 +10,7 @@ > */ > > #include "qemu/osdep.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "sysemu/sysemu.h" > #include "sysemu/qtest.h" > #include "hw/irq.h" > diff --git a/hw/ppc/pnv_homer.c b/hw/ppc/pnv_homer.c > index 9a262629b73..394425232ed 100644 > --- a/hw/ppc/pnv_homer.c > +++ b/hw/ppc/pnv_homer.c > @@ -19,7 +19,7 @@ > #include "qemu/osdep.h" > #include "qemu/log.h" > #include "qapi/error.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "exec/memory.h" > #include "sysemu/cpus.h" > #include "hw/qdev-core.h" > diff --git a/tests/qtest/microbit-test.c b/tests/qtest/microbit-test.c > index 2b255579dfd..4d1c7fd863b 100644 > --- a/tests/qtest/microbit-test.c > +++ b/tests/qtest/microbit-test.c > @@ -15,7 +15,7 @@ > > > #include "qemu/osdep.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > #include "libqos/libqtest.h" > > #include "hw/arm/nrf51.h" > diff --git a/MAINTAINERS b/MAINTAINERS > index 78561a223f9..f1ae9934349 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -129,6 +129,7 @@ F: include/exec/cpu*.h > F: include/exec/exec-all.h > F: include/exec/helper*.h > F: include/exec/tb-hash.h > +F: include/exec/sysemu/hwaddr.h > F: include/sysemu/cpus.h > F: include/sysemu/tcg.h > F: include/hw/core/tcg-cpu-ops.h > diff --git a/scripts/codeconverter/codeconverter/test_regexps.py > b/scripts/codeconverter/codeconverter/test_regexps.py > index a445634d88a..41a2cd9c14c 100644 > --- a/scripts/codeconverter/codeconverter/test_regexps.py > +++ b/scripts/codeconverter/codeconverter/test_regexps.py > @@ -228,7 +228,7 @@ def test_initial_includes(): > /* NOR flash devices */ > > #include "qom/object.h" > -#include "exec/hwaddr.h" > +#include "exec/sysemu/hwaddr.h" > > /* pflash_cfi01.c */ > ''' > @@ -236,7 +236,7 @@ def test_initial_includes(): > m = InitialIncludes.domatch(c) > assert m > print(repr(m.group(0))) > - assert m.group(0).endswith('#include "exec/hwaddr.h"\n') > + assert m.group(0).endswith('#include "exec/sysemu/hwaddr.h"\n') > > c = '''#ifndef QEMU_VIRTIO_9P_H > #define QEMU_VIRTIO_9P_H