test/plugin: make insn plugin less noisy by default
While the match functionality is useful lets make the verbosity optional while we are actually running. Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20240705084047.857176-22-alex.bennee@linaro.org>
This commit is contained in:
parent
a4ad4db484
commit
03295660c8
@ -20,6 +20,7 @@ static qemu_plugin_u64 insn_count;
|
|||||||
|
|
||||||
static bool do_inline;
|
static bool do_inline;
|
||||||
static bool do_size;
|
static bool do_size;
|
||||||
|
static bool do_trace;
|
||||||
static GArray *sizes;
|
static GArray *sizes;
|
||||||
|
|
||||||
typedef struct {
|
typedef struct {
|
||||||
@ -73,18 +74,19 @@ static void vcpu_insn_matched_exec_before(unsigned int cpu_index, void *udata)
|
|||||||
MatchCount *match = qemu_plugin_scoreboard_find(insn_match->counts,
|
MatchCount *match = qemu_plugin_scoreboard_find(insn_match->counts,
|
||||||
cpu_index);
|
cpu_index);
|
||||||
|
|
||||||
g_autoptr(GString) ts = g_string_new("");
|
|
||||||
|
|
||||||
insn->hits++;
|
insn->hits++;
|
||||||
g_string_append_printf(ts, "0x%" PRIx64 ", '%s', %"PRId64 " hits",
|
|
||||||
insn->vaddr, insn->disas, insn->hits);
|
|
||||||
|
|
||||||
uint64_t icount = qemu_plugin_u64_get(insn_count, cpu_index);
|
uint64_t icount = qemu_plugin_u64_get(insn_count, cpu_index);
|
||||||
uint64_t delta = icount - match->last_hit;
|
uint64_t delta = icount - match->last_hit;
|
||||||
|
|
||||||
match->hits++;
|
match->hits++;
|
||||||
match->total_delta += delta;
|
match->total_delta += delta;
|
||||||
|
match->last_hit = icount;
|
||||||
|
|
||||||
|
if (do_trace) {
|
||||||
|
g_autoptr(GString) ts = g_string_new("");
|
||||||
|
g_string_append_printf(ts, "0x%" PRIx64 ", '%s', %"PRId64 " hits",
|
||||||
|
insn->vaddr, insn->disas, insn->hits);
|
||||||
g_string_append_printf(ts,
|
g_string_append_printf(ts,
|
||||||
" , cpu %u,"
|
" , cpu %u,"
|
||||||
" %"PRId64" match hits,"
|
" %"PRId64" match hits,"
|
||||||
@ -94,10 +96,9 @@ static void vcpu_insn_matched_exec_before(unsigned int cpu_index, void *udata)
|
|||||||
match->hits, delta,
|
match->hits, delta,
|
||||||
match->total_delta / match->hits);
|
match->total_delta / match->hits);
|
||||||
|
|
||||||
match->last_hit = icount;
|
|
||||||
|
|
||||||
qemu_plugin_outs(ts->str);
|
qemu_plugin_outs(ts->str);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
static void vcpu_tb_trans(qemu_plugin_id_t id, struct qemu_plugin_tb *tb)
|
static void vcpu_tb_trans(qemu_plugin_id_t id, struct qemu_plugin_tb *tb)
|
||||||
{
|
{
|
||||||
@ -216,6 +217,11 @@ QEMU_PLUGIN_EXPORT int qemu_plugin_install(qemu_plugin_id_t id,
|
|||||||
}
|
}
|
||||||
} else if (g_strcmp0(tokens[0], "match") == 0) {
|
} else if (g_strcmp0(tokens[0], "match") == 0) {
|
||||||
parse_match(tokens[1]);
|
parse_match(tokens[1]);
|
||||||
|
} else if (g_strcmp0(tokens[0], "trace") == 0) {
|
||||||
|
if (!qemu_plugin_bool_parse(tokens[0], tokens[1], &do_trace)) {
|
||||||
|
fprintf(stderr, "boolean argument parsing failed: %s\n", opt);
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
} else {
|
} else {
|
||||||
fprintf(stderr, "option parsing failed: %s\n", opt);
|
fprintf(stderr, "option parsing failed: %s\n", opt);
|
||||||
return -1;
|
return -1;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user