lkml.org 
[lkml]   [2021]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.14 446/849] ACPI: PM: Turn off unused wakeup power resources
    Date
    From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

    [ Upstream commit 7a63296d6f579a02b2675b4b0fe5b1cd3235e8d3 ]

    If an ACPI power resource is found to be "on" during the
    initialization of the list of wakeup power resources of a device,
    it is reference counted and its wakeup_enabled flag is set, which is
    problematic if the deivce in question is the only user of the given
    power resource, it is never runtime-suspended and it is not allowed
    to wake up the system from sleep, because in that case the given
    power resource will stay "on" until the system reboots and energy
    will be wasted.

    It is better to simply turn off wakeup power resources that are "on"
    during the initialization unless their reference counters are not
    zero, because that may be the only opportunity to prevent them from
    staying in the "on" state all the time.

    Fixes: b5d667eb392e ("ACPI / PM: Take unusual configurations of power resources into account")
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/acpi/power.c | 19 +++++++++----------
    1 file changed, 9 insertions(+), 10 deletions(-)

    diff --git a/drivers/acpi/power.c b/drivers/acpi/power.c
    index eba7785047cad..dfe760bd7157f 100644
    --- a/drivers/acpi/power.c
    +++ b/drivers/acpi/power.c
    @@ -606,20 +606,19 @@ int acpi_power_wakeup_list_init(struct list_head *list, int *system_level_p)

    list_for_each_entry(entry, list, node) {
    struct acpi_power_resource *resource = entry->resource;
    - int result;
    u8 state;

    mutex_lock(&resource->resource_lock);

    - result = acpi_power_get_state(resource, &state);
    - if (result) {
    - mutex_unlock(&resource->resource_lock);
    - return result;
    - }
    - if (state == ACPI_POWER_RESOURCE_STATE_ON) {
    - resource->ref_count++;
    - resource->wakeup_enabled = true;
    - }
    + /*
    + * Make sure that the power resource state and its reference
    + * counter value are consistent with each other.
    + */
    + if (!resource->ref_count &&
    + !acpi_power_get_state(resource, &state) &&
    + state == ACPI_POWER_RESOURCE_STATE_ON)
    + __acpi_power_off(resource);
    +
    if (system_level > resource->system_level)
    system_level = resource->system_level;

    --
    2.33.0


    \
     
     \ /
      Last update: 2021-11-16 00:03    [W:3.540 / U:0.108 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site