[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/10] armv7m: R14 should reset to 0xffffffff
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH 10/10] armv7m: R14 should reset to 0xffffffff |
Date: |
Fri, 27 Jan 2017 13:58:01 +0000 |
User-agent: |
mu4e 0.9.19; emacs 25.1.91.4 |
Peter Maydell <address@hidden> writes:
> For M profile (unlike A profile) the reset value of R14 is specified
> as 0xffffffff. (The rationale is that this is an illegal exception
> return value, so if guest code tries to return to it it will result
> in a helpful exception.)
>
> Registers r0 to r12 and the flags are architecturally UNKNOWN on
> reset, so we leave those at zero.
>
> Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
> ---
> target/arm/cpu.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/target/arm/cpu.c b/target/arm/cpu.c
> index 0814f73..e9f10f7 100644
> --- a/target/arm/cpu.c
> +++ b/target/arm/cpu.c
> @@ -196,6 +196,9 @@ static void arm_cpu_reset(CPUState *s)
> */
> env->v7m.ccr = R_V7M_CCR_STKALIGN_MASK;
>
> + /* Unlike A/R profile, M profile defines the reset LR value */
> + env->regs[14] = 0xffffffff;
> +
> /* Load the initial SP and PC from the vector table at address 0 */
> rom = rom_ptr(0);
> if (rom) {
--
Alex Bennée
- [Qemu-devel] [PATCH 00/10] More M profile bugfixes, Peter Maydell, 2017/01/24
- [Qemu-devel] [PATCH 01/10] target/arm: Drop IS_M() macro, Peter Maydell, 2017/01/24
- [Qemu-devel] [PATCH 07/10] armv7m: Report no-coprocessor faults correctly, Peter Maydell, 2017/01/24
- [Qemu-devel] [PATCH 09/10] armv7m: FAULTMASK should be 0 on reset, Peter Maydell, 2017/01/24
- [Qemu-devel] [PATCH 10/10] armv7m: R14 should reset to 0xffffffff, Peter Maydell, 2017/01/24
- Re: [Qemu-devel] [PATCH 10/10] armv7m: R14 should reset to 0xffffffff,
Alex Bennée <=
- [Qemu-devel] [PATCH 08/10] armv7m: Honour CCR.USERSETMPEND, Peter Maydell, 2017/01/24
- [Qemu-devel] [PATCH 04/10] armv7m: implement CCR, CFSR, HFSR, DFSR, BFAR, and MMFAR, Peter Maydell, 2017/01/24
- [Qemu-devel] [PATCH 02/10] armv7m_nvic: keep a pointer to the CPU, Peter Maydell, 2017/01/24
- [Qemu-devel] [PATCH 03/10] armv7m: add state for v7M CCR, CFSR, HFSR, DFSR, MMFAR, BFAR, Peter Maydell, 2017/01/24