On 24 March 2015 at 14:32, Greg Bellows <greg.bellows@xxxxxxxxxx> wrote:
> On Mon, Mar 23, 2015 at 12:05 PM, Alex Bennée <alex.bennee@xxxxxxxxxx> wrote:
>> From: Peter Maydell <peter.maydell@xxxxxxxxxx>
>> @@ -523,7 +523,7 @@ void aarch64_cpu_do_interrupt(CPUState *cs)
>> aarch64_save_sp(env, arm_current_el(env));
>> env->elr_el[new_el] = env->pc;
>> } else {
>> - env->banked_spsr[0] = cpsr_read(env);
>> + env->banked_spsr[aarch64_banked_spsr_index(new_el)] = cpsr_read(env);
>
> Are the other banks (2-5) only used for KVM? It seems we go out of
> our way to manage this larger SPSR array then not use all of the slots
> in QEMU itself.
They're used in AArch32 (where they are the SPSR for various
32 bit modes). In AArch64 you can access those registers via
MSR/MRS (we probably haven't implemented those yet because they
are only accessible at EL2 and above) so hypervisors can do
worldswitches. But for exception entry and return (which is
what this code is) we only use SPSR_EL0/SPSR_EL1/SPSR_EL2/SPSR_EL3
which is a subset of the AArch32 SPSRs.
-- PMM
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
