From mboxrd@z Thu Jan 1 00:00:00 1970 From: will.deacon@arm.com (Will Deacon) Date: Wed, 4 Jul 2018 15:01:22 +0100 Subject: [PATCH 6/7] kvm/arm: use PSR_AA32 definitions In-Reply-To: References: <20180625144421.11511-1-mark.rutland@arm.com> <20180625144421.11511-7-mark.rutland@arm.com> <20180704131232.GB2798@arm.com> Message-ID: <20180704140121.GD4828@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Jul 04, 2018 at 02:23:05PM +0100, Marc Zyngier wrote: > On 04/07/18 14:12, Will Deacon wrote: > > On Mon, Jun 25, 2018 at 03:44:20PM +0100, Mark Rutland wrote: > >> Some code cares about the SPSR_ELx format for exceptions taken from > >> AArch32 to inspect or manipulate the SPSR_ELx value, which is already in > >> the SPSR_ELx format, and not in the AArch32 PSR format. > >> > >> To separate these from cases where we care about the AArch32 PSR format, > >> migrate these cases to use the PSR_AA32_* definitions rather than > >> COMPAT_PSR_*. > >> > >> There should be no functional change as a result of this patch. > >> > >> Note that arm64 KVM does not support a compat KVM API, and always uses > >> the SPSR_ELx format, even for AArch32 guests. > >> > >> Signed-off-by: Mark Rutland > >> Cc: Christoffer Dall > >> Cc: Marc Zyngier > >> --- > >> arch/arm/include/asm/kvm_emulate.h | 14 +++++++------- > >> arch/arm64/include/asm/kvm_emulate.h | 10 +++++----- > >> arch/arm64/kvm/guest.c | 14 +++++++------- > >> arch/arm64/kvm/hyp/vgic-v2-cpuif-proxy.c | 2 +- > >> arch/arm64/kvm/regmap.c | 22 +++++++++++----------- > >> arch/arm64/kvm/reset.c | 4 ++-- > >> virt/kvm/arm/aarch32.c | 20 ++++++++++---------- > >> 7 files changed, 43 insertions(+), 43 deletions(-) > > > > I need an ack from the KVM maintainers before I can take this. I can also > > put this series on a separate branch if necessary. > > Feel free to add my > > Acked-by: Marc Zyngier > > I don't have much queued for KVM ATM, so there is no need for a separate > branch. Pending stuff shouldn't conflict (famous last word...). Brill; thanks for the quick reply, Marc. Will