diff options
| author | Peter Maydell | 2016-02-23 16:36:44 +0100 |
|---|---|---|
| committer | Peter Maydell | 2016-02-26 16:09:41 +0100 |
| commit | 58ae2d1f037fae1d90eed4522053a85d79edfbec (patch) | |
| tree | 072a63c8bd2fe6e87fb3490ff896e664ea2c6a3e /target-arm/helper.c | |
| parent | target-arm: Add Hyp mode checks to bad_mode_switch() (diff) | |
| download | qemu-58ae2d1f037fae1d90eed4522053a85d79edfbec.tar.gz qemu-58ae2d1f037fae1d90eed4522053a85d79edfbec.tar.xz qemu-58ae2d1f037fae1d90eed4522053a85d79edfbec.zip | |
target-arm: Forbid mode switch to Mon from Secure EL1
In v8 trying to switch mode to Mon from Secure EL1 is an
illegal mode switch. (In v7 this is impossible as all secure
modes except User are at EL3.) We can handle this case by
making a switch to Mon valid only if the current EL is 3,
which then gives the correct answer whether EL3 is AArch32
or AArch64.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Sergey Fedorov <serge.fdrv@gmail.com>
Message-id: 1455556977-3644-9-git-send-email-peter.maydell@linaro.org
Diffstat (limited to 'target-arm/helper.c')
| -rw-r--r-- | target-arm/helper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target-arm/helper.c b/target-arm/helper.c index c43d66fd8d..5926b15506 100644 --- a/target-arm/helper.c +++ b/target-arm/helper.c @@ -5222,7 +5222,7 @@ static int bad_mode_switch(CPUARMState *env, int mode) return !arm_feature(env, ARM_FEATURE_EL2) || arm_current_el(env) < 2 || arm_is_secure(env); case ARM_CPU_MODE_MON: - return !arm_is_secure(env); + return arm_current_el(env) < 3; default: return 1; } |