Include qapi/qmp/qerror.h exactly where needed In particular, don't include it into headers. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Reviewed-by: Luiz Capitulino <lcapitulino@redhat.com>
diff --git a/backends/hostmem.c b/backends/hostmem.c index 7d74be0..61c1ac0 100644 --- a/backends/hostmem.c +++ b/backends/hostmem.c
@@ -13,7 +13,6 @@ #include "qapi/visitor.h" #include "qapi-types.h" #include "qapi-visit.h" -#include "qapi/qmp/qerror.h" #include "qemu/config-file.h" #include "qom/object_interfaces.h"
diff --git a/balloon.c b/balloon.c index 5f5202c..617187a 100644 --- a/balloon.c +++ b/balloon.c
@@ -30,6 +30,7 @@ #include "sysemu/balloon.h" #include "trace.h" #include "qmp-commands.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/qjson.h" static QEMUBalloonEvent *balloon_event_fn;
diff --git a/block.c b/block.c index b07f987..e2e33fd 100644 --- a/block.c +++ b/block.c
@@ -28,6 +28,7 @@ #include "block/blockjob.h" #include "qemu/error-report.h" #include "qemu/module.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/qjson.h" #include "sysemu/block-backend.h" #include "sysemu/sysemu.h"
diff --git a/block/backup.c b/block/backup.c index 35145cb..4a1af68 100644 --- a/block/backup.c +++ b/block/backup.c
@@ -19,6 +19,7 @@ #include "block/block.h" #include "block/block_int.h" #include "block/blockjob.h" +#include "qapi/qmp/qerror.h" #include "qemu/ratelimit.h" #define BACKUP_CLUSTER_BITS 16
diff --git a/block/commit.c b/block/commit.c index 6c01fea..7312a5b 100644 --- a/block/commit.c +++ b/block/commit.c
@@ -15,6 +15,7 @@ #include "trace.h" #include "block/block_int.h" #include "block/blockjob.h" +#include "qapi/qmp/qerror.h" #include "qemu/ratelimit.h" enum {
diff --git a/block/mirror.c b/block/mirror.c index e3e51a2..048e452 100644 --- a/block/mirror.c +++ b/block/mirror.c
@@ -14,6 +14,7 @@ #include "trace.h" #include "block/blockjob.h" #include "block/block_int.h" +#include "qapi/qmp/qerror.h" #include "qemu/ratelimit.h" #include "qemu/bitmap.h"
diff --git a/block/qcow.c b/block/qcow.c index 54152f9..733627f 100644 --- a/block/qcow.c +++ b/block/qcow.c
@@ -25,6 +25,7 @@ #include "block/block_int.h" #include "qemu/module.h" #include <zlib.h> +#include "qapi/qmp/qerror.h" #include "qemu/aes.h" #include "migration/migration.h"
diff --git a/block/quorum.c b/block/quorum.c index 9aceabd..a7df17c 100644 --- a/block/quorum.c +++ b/block/quorum.c
@@ -18,6 +18,7 @@ #include "block/block_int.h" #include "qapi/qmp/qbool.h" #include "qapi/qmp/qdict.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/qint.h" #include "qapi/qmp/qjson.h" #include "qapi/qmp/qlist.h"
diff --git a/block/snapshot.c b/block/snapshot.c index 7cd960c..19395ae 100644 --- a/block/snapshot.c +++ b/block/snapshot.c
@@ -24,6 +24,7 @@ #include "block/snapshot.h" #include "block/block_int.h" +#include "qapi/qmp/qerror.h" QemuOptsList internal_snapshot_opts = { .name = "snapshot",
diff --git a/block/stream.c b/block/stream.c index f17efdb..ab0bd05 100644 --- a/block/stream.c +++ b/block/stream.c
@@ -14,6 +14,7 @@ #include "trace.h" #include "block/block_int.h" #include "block/blockjob.h" +#include "qapi/qmp/qerror.h" #include "qemu/ratelimit.h" enum {
diff --git a/block/vmdk.c b/block/vmdk.c index b7f3f21..fbaab67 100644 --- a/block/vmdk.c +++ b/block/vmdk.c
@@ -25,6 +25,7 @@ #include "qemu-common.h" #include "block/block_int.h" +#include "qapi/qmp/qerror.h" #include "qemu/error-report.h" #include "qemu/module.h" #include "migration/migration.h"
diff --git a/blockdev.c b/blockdev.c index fdc7768..b354676 100644 --- a/blockdev.c +++ b/blockdev.c
@@ -41,6 +41,7 @@ #include "qemu/config-file.h" #include "qapi/qmp/types.h" #include "qapi-visit.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp-output-visitor.h" #include "qapi/util.h" #include "sysemu/sysemu.h"
diff --git a/blockjob.c b/blockjob.c index b033f68..ec46fad 100644 --- a/blockjob.c +++ b/blockjob.c
@@ -29,6 +29,7 @@ #include "block/block.h" #include "block/blockjob.h" #include "block/block_int.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/qjson.h" #include "block/coroutine.h" #include "qmp-commands.h"
diff --git a/dump.c b/dump.c index 9a02b51..78b7d84 100644 --- a/dump.c +++ b/dump.c
@@ -23,6 +23,7 @@ #include "sysemu/memory_mapping.h" #include "sysemu/cpus.h" #include "qapi/error.h" +#include "qapi/qmp/qerror.h" #include "qmp-commands.h" #include <zlib.h>
diff --git a/hmp.c b/hmp.c index fd10ab5..070aaf8 100644 --- a/hmp.c +++ b/hmp.c
@@ -25,6 +25,7 @@ #include "monitor/monitor.h" #include "monitor/qdev.h" #include "qapi/opts-visitor.h" +#include "qapi/qmp/qerror.h" #include "qapi/string-output-visitor.h" #include "qapi-visit.h" #include "ui/console.h"
diff --git a/hw/char/serial-pci.c b/hw/char/serial-pci.c index 653064f..1c8b9be 100644 --- a/hw/char/serial-pci.c +++ b/hw/char/serial-pci.c
@@ -27,7 +27,6 @@ #include "hw/char/serial.h" #include "hw/pci/pci.h" -#include "qapi/qmp/qerror.h" #define PCI_SERIAL_MAX_PORTS 4
diff --git a/hw/dma/xilinx_axidma.c b/hw/dma/xilinx_axidma.c index d06002d..cf842a3 100644 --- a/hw/dma/xilinx_axidma.c +++ b/hw/dma/xilinx_axidma.c
@@ -26,7 +26,6 @@ #include "qemu/timer.h" #include "hw/ptimer.h" #include "qemu/log.h" -#include "qapi/qmp/qerror.h" #include "qemu/main-loop.h" #include "hw/stream.h"
diff --git a/hw/misc/ivshmem.c b/hw/misc/ivshmem.c index 53d19fc..cc76989 100644 --- a/hw/misc/ivshmem.c +++ b/hw/misc/ivshmem.c
@@ -22,7 +22,6 @@ #include "hw/pci/msix.h" #include "sysemu/kvm.h" #include "migration/migration.h" -#include "qapi/qmp/qerror.h" #include "qemu/error-report.h" #include "qemu/event_notifier.h" #include "qemu/fifo8.h"
diff --git a/hw/net/xilinx_axienet.c b/hw/net/xilinx_axienet.c index 21efedf..9205770 100644 --- a/hw/net/xilinx_axienet.c +++ b/hw/net/xilinx_axienet.c
@@ -26,7 +26,6 @@ #include "qemu/log.h" #include "net/net.h" #include "net/checksum.h" -#include "qapi/qmp/qerror.h" #include "hw/stream.h"
diff --git a/hw/pci/pci-stub.c b/hw/pci/pci-stub.c index 9b51d37..063a7c2 100644 --- a/hw/pci/pci-stub.c +++ b/hw/pci/pci-stub.c
@@ -20,6 +20,7 @@ #include "sysemu/sysemu.h" #include "monitor/monitor.h" +#include "qapi/qmp/qerror.h" #include "hw/pci/pci.h" #include "qmp-commands.h"
diff --git a/hw/pci/pcie.c b/hw/pci/pcie.c index 1463e65..14c7711 100644 --- a/hw/pci/pcie.c +++ b/hw/pci/pcie.c
@@ -26,7 +26,6 @@ #include "hw/pci/pci_bus.h" #include "hw/pci/pcie_regs.h" #include "qemu/range.h" -#include "qapi/qmp/qerror.h" //#define DEBUG_PCIE #ifdef DEBUG_PCIE
diff --git a/hw/pci/shpc.c b/hw/pci/shpc.c index a706486..bfb4d31 100644 --- a/hw/pci/shpc.c +++ b/hw/pci/shpc.c
@@ -7,7 +7,6 @@ #include "hw/pci/pci.h" #include "hw/pci/pci_bus.h" #include "hw/pci/msi.h" -#include "qapi/qmp/qerror.h" /* TODO: model power only and disabled slot states. */ /* TODO: handle SERR and wakeups */
diff --git a/hw/ppc/virtex_ml507.c b/hw/ppc/virtex_ml507.c index 6feb3c7..439732f 100644 --- a/hw/ppc/virtex_ml507.c +++ b/hw/ppc/virtex_ml507.c
@@ -41,7 +41,6 @@ #include "ppc405.h" #include "sysemu/block-backend.h" -#include "qapi/qmp/qerror.h" #define EPAPR_MAGIC (0x45504150) #define FLASH_SIZE (16 * 1024 * 1024)
diff --git a/hw/s390x/s390-virtio.c b/hw/s390x/s390-virtio.c index be2ba86..4a67c3f 100644 --- a/hw/s390x/s390-virtio.c +++ b/hw/s390x/s390-virtio.c
@@ -22,6 +22,7 @@ */ #include "hw/hw.h" +#include "qapi/qmp/qerror.h" #include "sysemu/block-backend.h" #include "sysemu/blockdev.h" #include "sysemu/sysemu.h"
diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c index 110e014..4e9528e 100644 --- a/hw/usb/redirect.c +++ b/hw/usb/redirect.c
@@ -29,6 +29,7 @@ #include "qemu/timer.h" #include "monitor/monitor.h" #include "sysemu/sysemu.h" +#include "qapi/qmp/qerror.h" #include "qemu/error-report.h" #include "qemu/iov.h" #include "sysemu/char.h"
diff --git a/hw/virtio/virtio-rng.c b/hw/virtio/virtio-rng.c index 22b1d87..740ed31 100644 --- a/hw/virtio/virtio-rng.c +++ b/hw/virtio/virtio-rng.c
@@ -11,7 +11,6 @@ #include "qemu/iov.h" #include "hw/qdev.h" -#include "qapi/qmp/qerror.h" #include "hw/virtio/virtio.h" #include "hw/virtio/virtio-rng.h" #include "sysemu/rng.h"
diff --git a/include/block/block_int.h b/include/block/block_int.h index 888ec09..a81542c 100644 --- a/include/block/block_int.h +++ b/include/block/block_int.h
@@ -31,7 +31,6 @@ #include "block/coroutine.h" #include "qemu/timer.h" #include "qapi-types.h" -#include "qapi/qmp/qerror.h" #include "monitor/monitor.h" #include "qemu/hbitmap.h" #include "block/snapshot.h"
diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h index aa83104..9aff47e 100644 --- a/include/monitor/monitor.h +++ b/include/monitor/monitor.h
@@ -2,7 +2,6 @@ #define MONITOR_H #include "qemu-common.h" -#include "qapi/qmp/qerror.h" #include "qapi/qmp/qdict.h" #include "block/block.h" #include "qemu/readline.h"
diff --git a/include/qemu/sockets.h b/include/qemu/sockets.h index 7992ece..c174b5c 100644 --- a/include/qemu/sockets.h +++ b/include/qemu/sockets.h
@@ -28,7 +28,6 @@ #include "qemu/option.h" #include "qapi/error.h" -#include "qapi/qmp/qerror.h" #include "qapi-types.h" extern QemuOptsList socket_optslist;
diff --git a/migration/migration.c b/migration/migration.c index 25f6830..d7056d7 100644 --- a/migration/migration.c +++ b/migration/migration.c
@@ -21,6 +21,7 @@ #include "migration/qemu-file.h" #include "sysemu/sysemu.h" #include "block/block.h" +#include "qapi/qmp/qerror.h" #include "qemu/sockets.h" #include "migration/block.h" #include "qemu/thread.h"
diff --git a/migration/savevm.c b/migration/savevm.c index 2b4e02f..9e0e286 100644 --- a/migration/savevm.c +++ b/migration/savevm.c
@@ -37,6 +37,7 @@ #include "qemu/timer.h" #include "audio/audio.h" #include "migration/migration.h" +#include "qapi/qmp/qerror.h" #include "qemu/error-report.h" #include "qemu/sockets.h" #include "qemu/queue.h"
diff --git a/monitor.c b/monitor.c index 497fed2..aeea2b5 100644 --- a/monitor.c +++ b/monitor.c
@@ -49,6 +49,7 @@ #include "sysemu/kvm.h" #include "qemu/acl.h" #include "sysemu/tpm.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/qint.h" #include "qapi/qmp/qfloat.h" #include "qapi/qmp/qlist.h"
diff --git a/net/net.c b/net/net.c index 19184e4..cc36c7b 100644 --- a/net/net.c +++ b/net/net.c
@@ -32,6 +32,7 @@ #include "monitor/monitor.h" #include "qemu-common.h" +#include "qapi/qmp/qerror.h" #include "qemu/error-report.h" #include "qemu/sockets.h" #include "qemu/config-file.h"
diff --git a/numa.c b/numa.c index 28c857c..91fc6c1 100644 --- a/numa.c +++ b/numa.c
@@ -31,7 +31,6 @@ #include "qapi-visit.h" #include "qapi/opts-visitor.h" #include "qapi/dealloc-visitor.h" -#include "qapi/qmp/qerror.h" #include "hw/boards.h" #include "sysemu/hostmem.h" #include "qmp-commands.h"
diff --git a/qapi/qmp-output-visitor.c b/qapi/qmp-output-visitor.c index 7e0f7ce..efc19d5 100644 --- a/qapi/qmp-output-visitor.c +++ b/qapi/qmp-output-visitor.c
@@ -16,7 +16,6 @@ #include "qemu/queue.h" #include "qemu-common.h" #include "qapi/qmp/types.h" -#include "qapi/qmp/qerror.h" typedef struct QStackEntry {
diff --git a/qapi/string-output-visitor.c b/qapi/string-output-visitor.c index 1ab8574..b86ce2c 100644 --- a/qapi/string-output-visitor.c +++ b/qapi/string-output-visitor.c
@@ -13,7 +13,6 @@ #include "qemu-common.h" #include "qapi/string-output-visitor.h" #include "qapi/visitor-impl.h" -#include "qapi/qmp/qerror.h" #include "qemu/host-utils.h" #include <math.h> #include "qemu/range.h"
diff --git a/qdev-monitor.c b/qdev-monitor.c index 66f6d0e..f9e2d62 100644 --- a/qdev-monitor.c +++ b/qdev-monitor.c
@@ -23,6 +23,7 @@ #include "monitor/qdev.h" #include "qmp-commands.h" #include "sysemu/arch_init.h" +#include "qapi/qmp/qerror.h" #include "qemu/config-file.h" #include "qemu/error-report.h"
diff --git a/qemu-img.c b/qemu-img.c index 6d1b136..75f4ee4 100644 --- a/qemu-img.c +++ b/qemu-img.c
@@ -23,6 +23,7 @@ */ #include "qapi-visit.h" #include "qapi/qmp-output-visitor.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/qjson.h" #include "qemu-common.h" #include "qemu/option.h"
diff --git a/qmp.c b/qmp.c index a046b68..03eba96 100644 --- a/qmp.c +++ b/qmp.c
@@ -24,6 +24,7 @@ #include "hw/qdev.h" #include "sysemu/blockdev.h" #include "qom/qom-qobject.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/qobject.h" #include "qapi/qmp-input-visitor.h" #include "hw/boards.h"
diff --git a/qobject/json-parser.c b/qobject/json-parser.c index 015d785..ac991ba 100644 --- a/qobject/json-parser.c +++ b/qobject/json-parser.c
@@ -22,7 +22,6 @@ #include "qapi/qmp/qbool.h" #include "qapi/qmp/json-parser.h" #include "qapi/qmp/json-lexer.h" -#include "qapi/qmp/qerror.h" typedef struct JSONParserContext {
diff --git a/scripts/qapi-commands.py b/scripts/qapi-commands.py index a451a51..ca22acc 100644 --- a/scripts/qapi-commands.py +++ b/scripts/qapi-commands.py
@@ -329,7 +329,6 @@ fdef.write(mcgen(''' #include "qemu-common.h" #include "qemu/module.h" -#include "qapi/qmp/qerror.h" #include "qapi/qmp/types.h" #include "qapi/qmp/dispatch.h" #include "qapi/visitor.h"
diff --git a/stubs/dump.c b/stubs/dump.c index fac7019..8c24eda 100644 --- a/stubs/dump.c +++ b/stubs/dump.c
@@ -13,7 +13,6 @@ #include "qemu-common.h" #include "sysemu/dump-arch.h" -#include "qapi/qmp/qerror.h" #include "qmp-commands.h" int cpu_get_dump_info(ArchDumpInfo *info,
diff --git a/target-arm/cpu.c b/target-arm/cpu.c index b3d07ac..80669a6 100644 --- a/target-arm/cpu.c +++ b/target-arm/cpu.c
@@ -22,7 +22,6 @@ #include "internals.h" #include "qemu-common.h" #include "hw/qdev-properties.h" -#include "qapi/qmp/qerror.h" #if !defined(CONFIG_USER_ONLY) #include "hw/loader.h" #endif
diff --git a/target-i386/kvm.c b/target-i386/kvm.c index 5a236e3..daced5c 100644 --- a/target-i386/kvm.c +++ b/target-i386/kvm.c
@@ -36,7 +36,6 @@ #include <asm/hyperv.h> #include "hw/pci/pci.h" #include "migration/migration.h" -#include "qapi/qmp/qerror.h" #include "exec/memattrs.h" //#define DEBUG_KVM
diff --git a/ui/vnc.c b/ui/vnc.c index af16584..2ffd9e5 100644 --- a/ui/vnc.c +++ b/ui/vnc.c
@@ -34,6 +34,7 @@ #include "qemu/timer.h" #include "qemu/acl.h" #include "qemu/config-file.h" +#include "qapi/qmp/qerror.h" #include "qapi/qmp/types.h" #include "qmp-commands.h" #include "qemu/osdep.h"
diff --git a/util/qemu-config.c b/util/qemu-config.c index 6cfdd72..5fcfd0e 100644 --- a/util/qemu-config.c +++ b/util/qemu-config.c
@@ -2,7 +2,6 @@ #include "qemu/error-report.h" #include "qemu/option.h" #include "qemu/config-file.h" -#include "qapi/qmp/qerror.h" #include "qapi/error.h" #include "qmp-commands.h"