diff mbox

[10/50] target-m68k: make cpu-qom.h not target specific

Message ID 1463412982-6681-11-git-send-email-pbonzini@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Paolo Bonzini May 16, 2016, 3:35 p.m. UTC 
Make M68KCPU an opaque type within cpu-qom.h, and move all definitions of
private methods, as well as all type definitions that require knowledge
of the layout to cpu.h.  This helps making files independent of NEED_CPU_H
if they only need to pass around CPU pointers.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 target-m68k/cpu-qom.h | 34 +---------------------------------
 target-m68k/cpu.h     | 36 ++++++++++++++++++++++++++++++++++--
 2 files changed, 35 insertions(+), 35 deletions(-)

Comments

Thomas Huth May 17, 2016, 10:27 a.m. UTC | #1 
On 16.05.2016 17:35, Paolo Bonzini wrote:
> Make M68KCPU an opaque type within cpu-qom.h, and move all definitions of
> private methods, as well as all type definitions that require knowledge
> of the layout to cpu.h.  This helps making files independent of NEED_CPU_H
> if they only need to pass around CPU pointers.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  target-m68k/cpu-qom.h | 34 +---------------------------------
>  target-m68k/cpu.h     | 36 ++++++++++++++++++++++++++++++++++--
>  2 files changed, 35 insertions(+), 35 deletions(-)
> 
> diff --git a/target-m68k/cpu-qom.h b/target-m68k/cpu-qom.h
> index c28e55d..9885bba 100644
> --- a/target-m68k/cpu-qom.h
> +++ b/target-m68k/cpu-qom.h
> @@ -47,38 +47,6 @@ typedef struct M68kCPUClass {
>      void (*parent_reset)(CPUState *cpu);
>  } M68kCPUClass;
>  
> -/**
> - * M68kCPU:
> - * @env: #CPUM68KState
> - *
> - * A Motorola 68k CPU.
> - */
> -typedef struct M68kCPU {
> -    /*< private >*/
> -    CPUState parent_obj;
> -    /*< public >*/
> -
> -    CPUM68KState env;
> -} M68kCPU;
> -
> -static inline M68kCPU *m68k_env_get_cpu(CPUM68KState *env)
> -{
> -    return container_of(env, M68kCPU, env);
> -}
> -
> -#define ENV_GET_CPU(e) CPU(m68k_env_get_cpu(e))
> -
> -#define ENV_OFFSET offsetof(M68kCPU, env)
> -
> -void m68k_cpu_do_interrupt(CPUState *cpu);
> -bool m68k_cpu_exec_interrupt(CPUState *cpu, int int_req);
> -void m68k_cpu_dump_state(CPUState *cpu, FILE *f, fprintf_function cpu_fprintf,
> -                         int flags);
> -hwaddr m68k_cpu_get_phys_page_debug(CPUState *cpu, vaddr addr);
> -int m68k_cpu_gdb_read_register(CPUState *cpu, uint8_t *buf, int reg);
> -int m68k_cpu_gdb_write_register(CPUState *cpu, uint8_t *buf, int reg);
> -
> -void m68k_cpu_exec_enter(CPUState *cs);
> -void m68k_cpu_exec_exit(CPUState *cs);
> +typedef struct M68kCPU M68kCPU;
>  
>  #endif
> diff --git a/target-m68k/cpu.h b/target-m68k/cpu.h
> index 48b4c87..57c2c8d 100644
> --- a/target-m68k/cpu.h
> +++ b/target-m68k/cpu.h
> @@ -26,7 +26,7 @@
>  
>  #include "qemu-common.h"
>  #include "exec/cpu-defs.h"
> -
> +#include "cpu-qom.h"
>  #include "fpu/softfloat.h"
>  
>  #define MAX_QREGS 32
> @@ -109,7 +109,39 @@ typedef struct CPUM68KState {
>      uint32_t features;
>  } CPUM68KState;
>  
> -#include "cpu-qom.h"
> +/**
> + * M68kCPU:
> + * @env: #CPUM68KState
> + *
> + * A Motorola 68k CPU.
> + */
> +struct M68kCPU {
> +    /*< private >*/
> +    CPUState parent_obj;
> +    /*< public >*/
> +
> +    CPUM68KState env;
> +};
> +
> +static inline M68kCPU *m68k_env_get_cpu(CPUM68KState *env)
> +{
> +    return container_of(env, M68kCPU, env);
> +}
> +
> +#define ENV_GET_CPU(e) CPU(m68k_env_get_cpu(e))
> +
> +#define ENV_OFFSET offsetof(M68kCPU, env)
> +
> +void m68k_cpu_do_interrupt(CPUState *cpu);
> +bool m68k_cpu_exec_interrupt(CPUState *cpu, int int_req);
> +void m68k_cpu_dump_state(CPUState *cpu, FILE *f, fprintf_function cpu_fprintf,
> +                         int flags);
> +hwaddr m68k_cpu_get_phys_page_debug(CPUState *cpu, vaddr addr);
> +int m68k_cpu_gdb_read_register(CPUState *cpu, uint8_t *buf, int reg);
> +int m68k_cpu_gdb_write_register(CPUState *cpu, uint8_t *buf, int reg);
> +
> +void m68k_cpu_exec_enter(CPUState *cs);
> +void m68k_cpu_exec_exit(CPUState *cs);
>  
>  void m68k_tcg_init(void);
>  void m68k_cpu_init_gdb(M68kCPU *cpu);
> 

