diff mbox

target-tilegx: Support dumping working flow instructions and registers

Message ID 1458142686-4405-1-git-send-email-chengang@emindsoft.com.cn (mailing list archive)
State New, archived
Headers show

Commit Message

Chen Gang March 16, 2016, 3:38 p.m. UTC 
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(+)

Comments

Chen Gang March 16, 2016, 3:51 p.m. UTC | #1 
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;
>
Chen Gang March 27, 2016, 9:56 a.m. UTC | #2 
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 mbox

Patch

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;