On 8/6/19 5:14 PM, Markus Armbruster wrote: > migration/qemu-file.h neglects to include it even though it needs > ram_addr_t. Fix that. Drop a few superfluous inclusions elsewhere. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > accel/stubs/tcg-stub.c | 1 - > balloon.c | 1 - > hw/smbios/smbios.c | 1 - > include/hw/hw.h | 1 - > include/hw/ppc/ppc4xx.h | 1 - > include/migration/misc.h | 1 - > include/sysemu/kvm_int.h | 1 - > include/sysemu/memory_mapping.h | 1 - > migration/migration.h | 1 - > migration/qemu-file-channel.c | 1 - > migration/qemu-file.h | 1 + > migration/vmstate-types.c | 1 - > qom/cpu.c | 1 - > tcg/optimize.c | 1 - > tcg/tcg-common.c | 1 - > tcg/tcg.c | 1 - > 16 files changed, 1 insertion(+), 15 deletions(-) > > diff --git a/accel/stubs/tcg-stub.c b/accel/stubs/tcg-stub.c > index 76ae461749..e2d23edafe 100644 > --- a/accel/stubs/tcg-stub.c > +++ b/accel/stubs/tcg-stub.c > @@ -14,7 +14,6 @@ > #include "qemu-common.h" > #include "cpu.h" > #include "tcg/tcg.h" > -#include "exec/cpu-common.h" > #include "exec/exec-all.h" > > void tb_flush(CPUState *cpu) > diff --git a/balloon.c b/balloon.c > index 914b3662db..f104b42961 100644 > --- a/balloon.c > +++ b/balloon.c > @@ -26,7 +26,6 @@ > > #include "qemu/osdep.h" > #include "qemu/atomic.h" > -#include "exec/cpu-common.h" > #include "sysemu/kvm.h" > #include "sysemu/balloon.h" > #include "trace-root.h" > diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c > index 7bcd67b098..11d476c4a2 100644 > --- a/hw/smbios/smbios.c > +++ b/hw/smbios/smbios.c > @@ -28,7 +28,6 @@ > #include "hw/firmware/smbios.h" > #include "hw/loader.h" > #include "hw/boards.h" > -#include "exec/cpu-common.h" > #include "smbios_build.h" > > /* legacy structures and constants for <= 2.0 machines */ > diff --git a/include/hw/hw.h b/include/hw/hw.h > index 38d2fb1f40..86ff26b712 100644 > --- a/include/hw/hw.h > +++ b/include/hw/hw.h > @@ -6,7 +6,6 @@ > #error Cannot include hw/hw.h from user emulation > #endif > > -#include "exec/cpu-common.h" > #include "qom/object.h" > #include "exec/memory.h" > #include "migration/vmstate.h" > diff --git a/include/hw/ppc/ppc4xx.h b/include/hw/ppc/ppc4xx.h > index 90f8866138..7d82259051 100644 > --- a/include/hw/ppc/ppc4xx.h > +++ b/include/hw/ppc/ppc4xx.h > @@ -26,7 +26,6 @@ > #define PPC4XX_H > > #include "hw/ppc/ppc.h" > -#include "exec/cpu-common.h" > #include "exec/memory.h" > > /* PowerPC 4xx core initialization */ > diff --git a/include/migration/misc.h b/include/migration/misc.h > index 5cdbabd094..b9d8e787af 100644 > --- a/include/migration/misc.h > +++ b/include/migration/misc.h > @@ -14,7 +14,6 @@ > #ifndef MIGRATION_MISC_H > #define MIGRATION_MISC_H > > -#include "exec/cpu-common.h" > #include "qemu/notify.h" > #include "qapi/qapi-types-net.h" > > diff --git a/include/sysemu/kvm_int.h b/include/sysemu/kvm_int.h > index 787dbc7770..5d341cc29b 100644 > --- a/include/sysemu/kvm_int.h > +++ b/include/sysemu/kvm_int.h > @@ -9,7 +9,6 @@ > #ifndef QEMU_KVM_INT_H > #define QEMU_KVM_INT_H > > -#include "exec/cpu-common.h" > #include "exec/memory.h" > #include "sysemu/sysemu.h" > #include "sysemu/accel.h" > diff --git a/include/sysemu/memory_mapping.h b/include/sysemu/memory_mapping.h > index 1b440df486..4b20f1a639 100644 > --- a/include/sysemu/memory_mapping.h > +++ b/include/sysemu/memory_mapping.h > @@ -15,7 +15,6 @@ > #define MEMORY_MAPPING_H > > #include "qemu/queue.h" > -#include "exec/cpu-common.h" > #include "exec/cpu-defs.h" > #include "exec/memory.h" > > diff --git a/migration/migration.h b/migration/migration.h > index 1fdd7b21fd..26f01d00f6 100644 > --- a/migration/migration.h > +++ b/migration/migration.h > @@ -16,7 +16,6 @@ > > #include "qapi/qapi-types-migration.h" > #include "qemu/thread.h" > -#include "exec/cpu-common.h" > #include "qemu/coroutine_int.h" > #include "hw/qdev.h" > #include "io/channel.h" > diff --git a/migration/qemu-file-channel.c b/migration/qemu-file-channel.c > index 8e639eb496..78ef248820 100644 > --- a/migration/qemu-file-channel.c > +++ b/migration/qemu-file-channel.c > @@ -24,7 +24,6 @@ > > #include "qemu/osdep.h" > #include "qemu-file-channel.h" > -#include "exec/cpu-common.h" > #include "qemu-file.h" > #include "io/channel-socket.h" > #include "qemu/iov.h" > diff --git a/migration/qemu-file.h b/migration/qemu-file.h > index 13baf896bd..21f3ae4be2 100644 > --- a/migration/qemu-file.h > +++ b/migration/qemu-file.h > @@ -26,6 +26,7 @@ > #define MIGRATION_QEMU_FILE_H > > #include <zlib.h> > +#include "exec/cpu-common.h" > > /* Read a chunk of data from a file at the given position. The pos argument > * can be ignored if the file is only be used for streaming. The number of > diff --git a/migration/vmstate-types.c b/migration/vmstate-types.c > index 845f4649ee..bee658a1b2 100644 > --- a/migration/vmstate-types.c > +++ b/migration/vmstate-types.c > @@ -11,7 +11,6 @@ > */ > > #include "qemu/osdep.h" > -#include "exec/cpu-common.h" > #include "qemu-file.h" > #include "migration.h" > #include "migration/vmstate.h" > diff --git a/qom/cpu.c b/qom/cpu.c > index f376f782d8..fdc1af8f40 100644 > --- a/qom/cpu.c > +++ b/qom/cpu.c > @@ -25,7 +25,6 @@ > #include "qemu/notify.h" > #include "qemu/log.h" > #include "exec/log.h" > -#include "exec/cpu-common.h" > #include "qemu/error-report.h" > #include "qemu/qemu-print.h" > #include "sysemu/sysemu.h" > diff --git a/tcg/optimize.c b/tcg/optimize.c > index d2424de4af..cee2a36a60 100644 > --- a/tcg/optimize.c > +++ b/tcg/optimize.c > @@ -24,7 +24,6 @@ > */ > > #include "qemu/osdep.h" > -#include "exec/cpu-common.h" > #include "tcg-op.h" > > #define CASE_OP_32_64(x) \ > diff --git a/tcg/tcg-common.c b/tcg/tcg-common.c > index 907d58d0f0..97305a3efc 100644 > --- a/tcg/tcg-common.c > +++ b/tcg/tcg-common.c > @@ -23,7 +23,6 @@ > */ > > #include "qemu/osdep.h" > -#include "exec/cpu-common.h" > #include "tcg/tcg.h" > > #if defined(CONFIG_TCG_INTERPRETER) > diff --git a/tcg/tcg.c b/tcg/tcg.c > index be2c33c400..2db83ce2e8 100644 > --- a/tcg/tcg.c > +++ b/tcg/tcg.c > @@ -42,7 +42,6 @@ > #define NO_CPU_IO_DEFS > #include "cpu.h" > > -#include "exec/cpu-common.h" > #include "exec/exec-all.h" > > #if !defined(CONFIG_USER_ONLY) >
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> Tested-by: Philippe Mathieu-Daudé <phi...@redhat.com>