[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 05/25] target/arm: Move arm_singlestep_active out of line
From: |
Richard Henderson |
Subject: |
[PATCH v2 05/25] target/arm: Move arm_singlestep_active out of line |
Date: |
Mon, 6 Jun 2022 19:47:14 -0700 |
Move the function to debug_helper.c, and the
declaration to internals.h.
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
target/arm/cpu.h | 10 ----------
target/arm/internals.h | 1 +
target/arm/debug_helper.c | 12 ++++++++++++
3 files changed, 13 insertions(+), 10 deletions(-)
diff --git a/target/arm/cpu.h b/target/arm/cpu.h
index c1865ad5da..2e115a0281 100644
--- a/target/arm/cpu.h
+++ b/target/arm/cpu.h
@@ -3087,16 +3087,6 @@ static inline bool
arm_generate_debug_exceptions(CPUARMState *env)
}
}
-/* Is single-stepping active? (Note that the "is EL_D AArch64?" check
- * implicitly means this always returns false in pre-v8 CPUs.)
- */
-static inline bool arm_singlestep_active(CPUARMState *env)
-{
- return extract32(env->cp15.mdscr_el1, 0, 1)
- && arm_el_is_aa64(env, arm_debug_target_el(env))
- && arm_generate_debug_exceptions(env);
-}
-
static inline bool arm_sctlr_b(CPUARMState *env)
{
return
diff --git a/target/arm/internals.h b/target/arm/internals.h
index 36c33a6ac5..148f2a9252 100644
--- a/target/arm/internals.h
+++ b/target/arm/internals.h
@@ -1315,5 +1315,6 @@ void define_cortex_a72_a57_a53_cp_reginfo(ARMCPU *cpu);
void aa32_max_features(ARMCPU *cpu);
int exception_target_el(CPUARMState *env);
+bool arm_singlestep_active(CPUARMState *env);
#endif
diff --git a/target/arm/debug_helper.c b/target/arm/debug_helper.c
index 46893697cc..1abf41c5f8 100644
--- a/target/arm/debug_helper.c
+++ b/target/arm/debug_helper.c
@@ -11,6 +11,18 @@
#include "exec/exec-all.h"
#include "exec/helper-proto.h"
+
+/*
+ * Is single-stepping active? (Note that the "is EL_D AArch64?" check
+ * implicitly means this always returns false in pre-v8 CPUs.)
+ */
+bool arm_singlestep_active(CPUARMState *env)
+{
+ return extract32(env->cp15.mdscr_el1, 0, 1)
+ && arm_el_is_aa64(env, arm_debug_target_el(env))
+ && arm_generate_debug_exceptions(env);
+}
+
/* Return true if the linked breakpoint entry lbn passes its checks */
static bool linked_bp_matches(ARMCPU *cpu, int lbn)
{
--
2.34.1
- [PATCH v2 00/25] target/arm: tidy exception routing, Richard Henderson, 2022/06/06
- [PATCH v2 02/25] target/arm: Add coproc parameter to syn_fp_access_trap, Richard Henderson, 2022/06/06
- [PATCH v2 01/25] target/arm: Mark exception helpers as noreturn, Richard Henderson, 2022/06/06
- [PATCH v2 04/25] target/arm: Move exception_target_el out of line, Richard Henderson, 2022/06/06
- [PATCH v2 05/25] target/arm: Move arm_singlestep_active out of line,
Richard Henderson <=
- [PATCH v2 07/25] target/arm: Use is_a64 in arm_generate_debug_exceptions, Richard Henderson, 2022/06/06
- [PATCH v2 03/25] target/arm: Move fp access syndrome adjust out of raise_exception, Richard Henderson, 2022/06/06
- [PATCH v2 06/25] target/arm: Move arm_generate_debug_exceptions out of line, Richard Henderson, 2022/06/06
- [PATCH v2 08/25] target/arm: Move exception_bkpt_insn to debug_helper.c, Richard Henderson, 2022/06/06
- [PATCH v2 09/25] target/arm: Move arm_debug_exception_fsr to debug_helper.c, Richard Henderson, 2022/06/06
- [PATCH v2 11/25] target/arm: Introduce gen_exception_insn_el_v, Richard Henderson, 2022/06/06