| Message ID | 20240418192525.97451-23-philmd@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | include/exec: Rework (part 2) | expand |
On 4/19/24 00:55, Philippe Mathieu-Daudé wrote: > "exec/cpu-all.h" doesn't require "exec/tswap.h". Remove it, > including it in the sources when required. > > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> > --- > hw/xtensa/bootparam.h | 1 + > include/exec/cpu-all.h | 1 - > accel/tcg/translator.c | 1 + > hw/arm/boot.c | 1 + > hw/arm/npcm7xx.c | 1 + > hw/mips/fuloong2e.c | 1 + > hw/mips/malta.c | 1 + > hw/ppc/sam460ex.c | 1 + > hw/ppc/spapr.c | 1 + For spapr: Reviewed-by: Harsh Prateek Bora <harshpb@linux.ibm.com> > hw/ppc/virtex_ml507.c | 1 + > hw/sh4/r2d.c | 1 + > target/arm/gdbstub.c | 1 + > target/xtensa/xtensa-semi.c | 1 + > 13 files changed, 12 insertions(+), 1 deletion(-) > > diff --git a/hw/xtensa/bootparam.h b/hw/xtensa/bootparam.h > index f57ff850bc..e1d47b503c 100644 > --- a/hw/xtensa/bootparam.h > +++ b/hw/xtensa/bootparam.h > @@ -1,6 +1,7 @@ > #ifndef HW_XTENSA_BOOTPARAM_H > #define HW_XTENSA_BOOTPARAM_H > > +#include "exec/tswap.h" > #include "exec/cpu-common.h" > > #define BP_TAG_COMMAND_LINE 0x1001 /* command line (0-terminated string)*/ > diff --git a/include/exec/cpu-all.h b/include/exec/cpu-all.h > index 554b937ddb..cfbf51822c 100644 > --- a/include/exec/cpu-all.h > +++ b/include/exec/cpu-all.h > @@ -21,7 +21,6 @@ > > #include "exec/cpu-common.h" > #include "exec/memory.h" > -#include "exec/tswap.h" > #include "hw/core/cpu.h" > > /* some important defines: > diff --git a/accel/tcg/translator.c b/accel/tcg/translator.c > index 6832e55135..85950377d9 100644 > --- a/accel/tcg/translator.c > +++ b/accel/tcg/translator.c > @@ -12,6 +12,7 @@ > #include "qemu/error-report.h" > #include "exec/exec-all.h" > #include "exec/translator.h" > +#include "exec/tswap.h" > #include "exec/cpu_ldst.h" > #include "exec/plugin-gen.h" > #include "tcg/tcg-op-common.h" > diff --git a/hw/arm/boot.c b/hw/arm/boot.c > index 84ea6a807a..93945a1a15 100644 > --- a/hw/arm/boot.c > +++ b/hw/arm/boot.c > @@ -22,6 +22,7 @@ > #include "sysemu/reset.h" > #include "hw/loader.h" > #include "elf.h" > +#include "exec/tswap.h" > #include "sysemu/device_tree.h" > #include "qemu/config-file.h" > #include "qemu/option.h" > diff --git a/hw/arm/npcm7xx.c b/hw/arm/npcm7xx.c > index cc68b5d8f1..1ef303415b 100644 > --- a/hw/arm/npcm7xx.c > +++ b/hw/arm/npcm7xx.c > @@ -27,6 +27,7 @@ > #include "qemu/units.h" > #include "sysemu/sysemu.h" > #include "target/arm/cpu-qom.h" > +#include "exec/tswap.h" > > /* > * This covers the whole MMIO space. We'll use this to catch any MMIO accesses > diff --git a/hw/mips/fuloong2e.c b/hw/mips/fuloong2e.c > index a45aac368c..1d0613a76f 100644 > --- a/hw/mips/fuloong2e.c > +++ b/hw/mips/fuloong2e.c > @@ -40,6 +40,7 @@ > #include "sysemu/reset.h" > #include "sysemu/sysemu.h" > #include "qemu/error-report.h" > +#include "exec/tswap.h" > > #define ENVP_PADDR 0x2000 > #define ENVP_VADDR cpu_mips_phys_to_kseg0(NULL, ENVP_PADDR) > diff --git a/hw/mips/malta.c b/hw/mips/malta.c > index af74008c82..3dca0f100c 100644 > --- a/hw/mips/malta.c > +++ b/hw/mips/malta.c > @@ -56,6 +56,7 @@ > #include "semihosting/semihost.h" > #include "hw/mips/cps.h" > #include "hw/qdev-clock.h" > +#include "exec/tswap.h" > #include "target/mips/internal.h" > #include "trace.h" > #include "cpu.h" > diff --git a/hw/ppc/sam460ex.c b/hw/ppc/sam460ex.c > index d42b677898..abc02f0817 100644 > --- a/hw/ppc/sam460ex.c > +++ b/hw/ppc/sam460ex.c > @@ -24,6 +24,7 @@ > #include "hw/loader.h" > #include "elf.h" > #include "exec/memory.h" > +#include "exec/tswap.h" > #include "ppc440.h" > #include "hw/pci-host/ppc4xx.h" > #include "hw/block/flash.h" > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c > index e9bc97fee0..b4b1f43983 100644 > --- a/hw/ppc/spapr.c > +++ b/hw/ppc/spapr.c > @@ -74,6 +74,7 @@ > #include "hw/virtio/virtio-scsi.h" > #include "hw/virtio/vhost-scsi-common.h" > > +#include "exec/tswap.h" > #include "exec/ram_addr.h" > #include "hw/usb.h" > #include "qemu/config-file.h" > diff --git a/hw/ppc/virtex_ml507.c b/hw/ppc/virtex_ml507.c > index d02f330650..fd23afebf5 100644 > --- a/hw/ppc/virtex_ml507.c > +++ b/hw/ppc/virtex_ml507.c > @@ -38,6 +38,7 @@ > #include "qapi/error.h" > #include "qemu/error-report.h" > #include "qemu/option.h" > +#include "exec/tswap.h" > > #include "hw/intc/ppc-uic.h" > #include "hw/ppc/ppc.h" > diff --git a/hw/sh4/r2d.c b/hw/sh4/r2d.c > index e5ac6751bd..5f4420f534 100644 > --- a/hw/sh4/r2d.c > +++ b/hw/sh4/r2d.c > @@ -43,6 +43,7 @@ > #include "hw/loader.h" > #include "hw/usb.h" > #include "hw/block/flash.h" > +#include "exec/tswap.h" > > #define FLASH_BASE 0x00000000 > #define FLASH_SIZE (16 * MiB) > diff --git a/target/arm/gdbstub.c b/target/arm/gdbstub.c > index a3bb73cfa7..f2b001afdd 100644 > --- a/target/arm/gdbstub.c > +++ b/target/arm/gdbstub.c > @@ -20,6 +20,7 @@ > #include "qemu/osdep.h" > #include "cpu.h" > #include "exec/gdbstub.h" > +#include "exec/tswap.h" > #include "gdbstub/helpers.h" > #include "sysemu/tcg.h" > #include "internals.h" > diff --git a/target/xtensa/xtensa-semi.c b/target/xtensa/xtensa-semi.c > index fa21b7e11f..dbc42d1587 100644 > --- a/target/xtensa/xtensa-semi.c > +++ b/target/xtensa/xtensa-semi.c > @@ -29,6 +29,7 @@ > #include "cpu.h" > #include "chardev/char-fe.h" > #include "exec/helper-proto.h" > +#include "exec/tswap.h" > #include "semihosting/semihost.h" > #include "qapi/error.h" > #include "qemu/log.h"
diff --git a/hw/xtensa/bootparam.h b/hw/xtensa/bootparam.h index f57ff850bc..e1d47b503c 100644 --- a/hw/xtensa/bootparam.h +++ b/hw/xtensa/bootparam.h @@ -1,6 +1,7 @@ #ifndef HW_XTENSA_BOOTPARAM_H #define HW_XTENSA_BOOTPARAM_H +#include "exec/tswap.h" #include "exec/cpu-common.h" #define BP_TAG_COMMAND_LINE 0x1001 /* command line (0-terminated string)*/ diff --git a/include/exec/cpu-all.h b/include/exec/cpu-all.h index 554b937ddb..cfbf51822c 100644 --- a/include/exec/cpu-all.h +++ b/include/exec/cpu-all.h @@ -21,7 +21,6 @@ #include "exec/cpu-common.h" #include "exec/memory.h" -#include "exec/tswap.h" #include "hw/core/cpu.h" /* some important defines: diff --git a/accel/tcg/translator.c b/accel/tcg/translator.c index 6832e55135..85950377d9 100644 --- a/accel/tcg/translator.c +++ b/accel/tcg/translator.c @@ -12,6 +12,7 @@ #include "qemu/error-report.h" #include "exec/exec-all.h" #include "exec/translator.h" +#include "exec/tswap.h" #include "exec/cpu_ldst.h" #include "exec/plugin-gen.h" #include "tcg/tcg-op-common.h" diff --git a/hw/arm/boot.c b/hw/arm/boot.c index 84ea6a807a..93945a1a15 100644 --- a/hw/arm/boot.c +++ b/hw/arm/boot.c @@ -22,6 +22,7 @@ #include "sysemu/reset.h" #include "hw/loader.h" #include "elf.h" +#include "exec/tswap.h" #include "sysemu/device_tree.h" #include "qemu/config-file.h" #include "qemu/option.h" diff --git a/hw/arm/npcm7xx.c b/hw/arm/npcm7xx.c index cc68b5d8f1..1ef303415b 100644 --- a/hw/arm/npcm7xx.c +++ b/hw/arm/npcm7xx.c @@ -27,6 +27,7 @@ #include "qemu/units.h" #include "sysemu/sysemu.h" #include "target/arm/cpu-qom.h" +#include "exec/tswap.h" /* * This covers the whole MMIO space. We'll use this to catch any MMIO accesses diff --git a/hw/mips/fuloong2e.c b/hw/mips/fuloong2e.c index a45aac368c..1d0613a76f 100644 --- a/hw/mips/fuloong2e.c +++ b/hw/mips/fuloong2e.c @@ -40,6 +40,7 @@ #include "sysemu/reset.h" #include "sysemu/sysemu.h" #include "qemu/error-report.h" +#include "exec/tswap.h" #define ENVP_PADDR 0x2000 #define ENVP_VADDR cpu_mips_phys_to_kseg0(NULL, ENVP_PADDR) diff --git a/hw/mips/malta.c b/hw/mips/malta.c index af74008c82..3dca0f100c 100644 --- a/hw/mips/malta.c +++ b/hw/mips/malta.c @@ -56,6 +56,7 @@ #include "semihosting/semihost.h" #include "hw/mips/cps.h" #include "hw/qdev-clock.h" +#include "exec/tswap.h" #include "target/mips/internal.h" #include "trace.h" #include "cpu.h" diff --git a/hw/ppc/sam460ex.c b/hw/ppc/sam460ex.c index d42b677898..abc02f0817 100644 --- a/hw/ppc/sam460ex.c +++ b/hw/ppc/sam460ex.c @@ -24,6 +24,7 @@ #include "hw/loader.h" #include "elf.h" #include "exec/memory.h" +#include "exec/tswap.h" #include "ppc440.h" #include "hw/pci-host/ppc4xx.h" #include "hw/block/flash.h" diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index e9bc97fee0..b4b1f43983 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -74,6 +74,7 @@ #include "hw/virtio/virtio-scsi.h" #include "hw/virtio/vhost-scsi-common.h" +#include "exec/tswap.h" #include "exec/ram_addr.h" #include "hw/usb.h" #include "qemu/config-file.h" diff --git a/hw/ppc/virtex_ml507.c b/hw/ppc/virtex_ml507.c index d02f330650..fd23afebf5 100644 --- a/hw/ppc/virtex_ml507.c +++ b/hw/ppc/virtex_ml507.c @@ -38,6 +38,7 @@ #include "qapi/error.h" #include "qemu/error-report.h" #include "qemu/option.h" +#include "exec/tswap.h" #include "hw/intc/ppc-uic.h" #include "hw/ppc/ppc.h" diff --git a/hw/sh4/r2d.c b/hw/sh4/r2d.c index e5ac6751bd..5f4420f534 100644 --- a/hw/sh4/r2d.c +++ b/hw/sh4/r2d.c @@ -43,6 +43,7 @@ #include "hw/loader.h" #include "hw/usb.h" #include "hw/block/flash.h" +#include "exec/tswap.h" #define FLASH_BASE 0x00000000 #define FLASH_SIZE (16 * MiB) diff --git a/target/arm/gdbstub.c b/target/arm/gdbstub.c index a3bb73cfa7..f2b001afdd 100644 --- a/target/arm/gdbstub.c +++ b/target/arm/gdbstub.c @@ -20,6 +20,7 @@ #include "qemu/osdep.h" #include "cpu.h" #include "exec/gdbstub.h" +#include "exec/tswap.h" #include "gdbstub/helpers.h" #include "sysemu/tcg.h" #include "internals.h" diff --git a/target/xtensa/xtensa-semi.c b/target/xtensa/xtensa-semi.c index fa21b7e11f..dbc42d1587 100644 --- a/target/xtensa/xtensa-semi.c +++ b/target/xtensa/xtensa-semi.c @@ -29,6 +29,7 @@ #include "cpu.h" #include "chardev/char-fe.h" #include "exec/helper-proto.h" +#include "exec/tswap.h" #include "semihosting/semihost.h" #include "qapi/error.h" #include "qemu/log.h"
"exec/cpu-all.h" doesn't require "exec/tswap.h". Remove it, including it in the sources when required. Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> --- hw/xtensa/bootparam.h | 1 + include/exec/cpu-all.h | 1 - accel/tcg/translator.c | 1 + hw/arm/boot.c | 1 + hw/arm/npcm7xx.c | 1 + hw/mips/fuloong2e.c | 1 + hw/mips/malta.c | 1 + hw/ppc/sam460ex.c | 1 + hw/ppc/spapr.c | 1 + hw/ppc/virtex_ml507.c | 1 + hw/sh4/r2d.c | 1 + target/arm/gdbstub.c | 1 + target/xtensa/xtensa-semi.c | 1 + 13 files changed, 12 insertions(+), 1 deletion(-)