qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Qemu-arm] [PATCH] target/arm: Start AArch32 CPUs with


From: Philippe Mathieu-Daudé
Subject: Re: [Qemu-devel] [Qemu-arm] [PATCH] target/arm: Start AArch32 CPUs with EL2 but not EL3 in Hyp mode
Date: Tue, 25 Sep 2018 13:56:27 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0

On 8/23/18 3:50 PM, Peter Maydell wrote:
> The ARMv8 architecture defines that an AArch32 CPU starts
> in SVC mode, unless EL2 is the highest available EL, in
> which case it starts in Hyp mode. (In ARMv7 a CPU with EL2
> but not EL3 was not a valid configuration, but we don't
> specifically reject this if the user asks for one.)

ID090413 "G1.13 AArch32 state exception descriptions"

  G1.13.1 Reset

    When Reset is deasserted, the PE then starts executing
    instructions in the highest implemented Exception level.
    If that Exception level is using AArch32, then it starts
    execution:
    • In Secure state, if the implementation includes EL3.
    • With interrupts disabled:
    — In Hyp mode, if the highest implemented Exception level is EL2.
    — In Supervisor mode, otherwise.

> Signed-off-by: Peter Maydell <address@hidden>

Reviewed-by: Philippe Mathieu-Daudé <address@hidden>

> ---
>  target/arm/cpu.c | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/target/arm/cpu.c b/target/arm/cpu.c
> index 258ba6dcaad..b5e61cc1775 100644
> --- a/target/arm/cpu.c
> +++ b/target/arm/cpu.c
> @@ -199,8 +199,18 @@ static void arm_cpu_reset(CPUState *s)
>          env->cp15.c15_cpar = 1;
>      }
>  #else
> -    /* SVC mode with interrupts disabled.  */
> -    env->uncached_cpsr = ARM_CPU_MODE_SVC;
> +
> +    /*
> +     * If the highest available EL is EL2, AArch32 will start in Hyp
> +     * mode; otherwise it starts in SVC. Note that if we start in
> +     * AArch64 then these values in the uncached_cpsr will be ignored.
> +     */
> +    if (arm_feature(env, ARM_FEATURE_EL2) &&
> +        !arm_feature(env, ARM_FEATURE_EL3)) {
> +        env->uncached_cpsr = ARM_CPU_MODE_HYP;
> +    } else {
> +        env->uncached_cpsr = ARM_CPU_MODE_SVC;
> +    }
>      env->daif = PSTATE_D | PSTATE_A | PSTATE_I | PSTATE_F;
>  
>      if (arm_feature(env, ARM_FEATURE_M)) {
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]