Re: [PATCH v3 2/3] ACPI / ARM64: add BAD_MADT_GICC_ENTRY() macro

2015-07-06 Thread Al Stone
On 07/03/2015 05:54 PM, Rafael J. Wysocki wrote: > On Friday, July 03, 2015 01:51:36 PM Al Stone wrote: >> On 07/03/2015 08:06 AM, Catalin Marinas wrote: >>> On Thu, Jul 02, 2015 at 05:48:35PM -0600, Al Stone wrote: diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h >>>

Re: [PATCH v3 2/3] ACPI / ARM64: add BAD_MADT_GICC_ENTRY() macro

2015-07-06 Thread Catalin Marinas
On Sat, Jul 04, 2015 at 01:54:15AM +0200, Rafael J. Wysocki wrote: > On Friday, July 03, 2015 01:51:36 PM Al Stone wrote: > > Perhaps the sanity checking for the MADT subtables needs to be revisited > > and a more general solution provided -- this is not the only MADT subtable > > with this problem

Re: [PATCH v3 2/3] ACPI / ARM64: add BAD_MADT_GICC_ENTRY() macro

2015-07-03 Thread Rafael J. Wysocki
On Friday, July 03, 2015 01:51:36 PM Al Stone wrote: > On 07/03/2015 08:06 AM, Catalin Marinas wrote: > > On Thu, Jul 02, 2015 at 05:48:35PM -0600, Al Stone wrote: > >> diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h > >> index 39248d3..a3c26a4 100644 > >> --- a/arch/arm6

Re: [PATCH v3 2/3] ACPI / ARM64: add BAD_MADT_GICC_ENTRY() macro

2015-07-03 Thread Al Stone
On 07/03/2015 08:06 AM, Catalin Marinas wrote: > On Thu, Jul 02, 2015 at 05:48:35PM -0600, Al Stone wrote: >> diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h >> index 39248d3..a3c26a4 100644 >> --- a/arch/arm64/include/asm/acpi.h >> +++ b/arch/arm64/include/asm/acpi.h >>

Re: [PATCH v3 2/3] ACPI / ARM64: add BAD_MADT_GICC_ENTRY() macro

2015-07-03 Thread Catalin Marinas
On Thu, Jul 02, 2015 at 05:48:35PM -0600, Al Stone wrote: > diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h > index 39248d3..a3c26a4 100644 > --- a/arch/arm64/include/asm/acpi.h > +++ b/arch/arm64/include/asm/acpi.h > @@ -19,6 +19,17 @@ > #include > #include > > +/*

[PATCH v3 2/3] ACPI / ARM64: add BAD_MADT_GICC_ENTRY() macro

2015-07-02 Thread Al Stone
The BAD_MADT_ENTRY() macro is designed to work for all of the subtables of the MADT. In the ACPI 5.1 version of the spec, the struct for the GICC subtable (struct acpi_madt_generic_interrupt) is 76 bytes long; in ACPI 6.0, the struct is 80 bytes long. But, there is only one definition in ACPICA f