Re: [Qemu-devel] [PATCH] target-tilegx: Support dumping working flow instructions and registers

[Chen Gang]

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, address@hidden wrote:
> From: Chen Gang <address@hidden>
> 
> It is only for debug and analyzing tilegx qemu related issues.
> 
> Signed-off-by: Chen Gang <address@hidden>
> ---
>  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;
> 

-- 
Chen Gang (陈刚)

Managing Natural Environments is the Duty of Human Beings.