[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/13] target/arm: Define new TBFLAG for v8M sta
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 01/13] target/arm: Define new TBFLAG for v8M stack checking |
Date: |
Thu, 4 Oct 2018 18:02:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0 |
Hi Peter,
On 02/10/2018 18:35, Peter Maydell wrote:
> The Arm v8M architecture includes hardware stack limit checking.
> When certain instructions update the stack pointer, if the new
> value of SP is below the limit set in the associated limit register
> then an exception is taken. Add a TB flag that tracks whether
> the limit-checking code needs to be emitted.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> target/arm/cpu.h | 7 +++++++
> target/arm/translate.h | 1 +
> target/arm/helper.c | 10 ++++++++++
> target/arm/translate.c | 1 +
> 4 files changed, 19 insertions(+)
>
> diff --git a/target/arm/cpu.h b/target/arm/cpu.h
> index 65c0fa0a659..d2c1d005ed7 100644
> --- a/target/arm/cpu.h
> +++ b/target/arm/cpu.h
> @@ -1336,8 +1336,10 @@ FIELD(V7M_CCR, UNALIGN_TRP, 3, 1)
> FIELD(V7M_CCR, DIV_0_TRP, 4, 1)
> FIELD(V7M_CCR, BFHFNMIGN, 8, 1)
> FIELD(V7M_CCR, STKALIGN, 9, 1)
> +FIELD(V7M_CCR, STKOFHFNMIGN, 10, 1)
> FIELD(V7M_CCR, DC, 16, 1)
> FIELD(V7M_CCR, IC, 17, 1)
> +FIELD(V7M_CCR, BP, 18, 1)
>
> /* V7M SCR bits */
> FIELD(V7M_SCR, SLEEPONEXIT, 1, 1)
> @@ -2842,6 +2844,9 @@ static inline bool
> arm_cpu_data_is_big_endian(CPUARMState *env)
> /* For M profile only, Handler (ie not Thread) mode */
> #define ARM_TBFLAG_HANDLER_SHIFT 21
> #define ARM_TBFLAG_HANDLER_MASK (1 << ARM_TBFLAG_HANDLER_SHIFT)
> +/* For M profile only, whether we should generate stack-limit checks */
> +#define ARM_TBFLAG_STACKCHECK_SHIFT 22
> +#define ARM_TBFLAG_STACKCHECK_MASK (1 << ARM_TBFLAG_STACKCHECK_SHIFT)
>
> /* Bit usage when in AArch64 state */
> #define ARM_TBFLAG_TBI0_SHIFT 0 /* TBI0 for EL0/1 or TBI for EL2/3 */
> @@ -2884,6 +2889,8 @@ static inline bool
> arm_cpu_data_is_big_endian(CPUARMState *env)
> (((F) & ARM_TBFLAG_BE_DATA_MASK) >> ARM_TBFLAG_BE_DATA_SHIFT)
> #define ARM_TBFLAG_HANDLER(F) \
> (((F) & ARM_TBFLAG_HANDLER_MASK) >> ARM_TBFLAG_HANDLER_SHIFT)
> +#define ARM_TBFLAG_STACKCHECK(F) \
> + (((F) & ARM_TBFLAG_STACKCHECK_MASK) >> ARM_TBFLAG_STACKCHECK_SHIFT)
> #define ARM_TBFLAG_TBI0(F) \
> (((F) & ARM_TBFLAG_TBI0_MASK) >> ARM_TBFLAG_TBI0_SHIFT)
> #define ARM_TBFLAG_TBI1(F) \
> diff --git a/target/arm/translate.h b/target/arm/translate.h
> index 45f04244be8..c1b65f3efb0 100644
> --- a/target/arm/translate.h
> +++ b/target/arm/translate.h
> @@ -38,6 +38,7 @@ typedef struct DisasContext {
> int vec_stride;
> bool v7m_handler_mode;
> bool v8m_secure; /* true if v8M and we're in Secure mode */
> + bool v8m_stackcheck; /* true if we need to perform v8M stack limit
> checks */
OK
> /* Immediate value in AArch32 SVC insn; must be set if is_jmp ==
> DISAS_SWI
> * so that top level loop can generate correct syndrome information.
> */
> diff --git a/target/arm/helper.c b/target/arm/helper.c
> index 5e721a65272..6ed8631dbee 100644
> --- a/target/arm/helper.c
> +++ b/target/arm/helper.c
> @@ -12667,6 +12667,16 @@ void cpu_get_tb_cpu_state(CPUARMState *env,
> target_ulong *pc,
> flags |= ARM_TBFLAG_HANDLER_MASK;
> }
>
> + /* v8M always applies stack limit checks unless CCR.STKOFHFNMIGN is
> + * suppressing them because the requested execution priority is less
> than 0.
> + */
> + if (arm_feature(env, ARM_FEATURE_V8) &&
> + arm_feature(env, ARM_FEATURE_M) &&
> + !((mmu_idx & ARM_MMU_IDX_M_NEGPRI) &&
> + (env->v7m.ccr[env->v7m.secure] & R_V7M_CCR_STKOFHFNMIGN_MASK))) {
Specs:
CCR.STKOFHFNMIGN controls whether stack limit violations
are IGNORED while executing at a requested execution priority
that is negative.
The possible values of this bit are:
0 Stack limit faults not ignored.
1 Stack limit faults at requested priorities of less than 0 ignored.
OK, the '!((' notation was hard to read, hopefully the 2 indent spaces
in the 2nd line helped...
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> + flags |= ARM_TBFLAG_STACKCHECK_MASK;
> + }
> +
> *pflags = flags;
> *cs_base = 0;
> }
> diff --git a/target/arm/translate.c b/target/arm/translate.c
> index c6a5d2ac444..751d5811cee 100644
> --- a/target/arm/translate.c
> +++ b/target/arm/translate.c
> @@ -12451,6 +12451,7 @@ static void
> arm_tr_init_disas_context(DisasContextBase *dcbase, CPUState *cs)
> dc->v7m_handler_mode = ARM_TBFLAG_HANDLER(dc->base.tb->flags);
> dc->v8m_secure = arm_feature(env, ARM_FEATURE_M_SECURITY) &&
> regime_is_secure(env, dc->mmu_idx);
> + dc->v8m_stackcheck = ARM_TBFLAG_STACKCHECK(dc->base.tb->flags);
> dc->cp_regs = cpu->cp_regs;
> dc->features = env->features;
>
>
- [Qemu-devel] [PATCH 00/13] target/arm: Implement v8M stack limit checks, Peter Maydell, 2018/10/02
- [Qemu-devel] [PATCH 03/13] target/arm: Move v7m_using_psp() to internals.h, Peter Maydell, 2018/10/02
- [Qemu-devel] [PATCH 01/13] target/arm: Define new TBFLAG for v8M stack checking, Peter Maydell, 2018/10/02
- [Qemu-devel] [PATCH 02/13] target/arm: Define new EXCP type for v8M stack overflows, Peter Maydell, 2018/10/02
- [Qemu-devel] [PATCH 05/13] target/arm: Add some comments in Thumb decode, Peter Maydell, 2018/10/02
- [Qemu-devel] [PATCH 04/13] target/arm: Add v8M stack checks on ADD/SUB/MOV of SP, Peter Maydell, 2018/10/02
- [Qemu-devel] [PATCH 09/13] target/arm: Add v8M stack checks for Thumb2 LDM/STM, Peter Maydell, 2018/10/02