From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3BF2FC433DB for ; Thu, 31 Dec 2020 20:47:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0AC182079A for ; Thu, 31 Dec 2020 20:47:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726693AbgLaUqz (ORCPT ); Thu, 31 Dec 2020 15:46:55 -0500 Received: from cloudserver094114.home.pl ([79.96.170.134]:56294 "EHLO cloudserver094114.home.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726210AbgLaUqz (ORCPT ); Thu, 31 Dec 2020 15:46:55 -0500 Received: from 89-77-60-66.dynamic.chello.pl (89.77.60.66) (HELO kreacher.localnet) by serwer1319399.home.pl (79.96.170.134) with SMTP (IdeaSmtpServer 0.83.537) id adb218dee83d9126; Thu, 31 Dec 2020 21:46:12 +0100 From: "Rafael J. Wysocki" To: Sebastian Andrzej Siewior , Stephen Berman Cc: Zhang Rui , Robert Moore , Erik Kaneda , Len Brown , Thomas Gleixner , Peter Zijlstra , Linux Kernel Mailing List , ACPI Devel Maling List , "open list:ACPI COMPONENT ARCHITECTURE (ACPICA)" , "Rafael J. Wysocki" Subject: Re: power-off delay/hang due to commit 6d25be57 (mainline) Date: Thu, 31 Dec 2020 21:46:11 +0100 Message-ID: <9709109.MH8tSaV5v9@kreacher> In-Reply-To: References: <87blkbx1gt.fsf@gmx.net> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org On Wednesday, December 2, 2020 8:13:38 PM CET Rafael J. Wysocki wrote: > On Wed, Dec 2, 2020 at 7:31 PM Rafael J. Wysocki wrote: > > > > On Wed, Dec 2, 2020 at 7:03 PM Sebastian Andrzej Siewior > > wrote: > > > > > > On 2020-10-26 18:20:59 [+0100], To Rafael J. Wysocki wrote: > > > > > > > > Done as Bug 208877. > > > > > > Rafael, do you have any suggestions? > > > > > > > > > > I've lost track of this sorry. > > > > > > > > > > I have ideas, let me get back to this next week. > > > > > > > > :) > > > > > > Rafael, any update? If you outline an idea or so then I may be able to > > > form a patch out of it. Otherwise I have no idea how to fix this - other > > > than telling the driver to not poll in smaller intervals than > > > 30secs. > > > > The idea, roughly speaking, is to limit the number of outstanding work > > items in the queue (basically, if there's a notification occurring > > before the previous one can be handled, there is no need to queue up > > another work item for it). > > That's easier said than done, though, because of the way the work item > queue-up is hooked up into the ACPICA code. So scratch this and it wouldn't work in general anyway AFAICS. ATM, I'm tempted to do something like the patch below (with the rationale that it shouldn't be necessary to read the temperature right after updating the trip points if polling is in use, because the next update through polling will cause it to be read anyway and it will trigger trip point actions as needed). Stephen, can you give it a go, please? --- drivers/acpi/thermal.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) Index: linux-pm/drivers/acpi/thermal.c =================================================================== --- linux-pm.orig/drivers/acpi/thermal.c +++ linux-pm/drivers/acpi/thermal.c @@ -911,24 +911,25 @@ static void acpi_thermal_notify(struct a switch (event) { case ACPI_THERMAL_NOTIFY_TEMPERATURE: acpi_thermal_check(tz); - break; + return; case ACPI_THERMAL_NOTIFY_THRESHOLDS: acpi_thermal_trips_update(tz, ACPI_TRIPS_REFRESH_THRESHOLDS); - acpi_thermal_check(tz); - acpi_bus_generate_netlink_event(device->pnp.device_class, - dev_name(&device->dev), event, 0); break; case ACPI_THERMAL_NOTIFY_DEVICES: acpi_thermal_trips_update(tz, ACPI_TRIPS_REFRESH_DEVICES); - acpi_thermal_check(tz); - acpi_bus_generate_netlink_event(device->pnp.device_class, - dev_name(&device->dev), event, 0); break; default: ACPI_DEBUG_PRINT((ACPI_DB_INFO, "Unsupported event [0x%x]\n", event)); - break; + return; } + + /* Trigger an update of the thermal zone unless polling is in use. */ + if (!tz->polling_frequency) + acpi_thermal_check(tz); + + acpi_bus_generate_netlink_event(device->pnp.device_class, + dev_name(&device->dev), event, 0); } /*