linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: ahs3@redhat.com (Al Stone)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/5] ACPI: Provide better MADT subtable sanity checks
Date: Wed, 16 Sep 2015 13:26:32 -0600	[thread overview]
Message-ID: <55F9C268.5000604@redhat.com> (raw)
In-Reply-To: <55F9988D.5080002@redhat.com>

On 09/16/2015 10:27 AM, Al Stone wrote:
> On 09/16/2015 10:24 AM, Al Stone wrote:
>> On 09/15/2015 08:44 PM, Rafael J. Wysocki wrote:
>>> On Tuesday, September 15, 2015 03:13:12 PM Al Stone wrote:
>>>> On 09/09/2015 03:09 PM, Al Stone wrote:
>>>>> Currently, the BAD_MADT_ENTRY macro is used to do a very simple sanity
>>>>> check on the various subtables that are defined for the MADT.  The check
>>>>> compares the size of the subtable data structure as defined by ACPICA to
>>>>> the length entry in the subtable.  If they are not the same, the assumption
>>>>> is that the subtable is incorrect.
>>>>>
> 
> [snip...]
> 
>>> I'm going to apply this series if people have no problems with it.  I do think
>>> it is slightly overkill, but then as long as it works ...

Indeed it does.  We've found and fixed two MADTs with legitimate errors in them
with these patches (both arm64 tables still in development).

>>> Thanks,
>>> Rafael
>>>
> 
> Would you prefer I send out the updated version with Sudeep's fix to apply
> instead of this version?  Or have you picked that up already?  Easily updated
> and sent out, if it'll make your life simpler....
> 

Bah.  Never mind.  An inadvertent use of the 01day builder by someone else
found another nit; I'll spin out a v4 later today just so everything is very
clear.

-- 
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3 at redhat.com
-----------------------------------

  reply	other threads:[~2015-09-16 19:26 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-09 21:09 [PATCH v3 0/5] ACPI: Provide better MADT subtable sanity checks Al Stone
2015-09-09 21:09 ` [PATCH v3 1/5] ACPI: add in a bad_madt_entry() function to eventually replace the macro Al Stone
2015-09-10 12:14   ` [Linaro-acpi] " Graeme Gregory
2015-09-10 14:13     ` Al Stone
2015-09-09 21:09 ` [PATCH v3 2/5] ACPI / ARM64: remove usage of BAD_MADT_ENTRY/BAD_MADT_GICC_ENTRY Al Stone
2015-09-09 21:09 ` [PATCH v3 3/5] ACPI / IA64: remove usage of BAD_MADT_ENTRY Al Stone
2015-09-09 21:09 ` [PATCH v3 4/5] ACPI / X86: " Al Stone
2015-09-09 21:09 ` [PATCH v3 5/5] ACPI: remove definition of BAD_MADT_ENTRY macro Al Stone
2015-09-10 12:16 ` [Linaro-acpi] [PATCH v3 0/5] ACPI: Provide better MADT subtable sanity checks Graeme Gregory
2015-09-15 21:13 ` Al Stone
2015-09-16  2:44   ` Rafael J. Wysocki
2015-09-16 16:24     ` Al Stone
2015-09-16 16:27       ` Al Stone
2015-09-16 19:26         ` Al Stone [this message]
2015-09-17  0:34         ` Rafael J. Wysocki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55F9C268.5000604@redhat.com \
    --to=ahs3@redhat.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).