[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-tilegx: Support dumping working flow ins
From: |
Chen Gang |
Subject: |
Re: [Qemu-devel] [PATCH] target-tilegx: Support dumping working flow instructions and registers |
Date: |
Sun, 27 Mar 2016 17:56:57 +0800 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
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, 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.