Message ID | 1458142686-4405-1-git-send-email-chengang@emindsoft.com.cn |
---|---|
State | New |
Headers | show |
Hello all: It is only for analyzing issues, I guess it is helpful, so I send it, welcome any other ideas, suggestions, and completions. And next, I shall continue to improve the floating point instruction features: "remove (u)int64_to_float64 from fdouble implementation". Thanks. On 3/16/16 23:38, chengang@emindsoft.com.cn wrote: > From: Chen Gang <gang.chen.5i5j@gmail.com> > > It is only for debug and analyzing tilegx qemu related issues. > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > target-tilegx/helper.c | 34 ++++++++++++++++++++++++++++++++++ > target-tilegx/helper.h | 2 ++ > target-tilegx/translate.c | 37 +++++++++++++++++++++++++++++++++++++ > 3 files changed, 73 insertions(+) > > diff --git a/target-tilegx/helper.c b/target-tilegx/helper.c > index 616c5c7..c7cfe66 100644 > --- a/target-tilegx/helper.c > +++ b/target-tilegx/helper.c > @@ -160,3 +160,37 @@ uint64_t helper_cmul2(uint64_t srca, uint64_t srcb, int shift, int round) > > return deposit32(realr >> shift, 16, 16, imagr >> shift); > } > + > +extern char **debug_buf; > + > +#define DEBUG_DUMP_REGS > + > +static void dump_regs(CPUTLGState *env, int idx) > +{ > +#ifdef DEBUG_DUMP_REGS > + int i; > + > + if (debug_buf[idx][2] == '\n') { > + for (i = 0; i < TILEGX_R_COUNT; i++) { > + if (!(i % 4)) { > + fprintf(stderr, "\n"); > + } > + fprintf(stderr, "r%2.2u = %16.16lx, ", i, env->regs[i]); > + } > + fprintf(stderr, "\n"); > + for (i = 0; i < TILEGX_SPR_COUNT; i++) { > + if (!(i % 8)) { > + fprintf(stderr, "\n"); > + } > + fprintf(stderr, "s%2.2u = %16.16lx, ", i, env->spregs[i]); > + } > + fprintf(stderr, "\n"); > + } > +#endif > +} > + > +void helper_print_step(CPUTLGState *env, int idx) > +{ > + fprintf(stderr, "%s", debug_buf[idx]); > + dump_regs(env, idx); > +} > diff --git a/target-tilegx/helper.h b/target-tilegx/helper.h > index 9281d0f..74f796a 100644 > --- a/target-tilegx/helper.h > +++ b/target-tilegx/helper.h > @@ -24,3 +24,5 @@ DEF_HELPER_FLAGS_2(v1shrs, TCG_CALL_NO_RWG_SE, i64, i64, i64) > DEF_HELPER_FLAGS_2(v2shl, TCG_CALL_NO_RWG_SE, i64, i64, i64) > DEF_HELPER_FLAGS_2(v2shru, TCG_CALL_NO_RWG_SE, i64, i64, i64) > DEF_HELPER_FLAGS_2(v2shrs, TCG_CALL_NO_RWG_SE, i64, i64, i64) > + > +DEF_HELPER_2(print_step, void, env, int) > diff --git a/target-tilegx/translate.c b/target-tilegx/translate.c > index 03918eb..8489494 100644 > --- a/target-tilegx/translate.c > +++ b/target-tilegx/translate.c > @@ -47,6 +47,43 @@ static const char * const reg_names[64] = { > "sn", "idn0", "idn1", "udn0", "udn1", "udn2", "udn2", "zero" > }; > > +#define STATIC_DEBUG > + > +char **debug_buf; > + > +#ifdef STATIC_DEBUG > + > +#include <stdarg.h> > + > +#define qemu_log_mask(flag, fmt, ...) debug_print(flag, fmt, ##__VA_ARGS__) > + > +static int debug_idx; > + > +extern void debug_print(int flag, const char *fmt, ...); > + > +void debug_print(int flag, const char *fmt, ...) > +{ > + va_list args; > + char buf[0x400]; > + TCGv_i32 tmp; > + > + va_start(args, fmt); > + vsprintf(buf, fmt, args); > + va_end(args); > + > + if (!debug_buf) { > + debug_buf = malloc(sizeof(char *) * 0x400000); > + } > + debug_buf[debug_idx] = malloc(strlen(buf) + 1); > + strcpy(debug_buf[debug_idx], buf); > + > + tmp = tcg_const_i32(debug_idx++); > + gen_helper_print_step(cpu_env, tmp); > + tcg_temp_free_i32(tmp); > +} > + > +#endif > + > /* Modified registers are cached in temporaries until the end of the bundle. */ > typedef struct { > unsigned reg; >
Hello all: Is this patch helpful? If it shouldn't be merged into our master branch, please let me know, I shall treat it as my own local patch (when I update my local master branch, I'll merge it at last). Thanks. On 3/16/16 23:51, Chen Gang wrote: > Hello all: > > It is only for analyzing issues, I guess it is helpful, so I send it, > welcome any other ideas, suggestions, and completions. > > And next, I shall continue to improve the floating point instruction > features: "remove (u)int64_to_float64 from fdouble implementation". > > Thanks. > > On 3/16/16 23:38, chengang@emindsoft.com.cn wrote: >> From: Chen Gang <gang.chen.5i5j@gmail.com> >> >> It is only for debug and analyzing tilegx qemu related issues. >> >> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> >> --- >> target-tilegx/helper.c | 34 ++++++++++++++++++++++++++++++++++ >> target-tilegx/helper.h | 2 ++ >> target-tilegx/translate.c | 37 +++++++++++++++++++++++++++++++++++++ >> 3 files changed, 73 insertions(+) >> >> diff --git a/target-tilegx/helper.c b/target-tilegx/helper.c >> index 616c5c7..c7cfe66 100644 >> --- a/target-tilegx/helper.c >> +++ b/target-tilegx/helper.c >> @@ -160,3 +160,37 @@ uint64_t helper_cmul2(uint64_t srca, uint64_t srcb, int shift, int round) >> >> return deposit32(realr >> shift, 16, 16, imagr >> shift); >> } >> + >> +extern char **debug_buf; >> + >> +#define DEBUG_DUMP_REGS >> + >> +static void dump_regs(CPUTLGState *env, int idx) >> +{ >> +#ifdef DEBUG_DUMP_REGS >> + int i; >> + >> + if (debug_buf[idx][2] == '\n') { >> + for (i = 0; i < TILEGX_R_COUNT; i++) { >> + if (!(i % 4)) { >> + fprintf(stderr, "\n"); >> + } >> + fprintf(stderr, "r%2.2u = %16.16lx, ", i, env->regs[i]); >> + } >> + fprintf(stderr, "\n"); >> + for (i = 0; i < TILEGX_SPR_COUNT; i++) { >> + if (!(i % 8)) { >> + fprintf(stderr, "\n"); >> + } >> + fprintf(stderr, "s%2.2u = %16.16lx, ", i, env->spregs[i]); >> + } >> + fprintf(stderr, "\n"); >> + } >> +#endif >> +} >> + >> +void helper_print_step(CPUTLGState *env, int idx) >> +{ >> + fprintf(stderr, "%s", debug_buf[idx]); >> + dump_regs(env, idx); >> +} >> diff --git a/target-tilegx/helper.h b/target-tilegx/helper.h >> index 9281d0f..74f796a 100644 >> --- a/target-tilegx/helper.h >> +++ b/target-tilegx/helper.h >> @@ -24,3 +24,5 @@ DEF_HELPER_FLAGS_2(v1shrs, TCG_CALL_NO_RWG_SE, i64, i64, i64) >> DEF_HELPER_FLAGS_2(v2shl, TCG_CALL_NO_RWG_SE, i64, i64, i64) >> DEF_HELPER_FLAGS_2(v2shru, TCG_CALL_NO_RWG_SE, i64, i64, i64) >> DEF_HELPER_FLAGS_2(v2shrs, TCG_CALL_NO_RWG_SE, i64, i64, i64) >> + >> +DEF_HELPER_2(print_step, void, env, int) >> diff --git a/target-tilegx/translate.c b/target-tilegx/translate.c >> index 03918eb..8489494 100644 >> --- a/target-tilegx/translate.c >> +++ b/target-tilegx/translate.c >> @@ -47,6 +47,43 @@ static const char * const reg_names[64] = { >> "sn", "idn0", "idn1", "udn0", "udn1", "udn2", "udn2", "zero" >> }; >> >> +#define STATIC_DEBUG >> + >> +char **debug_buf; >> + >> +#ifdef STATIC_DEBUG >> + >> +#include <stdarg.h> >> + >> +#define qemu_log_mask(flag, fmt, ...) debug_print(flag, fmt, ##__VA_ARGS__) >> + >> +static int debug_idx; >> + >> +extern void debug_print(int flag, const char *fmt, ...); >> + >> +void debug_print(int flag, const char *fmt, ...) >> +{ >> + va_list args; >> + char buf[0x400]; >> + TCGv_i32 tmp; >> + >> + va_start(args, fmt); >> + vsprintf(buf, fmt, args); >> + va_end(args); >> + >> + if (!debug_buf) { >> + debug_buf = malloc(sizeof(char *) * 0x400000); >> + } >> + debug_buf[debug_idx] = malloc(strlen(buf) + 1); >> + strcpy(debug_buf[debug_idx], buf); >> + >> + tmp = tcg_const_i32(debug_idx++); >> + gen_helper_print_step(cpu_env, tmp); >> + tcg_temp_free_i32(tmp); >> +} >> + >> +#endif >> + >> /* Modified registers are cached in temporaries until the end of the bundle. */ >> typedef struct { >> unsigned reg; >> >
diff --git a/target-tilegx/helper.c b/target-tilegx/helper.c index 616c5c7..c7cfe66 100644 --- a/target-tilegx/helper.c +++ b/target-tilegx/helper.c @@ -160,3 +160,37 @@ uint64_t helper_cmul2(uint64_t srca, uint64_t srcb, int shift, int round) return deposit32(realr >> shift, 16, 16, imagr >> shift); } + +extern char **debug_buf; + +#define DEBUG_DUMP_REGS + +static void dump_regs(CPUTLGState *env, int idx) +{ +#ifdef DEBUG_DUMP_REGS + int i; + + if (debug_buf[idx][2] == '\n') { + for (i = 0; i < TILEGX_R_COUNT; i++) { + if (!(i % 4)) { + fprintf(stderr, "\n"); + } + fprintf(stderr, "r%2.2u = %16.16lx, ", i, env->regs[i]); + } + fprintf(stderr, "\n"); + for (i = 0; i < TILEGX_SPR_COUNT; i++) { + if (!(i % 8)) { + fprintf(stderr, "\n"); + } + fprintf(stderr, "s%2.2u = %16.16lx, ", i, env->spregs[i]); + } + fprintf(stderr, "\n"); + } +#endif +} + +void helper_print_step(CPUTLGState *env, int idx) +{ + fprintf(stderr, "%s", debug_buf[idx]); + dump_regs(env, idx); +} diff --git a/target-tilegx/helper.h b/target-tilegx/helper.h index 9281d0f..74f796a 100644 --- a/target-tilegx/helper.h +++ b/target-tilegx/helper.h @@ -24,3 +24,5 @@ DEF_HELPER_FLAGS_2(v1shrs, TCG_CALL_NO_RWG_SE, i64, i64, i64) DEF_HELPER_FLAGS_2(v2shl, TCG_CALL_NO_RWG_SE, i64, i64, i64) DEF_HELPER_FLAGS_2(v2shru, TCG_CALL_NO_RWG_SE, i64, i64, i64) DEF_HELPER_FLAGS_2(v2shrs, TCG_CALL_NO_RWG_SE, i64, i64, i64) + +DEF_HELPER_2(print_step, void, env, int) diff --git a/target-tilegx/translate.c b/target-tilegx/translate.c index 03918eb..8489494 100644 --- a/target-tilegx/translate.c +++ b/target-tilegx/translate.c @@ -47,6 +47,43 @@ static const char * const reg_names[64] = { "sn", "idn0", "idn1", "udn0", "udn1", "udn2", "udn2", "zero" }; +#define STATIC_DEBUG + +char **debug_buf; + +#ifdef STATIC_DEBUG + +#include <stdarg.h> + +#define qemu_log_mask(flag, fmt, ...) debug_print(flag, fmt, ##__VA_ARGS__) + +static int debug_idx; + +extern void debug_print(int flag, const char *fmt, ...); + +void debug_print(int flag, const char *fmt, ...) +{ + va_list args; + char buf[0x400]; + TCGv_i32 tmp; + + va_start(args, fmt); + vsprintf(buf, fmt, args); + va_end(args); + + if (!debug_buf) { + debug_buf = malloc(sizeof(char *) * 0x400000); + } + debug_buf[debug_idx] = malloc(strlen(buf) + 1); + strcpy(debug_buf[debug_idx], buf); + + tmp = tcg_const_i32(debug_idx++); + gen_helper_print_step(cpu_env, tmp); + tcg_temp_free_i32(tmp); +} + +#endif + /* Modified registers are cached in temporaries until the end of the bundle. */ typedef struct { unsigned reg;