[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [Qemu-devel] [PATCH 03/13] target/arm: Move v7m_using_psp
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-arm] [Qemu-devel] [PATCH 03/13] target/arm: Move v7m_using_psp() to internals.h |
Date: |
Wed, 3 Oct 2018 10:52:43 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0 |
On 02/10/2018 18:35, Peter Maydell wrote:
> We're going to want v7m_using_psp() in op_helper.c in the
> next patch, so move it from helper.c to internals.h.
>
> Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> target/arm/internals.h | 15 +++++++++++++++
> target/arm/helper.c | 12 ------------
> 2 files changed, 15 insertions(+), 12 deletions(-)
>
> diff --git a/target/arm/internals.h b/target/arm/internals.h
> index dc9357766c9..bc4c01ccd92 100644
> --- a/target/arm/internals.h
> +++ b/target/arm/internals.h
> @@ -796,4 +796,19 @@ static inline uint32_t
> arm_debug_exception_fsr(CPUARMState *env)
> }
> }
>
> +/**
> + * v7m_using_psp: Return true if using process stack pointer
> + * Return true if the CPU is currently using the process stack
> + * pointer, or false if it is using the main stack pointer.
> + */
> +static inline bool v7m_using_psp(CPUARMState *env)
> +{
> + /* Handler mode always uses the main stack; for thread mode
> + * the CONTROL.SPSEL bit determines the answer.
> + * Note that in v7M it is not possible to be in Handler mode with
> + * CONTROL.SPSEL non-zero, but in v8M it is, so we must check both.
> + */
> + return !arm_v7m_is_handler_mode(env) &&
> + env->v7m.control[env->v7m.secure] & R_V7M_CONTROL_SPSEL_MASK;
> +}
> #endif
> diff --git a/target/arm/helper.c b/target/arm/helper.c
> index c303dc453f1..ef8c244fb84 100644
> --- a/target/arm/helper.c
> +++ b/target/arm/helper.c
> @@ -6554,18 +6554,6 @@ pend_fault:
> return false;
> }
>
> -/* Return true if we're using the process stack pointer (not the MSP) */
> -static bool v7m_using_psp(CPUARMState *env)
> -{
> - /* Handler mode always uses the main stack; for thread mode
> - * the CONTROL.SPSEL bit determines the answer.
> - * Note that in v7M it is not possible to be in Handler mode with
> - * CONTROL.SPSEL non-zero, but in v8M it is, so we must check both.
> - */
> - return !arm_v7m_is_handler_mode(env) &&
> - env->v7m.control[env->v7m.secure] & R_V7M_CONTROL_SPSEL_MASK;
> -}
> -
> /* Write to v7M CONTROL.SPSEL bit for the specified security bank.
> * This may change the current stack pointer between Main and Process
> * stack pointers if it is done for the CONTROL register for the current
>
- [Qemu-arm] [PATCH 00/13] target/arm: Implement v8M stack limit checks, Peter Maydell, 2018/10/02
- [Qemu-arm] [PATCH 02/13] target/arm: Define new EXCP type for v8M stack overflows, Peter Maydell, 2018/10/02
- [Qemu-arm] [PATCH 03/13] target/arm: Move v7m_using_psp() to internals.h, Peter Maydell, 2018/10/02
- [Qemu-arm] [PATCH 01/13] target/arm: Define new TBFLAG for v8M stack checking, Peter Maydell, 2018/10/02
- [Qemu-arm] [PATCH 05/13] target/arm: Add some comments in Thumb decode, Peter Maydell, 2018/10/02
- [Qemu-arm] [PATCH 04/13] target/arm: Add v8M stack checks on ADD/SUB/MOV of SP, Peter Maydell, 2018/10/02
- [Qemu-arm] [PATCH 06/13] target/arm: Add v8M stack checks on exception entry, Peter Maydell, 2018/10/02