[PATCHv2 0/7] arm64/kvm: common ESR_ELx definitions and decoding
catalin.marinas at arm.com
Thu Jan 15 07:12:02 PST 2015
On Thu, Jan 15, 2015 at 01:08:00PM +0000, Christoffer Dall wrote:
> On Thu, Jan 15, 2015 at 1:42 PM, Mark Rutland <mark.rutland at arm.com> wrote:
> > On Mon, Jan 12, 2015 at 07:44:40PM +0000, Christoffer Dall wrote:
> >> On Mon, Jan 12, 2015 at 04:45:13PM +0000, Mark Rutland wrote:
> >> > Currently we have two sets of macros used for ESR_ELx handling, one used
> >> > by core arm64 code and the other used by KVM. These differ slightly in
> >> > naming convention and style of definition.
> >> >
> >> > This patch series introduces and migrates all users to a common set of
> >> > macros for ESR_ELx handling, preventing further drift.
> >> >
> >> > Additionally this series adds exception class decoding when reporting
> >> > exceptions, saving deveopers from having to perform tedious mental
> >> > arithmetic to figure out what the likely cause of an unexpected
> >> > exception was.
> >> >
> >> > Since v1 :
> >> > * Reorder patches to maintain KVM bisectability.
> >> > * Fix bad definitions (ESR_ELx_SAS and ESR_ELx_FSC_PERM).
> >> > * Introcuce ESR_ELx_SAS_SHIFT and undo bad rework of
> >> > kvm_vcpu_dabt_get_as.
> >> > * Make "Unallocated EC" comments consistent in ESR_ELx_EC_* definition
> >> > list.
> >> >
> >> For the series:
> >> Reviewed-by: Christoffer Dall <christoffer.dall at linaro.org>
> >> I also tested this with KVM on APM XGene and Juno with UEFI+Linux as a
> >> guest.
> > Thanks, Christoffer. It's much appreciated.
> > Catalin, Christoffer, do you have a preferred path for merging this?
> > Should this go through the arm64 or KVM tree as a whole?
> Would make sense to merge it as whole.
> I don't care which path, let's
> just minimize conflicts. For big things, I have GICv3, dirty page
> logging, and Marc's fixes, but they shouldn't touch much of the same
> code (maybe the same files, but I don't think the same lines), so it's
> up to you Catalin?
I don't have a strong preference but since Christoffer doesn't have one
either, I'll merge them via the arm64 tree (there are 4 arm64 patches vs
3 kvm ones ;)).
More information about the linux-arm-kernel