Signed-off-by: Xuebing Wang <address@hidden>
---
arch_init.c | 1 -
cpu-exec.c | 7 ++++++-
cputlb.c | 6 +++++-
disas.c | 7 ++++++-
dump.c | 2 --
exec.c | 2 --
gdbstub.c | 8 +++++++-
kvm-stub.c | 1 -
memory_mapping.c | 6 +++---
monitor.c | 1 -
translate-all.c | 7 ++++++-
user-exec.c | 7 ++++++-
12 files changed, 39 insertions(+), 16 deletions(-)
diff --git a/arch_init.c b/arch_init.c
index fe17279..76e8630 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -47,7 +47,6 @@
#include "qemu/config-file.h"
#include "qmp-commands.h"
#include "trace.h"
-#include "exec/cpu-all.h"
#include "exec/ram_addr.h"
#include "hw/acpi/acpi.h"
#include "qemu/host-utils.h"
diff --git a/cpu-exec.c b/cpu-exec.c
index 1b0f617..858825f 100644
--- a/cpu-exec.c
+++ b/cpu-exec.c
@@ -16,8 +16,13 @@
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, see <http://www.gnu.org/licenses/>.
*/
+
+#ifndef NEED_CPU_H
+#error target-xxx/cpu.h must be included because target-specific are required
+#endif
+#include "cpu.h" /* target-xxx/cpu.h, required for CPUArchState etc */
+
#include "config.h"
-#include "cpu.h"
#include "disas/disas.h"
#include "tcg.h"
#include "qemu/atomic.h"
diff --git a/cputlb.c b/cputlb.c
index 0fbaa39..3c6cb16 100644
--- a/cputlb.c
+++ b/cputlb.c
@@ -17,8 +17,12 @@
* License along with this library; if not, see <http://www.gnu.org/licenses/>.
*/
+#ifndef NEED_CPU_H
+#error target-xxx/cpu.h must be included because target-specific are required
+#endif
+#include "cpu.h" /* target-xxx/cpu.h, required for CPUArchState etc */
+
#include "config.h"
-#include "cpu.h"
#include "exec/exec-all.h"
#include "exec/memory.h"
#include "exec/address-spaces.h"
diff --git a/disas.c b/disas.c
index 79e6944..8dc51c8 100644
--- a/disas.c
+++ b/disas.c
@@ -1,10 +1,15 @@
/* General "disassemble this chunk" code. Used for debugging. */
+
+#ifndef NEED_CPU_H
+#error target-xxx/cpu.h must be included because target-specific are required
+#endif
+#include "cpu.h" /* target-xxx/cpu.h, required for CPUArchState etc */
+
#include "config.h"
#include "disas/bfd.h"
#include "elf.h"
#include <errno.h>
-#include "cpu.h"
#include "disas/disas.h"
typedef struct CPUDebug {
diff --git a/dump.c b/dump.c
index 80a9116..d75c2ea 100644
--- a/dump.c
+++ b/dump.c
@@ -13,8 +13,6 @@
#include "qemu-common.h"
#include "elf.h"
-#include "cpu.h"
-#include "exec/cpu-all.h"
#include "exec/hwaddr.h"
#include "monitor/monitor.h"
#include "sysemu/kvm.h"
diff --git a/exec.c b/exec.c
index b69fd29..3df2ffc 100644
--- a/exec.c
+++ b/exec.c
@@ -25,7 +25,6 @@
#endif
#include "qemu-common.h"
-#include "cpu.h"
#include "tcg.h"
#include "hw/hw.h"
#include "hw/qdev.h"
@@ -44,7 +43,6 @@
#include "sysemu/xen-mapcache.h"
#include "trace.h"
#endif
-#include "exec/cpu-all.h"
#include "exec/cputlb.h"
#include "translate-all.h"
diff --git a/gdbstub.c b/gdbstub.c
index e8ab0b2..a96add1 100644
--- a/gdbstub.c
+++ b/gdbstub.c
@@ -16,6 +16,13 @@
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, see <http://www.gnu.org/licenses/>.
*/
+
+#ifndef NEED_CPU_H
+#error target-xxx/cpu.h must be included because target-specific are required
+#endif
+#include "cpu.h" /* target-xxx/cpu.h, required for CPUArchState etc */
+/* TODO: to remove using CPUArchState to make thie file arch-independent. */
+
#include "config.h"
#include "qemu-common.h"
#ifdef CONFIG_USER_ONLY
@@ -37,7 +44,6 @@
#define MAX_PACKET_LENGTH 4096
-#include "cpu.h"
#include "qemu/sockets.h"
#include "sysemu/kvm.h"
diff --git a/kvm-stub.c b/kvm-stub.c
index e979f76..581350a 100644
--- a/kvm-stub.c
+++ b/kvm-stub.c
@@ -12,7 +12,6 @@
#include "qemu-common.h"
#include "hw/hw.h"
-#include "cpu.h"
#include "sysemu/kvm.h"
#ifndef CONFIG_USER_ONLY
diff --git a/memory_mapping.c b/memory_mapping.c
index 87a6ed5..b17eea1 100644
--- a/memory_mapping.c
+++ b/memory_mapping.c
@@ -13,11 +13,11 @@
#include <glib.h>
-#include "cpu.h"
-#include "exec/cpu-all.h"
-#include "sysemu/memory_mapping.h"
+#include "qemu-common.h"
+#include "exec/cpu-common.h" /* for ram_addr_t */
#include "exec/memory.h"
#include "exec/address-spaces.h"
+#include "sysemu/memory_mapping.h"
//#define DEBUG_GUEST_PHYS_REGION_ADD
diff --git a/monitor.c b/monitor.c
index aebcbd8..1a8dd24 100644
--- a/monitor.c
+++ b/monitor.c
@@ -58,7 +58,6 @@
#include "qapi/qmp/json-parser.h"
#include <qom/object_interfaces.h>
#include "qemu/osdep.h"
-#include "cpu.h"
#include "trace.h"
#include "trace/control.h"
#ifdef CONFIG_TRACE_SIMPLE
diff --git a/translate-all.c b/translate-all.c
index 1ac0246..be220fb 100644
--- a/translate-all.c
+++ b/translate-all.c
@@ -16,6 +16,12 @@
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, see <http://www.gnu.org/licenses/>.
*/
+
+#ifndef NEED_CPU_H
+#error target-xxx/cpu.h must be included because target-specific are required
+#endif
+#include "cpu.h" /* target-xxx/cpu.h, required for CPUArchState etc */
+
#ifdef _WIN32
#include <windows.h>
#else
@@ -32,7 +38,6 @@
#include "qemu-common.h"
#define NO_CPU_IO_DEFS
-#include "cpu.h"
#include "disas/disas.h"
#include "tcg.h"
#if defined(CONFIG_USER_ONLY)
diff --git a/user-exec.c b/user-exec.c
index 82bfa66..c4c1e3f 100644
--- a/user-exec.c
+++ b/user-exec.c
@@ -16,8 +16,13 @@
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, see <http://www.gnu.org/licenses/>.
*/
+
+#ifndef NEED_CPU_H
+#error target-xxx/cpu.h must be included because target-specific are required
+#endif
+#include "cpu.h" /* target-xxx/cpu.h, required for CPUArchState etc */
+
#include "config.h"
-#include "cpu.h"
#include "disas/disas.h"
#include "tcg.h"
#include "qemu/bitops.h"