Message ID | 1506092407-26985-5-git-send-email-peter.maydell@linaro.org |
---|---|
State | Superseded |
Headers | show |
Series | ARM v8M: exception entry, exit and security | expand |
On 09/22/2017 10:59 AM, Peter Maydell wrote: > Now that we can handle the CONTROL.SPSEL bit not necessarily being > in sync with the current stack pointer, we can restore the correct > security state on exception return. This happens before we start > to read registers off the stack frame, but after we have taken > possible usage faults for bad exception return magic values and > updated CONTROL.SPSEL. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> > --- > target/arm/helper.c | 2 ++ > 1 file changed, 2 insertions(+) Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~
diff --git a/target/arm/helper.c b/target/arm/helper.c index 509a1aa..a3c63c3 100644 --- a/target/arm/helper.c +++ b/target/arm/helper.c @@ -6371,6 +6371,8 @@ static void do_v7m_exception_exit(ARMCPU *cpu) */ write_v7m_control_spsel(env, return_to_sp_process); + switch_v7m_security_state(env, return_to_secure); + { /* The stack pointer we should be reading the exception frame from * depends on bits in the magic exception return type value (and
Now that we can handle the CONTROL.SPSEL bit not necessarily being in sync with the current stack pointer, we can restore the correct security state on exception return. This happens before we start to read registers off the stack frame, but after we have taken possible usage faults for bad exception return magic values and updated CONTROL.SPSEL. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> --- target/arm/helper.c | 2 ++ 1 file changed, 2 insertions(+) -- 2.7.4