On 22/07/16 15:38, Andrew Jones wrote: > We need to set cpsr before determining the spsr bank, as the bank > depends on the target exception level of the injection, not the > current mode of the vcpu. Normally this is one in the same (EL1), > but not when we manage to trap an EL0 fault. It still doesn't really > matter for the 64-bit EL0 case though, as vcpu_spsr() unconditionally > uses the EL1 bank for that. However the 32-bit EL0 case gets fun, as > that path will lead to the BUG() in vcpu_spsr32() > > This patch fixes the assignment order and also modifies some white > space in order to better group pairs of lines that have strict order. > > Signed-off-by: Andrew Jones <drjones@xxxxxxxxxx> Ouch. Nice catch. Can you find what's the earliest release this would cleanly apply to? Definitely deserves a CC stable. Thanks, M. -- Jazz is not dead. It just smells funny... -- 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
