[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH v4 09/11] target-arm: dfilter support for in_a
From: |
Alex Bennée |
Subject: |
Re: [Qemu-trivial] [PATCH v4 09/11] target-arm: dfilter support for in_asm, op, opt_op |
Date: |
Tue, 04 Aug 2015 18:26:23 +0100 |
Richard Henderson <address@hidden> writes:
> On 08/03/2015 02:14 AM, Alex Bennée wrote:
>> Each individual architecture needs to use the qemu_log_in_addr_range()
>> feature for enabling in_asm and marking blocks for op/opt_op output.
>>
>> Signed-off-by: Alex Bennée <address@hidden>
>> ---
>> target-arm/translate-a64.c | 6 ++++--
>> target-arm/translate.c | 6 ++++--
>> 2 files changed, 8 insertions(+), 4 deletions(-)
>>
>> diff --git a/target-arm/translate-a64.c b/target-arm/translate-a64.c
>> index 689f2be..0b0f4ae 100644
>> --- a/target-arm/translate-a64.c
>> +++ b/target-arm/translate-a64.c
>> @@ -11026,7 +11026,8 @@ void gen_intermediate_code_internal_a64(ARMCPU *cpu,
>> gen_io_start();
>> }
>>
>> - if (unlikely(qemu_loglevel_mask(CPU_LOG_TB_OP |
>> CPU_LOG_TB_OP_OPT))) {
>> + if (unlikely(qemu_loglevel_mask(CPU_LOG_TB_OP | CPU_LOG_TB_OP_OPT)
>> &&
>> + qemu_log_in_addr_range(dc->pc))) {
>> tcg_gen_debug_insn_start(dc->pc);
>> }
>
> If there's more than one or two ranges, it's probably quicker to
> generate the debug opcode regardless of the range. Remember, this
> check is happening once per insn, not once per tb.
Maybe I should hoist the check up to the start of a block? This would
mean we would dump all instructions in a block even if they went past
the end-point but the reverse case is probably just confusing.
We'll still not dump anything that starts outside the range.
>
>
> r~
--
Alex Bennée
- Re: [Qemu-trivial] [PATCH v4 01/11] tcg: add ability to dump /tmp/perf-<pid>.map files, (continued)
Re: [Qemu-trivial] [PATCH v4 01/11] tcg: add ability to dump /tmp/perf-<pid>.map files, Aurelien Jarno, 2015/08/04
[Qemu-trivial] [PATCH v4 05/11] qemu-log: Improve the "exec" TB execution logging, Alex Bennée, 2015/08/03
[Qemu-trivial] [PATCH v4 09/11] target-arm: dfilter support for in_asm, op, opt_op, Alex Bennée, 2015/08/03
[Qemu-trivial] [PATCH v4 07/11] qemu-log: new option -dfilter to limit output, Alex Bennée, 2015/08/03
[Qemu-trivial] [PATCH v4 06/11] qemu-log: support simple pid substitution in logfile, Alex Bennée, 2015/08/03
[Qemu-trivial] [PATCH v4 08/11] qemu-log: dfilter-ise exec, out_asm, and op_opt, Alex Bennée, 2015/08/03
[Qemu-trivial] [PATCH v4 10/11] vl.c: log system invocation when enabled, Alex Bennée, 2015/08/03