From: Wen Congyang <we...@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 <we...@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