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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0DA66C433F5 for ; Wed, 16 Mar 2022 16:00:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240477AbiCPQBx (ORCPT ); Wed, 16 Mar 2022 12:01:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55728 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237312AbiCPQBx (ORCPT ); Wed, 16 Mar 2022 12:01:53 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id D493F46640 for ; Wed, 16 Mar 2022 09:00:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1647446436; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=fpm1syxt/668a84vrAP57Sr17b94ynmh10cvSiUgK9Q=; b=L/CFSx4HyNpnxT0ci68eG1XP6fU/2si17A++fQ5XtDofu8nEll0hzHNA7Y77esFVROcSuG SLGvM5dYWxjV6W022lK8ynYykpdOnyyAeylA5BSjWFMMUvwPVwleiu5IGPIlUXrx1Aw/9Q EI7OYjsXl8llSY3JPXwwrncEKOpqMCs= Received: from mail-ed1-f70.google.com (mail-ed1-f70.google.com [209.85.208.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-584-pgbFC8rtMgyfHxretRn76w-1; Wed, 16 Mar 2022 12:00:33 -0400 X-MC-Unique: pgbFC8rtMgyfHxretRn76w-1 Received: by mail-ed1-f70.google.com with SMTP id cf6-20020a0564020b8600b00415e9b35c81so1563700edb.9 for ; Wed, 16 Mar 2022 09:00:33 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=fpm1syxt/668a84vrAP57Sr17b94ynmh10cvSiUgK9Q=; b=f5eXBnpXx8qbYLqR1I9q4bkgXKph5zL1xxPsHr+k31N+sNNsLDadKKLmWJTryOoy5T eIKZAZG7k1Wl1T2Q0qSX6O471wyMWsxxrJLHP/Is5/1iaKwwR2lCPWQvh2yCMC/NCfII N0lWfaO/IsU1rm6/+u83aQPc9f5H0CpylCp62xiWY2x/ga0GtW03wr4RMADk39gW4taD 6Lr5hTYkaQdlFklxeiEWXAanYOWOpWLJvE6Lrftc4MscnMn5Wb0mD5VgGVV5nev+aYY6 NXa2F6GN910m1UOWkQQ0ziiRPnXNhSvYgc2wx08l4LlmwhCQwyHz+SZPxAwFnYiOLlxU BMdA== X-Gm-Message-State: AOAM533z7WZQLyvX/dl9JKt+dJn7DipSNZD52hFkaRJMlqVVSzHO2/NM 8UBrPhGGXXivgtGdxDIA6P5Q3SDpH13MPv5t9jvxcG7ikXXGfoKlz1LXYbD9Flbv1TyN7pI6qSA KlOF6CZOzcGOOfg1q X-Received: by 2002:a17:907:7f23:b0:6de:6ac2:93c with SMTP id qf35-20020a1709077f2300b006de6ac2093cmr575150ejc.462.1647446432001; Wed, 16 Mar 2022 09:00:32 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwlB7LYYCh9S/mTWzBMqQ7g5VxJEALZHNJjeZJne97yd3oQfZ+UdtxkWSZrNaP55HxNrbBWUw== X-Received: by 2002:a17:907:7f23:b0:6de:6ac2:93c with SMTP id qf35-20020a1709077f2300b006de6ac2093cmr575116ejc.462.1647446431725; Wed, 16 Mar 2022 09:00:31 -0700 (PDT) Received: from ?IPV6:2001:1c00:c1e:bf00:cdb2:2781:c55:5db0? (2001-1c00-0c1e-bf00-cdb2-2781-0c55-5db0.cable.dynamic.v6.ziggo.nl. [2001:1c00:c1e:bf00:cdb2:2781:c55:5db0]) by smtp.gmail.com with ESMTPSA id h30-20020a056402095e00b00412b81dd96esm1173739edz.29.2022.03.16.09.00.31 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 16 Mar 2022 09:00:31 -0700 (PDT) Message-ID: Date: Wed, 16 Mar 2022 17:00:30 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.4.0 Subject: Re: Many reports of laptops getting hot while suspended with kernels >= 5.16.10 || >= 5.17-rc1 Content-Language: en-US To: "Limonciello, Mario" , "Rafael J. Wysocki" Cc: Linux PM , Stable , Justin Forbes , Mark Pearson , ACPI Devel Maling List , Kai-Heng Feng References: <31b9d1cd-6a67-218b-4ada-12f72e6f00dc@redhat.com> <5fb0cbe8-5f9d-1c75-ae0a-5909624189d3@redhat.com> From: Hans de Goede In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org Hi, On 3/16/22 14:37, Limonciello, Mario wrote: > [Public] > >>> Just FWIW this fix that was backported to stable also fixed keyboard >>> wakeup from s2idle on a number of HP laptops too. I know for sure that >>> it fixed it on the AMD versions of them, and Kai Heng Feng suspected it >>> will also fix it for the Intel versions. So if there is another commit >>> that can be backported from 5.17 to make it safer for the other systems, >>> I think we should consider doing that to solve it too. >> >> There is a series of ACPI EC driver commits that are present in >> 5.17-rc, but have not been included in any "stable" series: >> >> befd9b5b0c62 ACPI: EC: Relocate acpi_ec_create_query() and drop >> acpi_ec_delete_query() >> c33676aa4824 ACPI: EC: Make the event work state machine visible >> c793570d8725 ACPI: EC: Avoid queuing unnecessary work in >> acpi_ec_submit_event() >> eafe7509ab8c ACPI: EC: Rename three functions >> a105acd7e384 ACPI: EC: Simplify locking in acpi_ec_event_handler() >> 388fb77dcf97 ACPI: EC: Rearrange the loop in acpi_ec_event_handler() >> 98d364509d77 ACPI: EC: Fold acpi_ec_check_event() into >> acpi_ec_event_handler() >> 1f2350443dd2 ACPI: EC: Pass one argument to acpi_ec_query() >> ca8283dcd933 ACPI: EC: Call advance_transaction() from >> acpi_ec_dispatch_gpe() >> >> It is likely that they prevent the problem exposed by the problematic >> commit from occurring, but I'm not sure which ones do that. Some of >> them are clearly cosmetic, but the ordering matters. > > Hans, > > Do you think you could get one of the folks who reported this regression to do > a bisect to see which one "fixed" it? We already know which commit is causing the regression. As Rafael already said the question is why things are not broken in 5.17 and that is not a straight forward bisect. So figuring this out is going to be a lot of work and I'm not sure of that it is worth it. I certainly don't have time to help users with debugging this. > If we get lucky we can come down to > some smaller hunks of code that can come back to stable instead of reverting. 5.17 is almost done and in a couple of weeks Fedora (and Arch and other distros tracking the mainline kernel) will move to 5.17 resolving the wakeup by keyboard issue not working there. 5.16 is not a LTS kernel, so for other distros we would at a minimum figure out what needs to be backported to make things work with 5.15 making the delta / set of possible patches we need even bigger. So as already said IMHO this is not worth it, at least assuming that nothing bad happens when attempting wakeup by keyboard, iow it just does not work and does not put the laptop in some bad state? Regards, Hans