[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 3/6] target-xtensa: avoid double-stopping at breakpo
From: |
Max Filippov |
Subject: |
[Qemu-devel] [PATCH 3/6] target-xtensa: avoid double-stopping at breakpoints |
Date: |
Sun, 21 Jul 2013 16:05:03 +0400 |
env->exception_taken is set every time an exception is taken. It is used
to allow single-stepping to stop at the first exception handler
instruction. This however must exclude debug exceptions, as otherwise
first step from the instruction where breakpoint was hit stops at that
same instruction.
Also don't check env->exception_taken directly from the
gen_intermediate_code_internal, instead allocate and use TB flag
XTENSA_TBFLAG_EXCEPTION.
Signed-off-by: Max Filippov <address@hidden>
---
target-xtensa/cpu.h | 4 ++++
target-xtensa/op_helper.c | 3 +++
target-xtensa/translate.c | 3 +--
3 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/target-xtensa/cpu.h b/target-xtensa/cpu.h
index 6c9fc35..c19aeb9 100644
--- a/target-xtensa/cpu.h
+++ b/target-xtensa/cpu.h
@@ -484,6 +484,7 @@ static inline int cpu_mmu_index(CPUXtensaState *env)
#define XTENSA_TBFLAG_ICOUNT 0x20
#define XTENSA_TBFLAG_CPENABLE_MASK 0x3fc0
#define XTENSA_TBFLAG_CPENABLE_SHIFT 6
+#define XTENSA_TBFLAG_EXCEPTION 0x4000
static inline void cpu_get_tb_cpu_state(CPUXtensaState *env, target_ulong *pc,
target_ulong *cs_base, int *flags)
@@ -510,6 +511,9 @@ static inline void cpu_get_tb_cpu_state(CPUXtensaState
*env, target_ulong *pc,
if (xtensa_option_enabled(env->config, XTENSA_OPTION_COPROCESSOR)) {
*flags |= env->sregs[CPENABLE] << XTENSA_TBFLAG_CPENABLE_SHIFT;
}
+ if (env->singlestep_enabled && env->exception_taken) {
+ *flags |= XTENSA_TBFLAG_EXCEPTION;
+ }
}
#include "exec/cpu-all.h"
diff --git a/target-xtensa/op_helper.c b/target-xtensa/op_helper.c
index 834fe90..6ca912c 100644
--- a/target-xtensa/op_helper.c
+++ b/target-xtensa/op_helper.c
@@ -96,6 +96,9 @@ static void tb_invalidate_virtual_addr(CPUXtensaState *env,
uint32_t vaddr)
void HELPER(exception)(CPUXtensaState *env, uint32_t excp)
{
env->exception_index = excp;
+ if (excp == EXCP_DEBUG) {
+ env->exception_taken = 0;
+ }
cpu_loop_exit(env);
}
diff --git a/target-xtensa/translate.c b/target-xtensa/translate.c
index e4cf828..ef97db0 100644
--- a/target-xtensa/translate.c
+++ b/target-xtensa/translate.c
@@ -2917,8 +2917,7 @@ void gen_intermediate_code_internal(XtensaCPU *cpu,
gen_tb_start();
- if (env->singlestep_enabled && env->exception_taken) {
- env->exception_taken = 0;
+ if (tb->flags & XTENSA_TBFLAG_EXCEPTION) {
tcg_gen_movi_i32(cpu_pc, dc.pc);
gen_exception(&dc, EXCP_DEBUG);
}
--
1.8.1.4
- [Qemu-devel] [PATCH 0/6] target-xtensa queue, Max Filippov, 2013/07/21
- [Qemu-devel] [PATCH 1/6] target-xtensa: add extui unit test, Max Filippov, 2013/07/21
- [Qemu-devel] [PATCH 2/6] target-xtensa: add fallthrough markers, Max Filippov, 2013/07/21
- [Qemu-devel] [PATCH 3/6] target-xtensa: avoid double-stopping at breakpoints,
Max Filippov <=
- [Qemu-devel] [PATCH 4/6] tests/tcg/xtensa: Fix out-of-tree build, Max Filippov, 2013/07/21
- [Qemu-devel] [PATCH 5/6] target-xtensa: don't generate dead code to access invalid SRs, Max Filippov, 2013/07/21
- [Qemu-devel] [PATCH 6/6] target-xtensa: check register window inline, Max Filippov, 2013/07/21