motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-22 01:21:53 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

accel/tcg: Return bool from TranslatorOps.disas_log

Browse source 
We have eliminated most uses of this hook.  Reduce
further by allowing the hook to handle only the
special cases, returning false for normal processing.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
Richard Henderson 2024-04-02 23:07:53 -10:00
parent 962a145cdc
commit b67c567b79
4 changed files with 12 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

5
accel/tcg/translator.c
View file

@ -228,9 +228,8 @@ void translator_loop(CPUState *cpu, TranslationBlock *tb, int *max_insns,
if (logfile) { if (logfile) {
fprintf(logfile, "----------------\n"); fprintf(logfile, "----------------\n");
if (ops->disas_log) { if (!ops->disas_log ||
ops->disas_log(db, cpu, logfile); !ops->disas_log(db, cpu, logfile)) {
} else {
fprintf(logfile, "IN: %s\n", lookup_symbol(db->pc_first)); fprintf(logfile, "IN: %s\n", lookup_symbol(db->pc_first));
target_disas(logfile, cpu, db->pc_first, db->tb->size); target_disas(logfile, cpu, db->pc_first, db->tb->size);
} }

2
include/exec/translator.h
View file

@ -135,7 +135,7 @@ typedef struct TranslatorOps {
void (*insn_start)(DisasContextBase *db, CPUState *cpu); void (*insn_start)(DisasContextBase *db, CPUState *cpu);
void (*translate_insn)(DisasContextBase *db, CPUState *cpu); void (*translate_insn)(DisasContextBase *db, CPUState *cpu);
void (*tb_stop)(DisasContextBase *db, CPUState *cpu); void (*tb_stop)(DisasContextBase *db, CPUState *cpu);
void (*disas_log)(const DisasContextBase *db, CPUState *cpu, FILE *f); bool (*disas_log)(const DisasContextBase *db, CPUState *cpu, FILE *f);
} TranslatorOps; } TranslatorOps;
/** /**

15
target/hppa/translate.c
View file

@ -19,7 +19,6 @@
#include "qemu/osdep.h" #include "qemu/osdep.h"
#include "cpu.h" #include "cpu.h"
#include "disas/disas.h"
#include "qemu/host-utils.h" #include "qemu/host-utils.h"
#include "exec/exec-all.h" #include "exec/exec-all.h"
#include "exec/page-protection.h" #include "exec/page-protection.h"
@ -4817,7 +4816,7 @@ static void hppa_tr_tb_stop(DisasContextBase *dcbase, CPUState *cs)
} }
#ifdef CONFIG_USER_ONLY #ifdef CONFIG_USER_ONLY
static void hppa_tr_disas_log(const DisasContextBase *dcbase, static bool hppa_tr_disas_log(const DisasContextBase *dcbase,
CPUState *cs, FILE *logfile) CPUState *cs, FILE *logfile)
{ {
target_ulong pc = dcbase->pc_first; target_ulong pc = dcbase->pc_first;
@ -4825,20 +4824,18 @@ static void hppa_tr_disas_log(const DisasContextBase *dcbase,
switch (pc) { switch (pc) {
case 0x00: case 0x00:
fprintf(logfile, "IN:\n0x00000000: (null)\n"); fprintf(logfile, "IN:\n0x00000000: (null)\n");
return; return true;
case 0xb0: case 0xb0:
fprintf(logfile, "IN:\n0x000000b0: light-weight-syscall\n"); fprintf(logfile, "IN:\n0x000000b0: light-weight-syscall\n");
return; return true;
case 0xe0: case 0xe0:
fprintf(logfile, "IN:\n0x000000e0: set-thread-pointer-syscall\n"); fprintf(logfile, "IN:\n0x000000e0: set-thread-pointer-syscall\n");
return; return true;
case 0x100: case 0x100:
fprintf(logfile, "IN:\n0x00000100: syscall\n"); fprintf(logfile, "IN:\n0x00000100: syscall\n");
return; return true;
} }
return false;
fprintf(logfile, "IN: %s\n", lookup_symbol(pc));
target_disas(logfile, cs, pc, dcbase->tb->size);
} }
#endif #endif

8
target/s390x/tcg/translate.c
View file

@ -31,7 +31,6 @@
#include "qemu/osdep.h" #include "qemu/osdep.h"
#include "cpu.h" #include "cpu.h"
#include "s390x-internal.h" #include "s390x-internal.h"
#include "disas/disas.h"
#include "exec/exec-all.h" #include "exec/exec-all.h"
#include "tcg/tcg-op.h" #include "tcg/tcg-op.h"
#include "tcg/tcg-op-gvec.h" #include "tcg/tcg-op-gvec.h"
@ -6520,7 +6519,7 @@ static void s390x_tr_tb_stop(DisasContextBase *dcbase, CPUState *cs)
} }
} }
static void s390x_tr_disas_log(const DisasContextBase *dcbase, static bool s390x_tr_disas_log(const DisasContextBase *dcbase,
CPUState *cs, FILE *logfile) CPUState *cs, FILE *logfile)
{ {
DisasContext *dc = container_of(dcbase, DisasContext, base); DisasContext *dc = container_of(dcbase, DisasContext, base);
@ -6528,10 +6527,9 @@ static void s390x_tr_disas_log(const DisasContextBase *dcbase,
if (unlikely(dc->ex_value)) { if (unlikely(dc->ex_value)) {
/* ??? Unfortunately target_disas can't use host memory. */ /* ??? Unfortunately target_disas can't use host memory. */
fprintf(logfile, "IN: EXECUTE %016" PRIx64, dc->ex_value); fprintf(logfile, "IN: EXECUTE %016" PRIx64, dc->ex_value);
} else { return true;
fprintf(logfile, "IN: %s\n", lookup_symbol(dc->base.pc_first));
target_disas(logfile, cs, dc->base.pc_first, dc->base.tb->size);
} }
return false;
} }
static const TranslatorOps s390x_tr_ops = { static const TranslatorOps s390x_tr_ops = {