Looks sane.

Reviewed-by: Thomas Huth <huth@tuxfamily.org>
diff mbox

Patch

diff --git a/target-m68k/cpu-qom.h b/target-m68k/cpu-qom.h
index c28e55d..9885bba 100644
--- a/target-m68k/cpu-qom.h
+++ b/target-m68k/cpu-qom.h
@@ -47,38 +47,6 @@  typedef struct M68kCPUClass {
     void (*parent_reset)(CPUState *cpu);
 } M68kCPUClass;
 
-/**
- * M68kCPU:
- * @env: #CPUM68KState
- *
- * A Motorola 68k CPU.
- */
-typedef struct M68kCPU {
-    /*< private >*/
-    CPUState parent_obj;
-    /*< public >*/
-
-    CPUM68KState env;
-} M68kCPU;
-
-static inline M68kCPU *m68k_env_get_cpu(CPUM68KState *env)
-{
-    return container_of(env, M68kCPU, env);
-}
-
-#define ENV_GET_CPU(e) CPU(m68k_env_get_cpu(e))
-
-#define ENV_OFFSET offsetof(M68kCPU, env)
-
-void m68k_cpu_do_interrupt(CPUState *cpu);
-bool m68k_cpu_exec_interrupt(CPUState *cpu, int int_req);
-void m68k_cpu_dump_state(CPUState *cpu, FILE *f, fprintf_function cpu_fprintf,
-                         int flags);
-hwaddr m68k_cpu_get_phys_page_debug(CPUState *cpu, vaddr addr);
-int m68k_cpu_gdb_read_register(CPUState *cpu, uint8_t *buf, int reg);
-int m68k_cpu_gdb_write_register(CPUState *cpu, uint8_t *buf, int reg);
-
-void m68k_cpu_exec_enter(CPUState *cs);
-void m68k_cpu_exec_exit(CPUState *cs);
+typedef struct M68kCPU M68kCPU;
 
 #endif
diff --git a/target-m68k/cpu.h b/target-m68k/cpu.h
index 48b4c87..57c2c8d 100644
--- a/target-m68k/cpu.h
+++ b/target-m68k/cpu.h
@@ -26,7 +26,7 @@ 
 
 #include "qemu-common.h"
 #include "exec/cpu-defs.h"
-
+#include "cpu-qom.h"
 #include "fpu/softfloat.h"
 
 #define MAX_QREGS 32
@@ -109,7 +109,39 @@  typedef struct CPUM68KState {
     uint32_t features;
 } CPUM68KState;
 
-#include "cpu-qom.h"
+/**
+ * M68kCPU:
+ * @env: #CPUM68KState
+ *
+ * A Motorola 68k CPU.
+ */
+struct M68kCPU {
+    /*< private >*/
+    CPUState parent_obj;
+    /*< public >*/
+
+    CPUM68KState env;
+};
+
+static inline M68kCPU *m68k_env_get_cpu(CPUM68KState *env)
+{
+    return container_of(env, M68kCPU, env);
+}
+
+#define ENV_GET_CPU(e) CPU(m68k_env_get_cpu(e))
+
+#define ENV_OFFSET offsetof(M68kCPU, env)
+
+void m68k_cpu_do_interrupt(CPUState *cpu);
+bool m68k_cpu_exec_interrupt(CPUState *cpu, int int_req);
+void m68k_cpu_dump_state(CPUState *cpu, FILE *f, fprintf_function cpu_fprintf,
+                         int flags);
+hwaddr m68k_cpu_get_phys_page_debug(CPUState *cpu, vaddr addr);
+int m68k_cpu_gdb_read_register(CPUState *cpu, uint8_t *buf, int reg);
+int m68k_cpu_gdb_write_register(CPUState *cpu, uint8_t *buf, int reg);
+
+void m68k_cpu_exec_enter(CPUState *cs);
+void m68k_cpu_exec_exit(CPUState *cs);
 
 void m68k_tcg_init(void);
 void m68k_cpu_init_gdb(M68kCPU *cpu);