Re: [Linaro-acpi] [PATCH v5 0/5] Provide better MADT subtable sanity checks

Hanjun Guo
2015-10-12T03:49:01Z

On 10/12/2015 11:08 AM, Pat Erley wrote:
> On 10/05/2015 10:12 AM, Al Stone wrote:
>> On 10/05/2015 07:39 AM, Rafael J. Wysocki wrote:
>>> On Wednesday, September 30, 2015 10:10:16 AM Al Stone wrote:
>>>> On 09/30/2015 03:00 AM, Hanjun Guo wrote:
>>>>> On 2015/9/30 7:45, Al Stone wrote:
>>>>>> NB: this patch set is for use against the linux-pm bleeding edge
>>>>>> branch.
>>>>>>
>>>>
>>>> [snip...]
>>>>
>>>>>
>>>>> For this patch set,
>>>>>
>>>>> Reviewed-by: Hanjun Guo <hanjun.guo@linaro.org>
>>>>>
>>>>> Thanks
>>>>> Hanjun
>>>>
>>>> Thanks, Hanjun!
>>>
>>> Series applied, thanks!
>>>
>>> Rafael
>>>
>>
>> Thanks, Rafael!
>>
>
> Just decided to test out linux-next (to see the new nouveau cleanups).
> This change set prevents my Lenovo W510 from booting properly.
>
> Reverting: 7494b0 "ACPI: add in a bad_madt_entry() function to
> eventually replace the macro"
>
> Gets the system booting again. I'm attaching my dmesg from the failed
> boot, who wants the acpidump?

[ 0.000000] ACPI: undefined version for either FADT 4.0 or MADT 1
[ 0.000000] ACPI: Error parsing LAPIC address override entry
[ 0.000000] ACPI: Invalid BIOS MADT, disabling ACPI

Seems the MADT revision is not right, could you dump the ACPI MADT
(APIC) table and send it out? I will take a look :)

Thanks
Hanjun