Re: [PATCH 1/2] arm64: KVM: Fix AArch32 to AArch64 register mapping

2015-11-17 Thread Robin Murphy
Hi Marc, On 16/11/15 10:28, Marc Zyngier wrote: When running a 32bit guest under a 64bit hypervisor, the ARMv8 architecture defines a mapping of the 32bit registers in the 64bit space. This includes banked registers that are being demultiplexed over the 64bit ones. On exception caused by an

Re: [PATCH 12/21] arm64: KVM: Implement fpsimd save/restore

2015-11-17 Thread Marc Zyngier
On 17/11/15 11:49, Steve Capper wrote: > On 17 November 2015 at 11:25, Marc Zyngier wrote: >> On 17/11/15 11:13, Steve Capper wrote: >>> On 16 November 2015 at 13:11, Marc Zyngier wrote: Implement the fpsimd save restore, keeping the lazy part in

Re: [PATCH 12/21] arm64: KVM: Implement fpsimd save/restore

2015-11-17 Thread Marc Zyngier
On 17/11/15 11:13, Steve Capper wrote: > On 16 November 2015 at 13:11, Marc Zyngier wrote: >> Implement the fpsimd save restore, keeping the lazy part in >> assembler (as returning to C would be overkill). >> >> Signed-off-by: Marc Zyngier >> --- >>

Re: [PATCH 2/2] arm64: KVM: Add workaround for Cortex-A57 erratum 834220

2015-11-17 Thread Will Deacon
Hi Marc, On Mon, Nov 16, 2015 at 10:28:18AM +, Marc Zyngier wrote: > Cortex-A57 parts up to r1p2 can misreport Stage 2 translation faults > when a Stage 1 permission fault or device alignment fault should > have been reported. > > This patch implements the workaround (which is to validate

Re: [PATCH 12/21] arm64: KVM: Implement fpsimd save/restore

2015-11-17 Thread Steve Capper
On 16 November 2015 at 13:11, Marc Zyngier wrote: > Implement the fpsimd save restore, keeping the lazy part in > assembler (as returning to C would be overkill). > > Signed-off-by: Marc Zyngier > --- > arch/arm64/kvm/hyp/Makefile | 1 + >

Re: [PATCH 12/21] arm64: KVM: Implement fpsimd save/restore

2015-11-17 Thread Steve Capper
On 17 November 2015 at 11:25, Marc Zyngier wrote: > On 17/11/15 11:13, Steve Capper wrote: >> On 16 November 2015 at 13:11, Marc Zyngier wrote: >>> Implement the fpsimd save restore, keeping the lazy part in >>> assembler (as returning to C would be