Patchwork [10/11,v10] make gdb_id() generally avialable

login
register
mail settings
Submitter Wen Congyang
Date March 20, 2012, 3:56 a.m.
Message ID <4F67FFF6.8010501@cn.fujitsu.com>
Download mbox | patch
Permalink /patch/147735/
State New
Headers show

Comments

Wen Congyang - March 20, 2012, 3:56 a.m.
The following patch also needs this API, so make it generally avialable

Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
---
 gdbstub.c |    9 ---------
 gdbstub.h |    9 +++++++++
 2 files changed, 9 insertions(+), 9 deletions(-)
HATAYAMA Daisuke - March 23, 2012, 10:03 a.m.
From: Wen Congyang <wency@cn.fujitsu.com>
Subject: [PATCH 10/11 v10] make gdb_id() generally avialable
Date: Tue, 20 Mar 2012 11:56:38 +0800

> The following patch also needs this API, so make it generally avialable
> 
> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
> ---
>  gdbstub.c |    9 ---------
>  gdbstub.h |    9 +++++++++
>  2 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/gdbstub.c b/gdbstub.c
> index f4e97f7..7f8ac2f 100644
> --- a/gdbstub.c
> +++ b/gdbstub.c
> @@ -1939,15 +1939,6 @@ static void gdb_set_cpu_pc(GDBState *s, target_ulong pc)
>  #endif
>  }
>  
> -static inline int gdb_id(CPUArchState *env)
> -{
> -#if defined(CONFIG_USER_ONLY) && defined(CONFIG_USE_NPTL)
> -    return env->host_tid;
> -#else
> -    return env->cpu_index + 1;
> -#endif
> -}
> -
>  static CPUArchState *find_cpu(uint32_t thread_id)
>  {
>      CPUArchState *env;
> diff --git a/gdbstub.h b/gdbstub.h
> index b44e275..f82ec70 100644
> --- a/gdbstub.h
> +++ b/gdbstub.h
> @@ -30,6 +30,15 @@ void gdb_register_coprocessor(CPUArchState *env,
>                                gdb_reg_cb get_reg, gdb_reg_cb set_reg,
>                                int num_regs, const char *xml, int g_pos);
>  
> +static inline int gdb_id(CPUArchState *env)
> +{
> +#if defined(CONFIG_USER_ONLY) && defined(CONFIG_USE_NPTL)
> +    return env->host_tid;
> +#else
> +    return env->cpu_index + 1;
> +#endif
> +}
> +

It seems to me more reasonable to newly introduce helper function
cpu_index(), then use it in gdb_id() and in qemu dump.

Thanks.
HATAYAMA, Daisuke

Patch

diff --git a/gdbstub.c b/gdbstub.c
index f4e97f7..7f8ac2f 100644
--- a/gdbstub.c
+++ b/gdbstub.c
@@ -1939,15 +1939,6 @@  static void gdb_set_cpu_pc(GDBState *s, target_ulong pc)
 #endif
 }
 
-static inline int gdb_id(CPUArchState *env)
-{
-#if defined(CONFIG_USER_ONLY) && defined(CONFIG_USE_NPTL)
-    return env->host_tid;
-#else
-    return env->cpu_index + 1;
-#endif
-}
-
 static CPUArchState *find_cpu(uint32_t thread_id)
 {
     CPUArchState *env;
diff --git a/gdbstub.h b/gdbstub.h
index b44e275..f82ec70 100644
--- a/gdbstub.h
+++ b/gdbstub.h
@@ -30,6 +30,15 @@  void gdb_register_coprocessor(CPUArchState *env,
                               gdb_reg_cb get_reg, gdb_reg_cb set_reg,
                               int num_regs, const char *xml, int g_pos);
 
+static inline int gdb_id(CPUArchState *env)
+{
+#if defined(CONFIG_USER_ONLY) && defined(CONFIG_USE_NPTL)
+    return env->host_tid;
+#else
+    return env->cpu_index + 1;
+#endif
+}
+
 #endif
 
 #ifdef CONFIG_USER_ONLY