Comment 11 for bug 1406767

Revision history for this message
Nikola Nikolov (nikolov-tmw) wrote : Re: High ACPI event count on Asus K750L, causing ~12% non-stop CPU utilization

Hi Christopher,

git bisect found the first good commit, here's the full output from the last command:

bd9b2f9aff26c185c1f8e0cd08a850ee4ace391a is the first bad commit
commit bd9b2f9aff26c185c1f8e0cd08a850ee4ace391a
Author: Rafael J. Wysocki <email address hidden>
Date: Mon Jul 14 22:41:41 2014 +0200

    ACPI / scan: No implicit wake notification for buttons

    The ACPI device enumeration code in Linux assumes that buttons always
    are wakeup devices, so it calls acpi_setup_gpe_for_wake() for them
    which leads to undesirable side effects. Namely, that function sets
    up implicit device wake notification mechanism for a given GPE if
    there is no handler method in the ACPI namespace, which from the
    ACPICA's perspective means that there always is a way to handle
    that GPE if enabled. However, we don't handle wake notify events
    for buttons, so if there are no handler methods for their GPEs in
    the namespace, enabling a button GPE at run time leads to a GPE
    storm in some cases (the GPE triggers, ACPICA carries out the
    implicit wake notification for it which isn't handled, so the
    GPE triggers again and so on).

    To prevent that from happening use acpi_mark_gpe_for_wake()
    instead of acpi_setup_gpe_for_wake() for buttons which will cause
    ACPICA to only enable button GPEs if there are handler methods for
    the in the namespace.

    Signed-off-by: Rafael J. Wysocki <email address hidden>

:040000 040000 fec6e6222f6b0f58a5106035c38c7091a48eef0c f6c36ad741c62668891689c8229c3703146dc92e M drivers

Let me know if there's anything else I need to do,
Nikola