[PATCHv2 0/7] arm64/kvm: common ESR_ELx definitions and decoding

[Christoffer Dall]

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 [1]:
>> > * 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?

-Christoffer