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=-7.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 4D817C4361B for ; Sun, 13 Dec 2020 11:03:31 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 123EA224F4 for ; Sun, 13 Dec 2020 11:03:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2394401AbgLMLC7 (ORCPT ); Sun, 13 Dec 2020 06:02:59 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47066 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727708AbgLMLC6 (ORCPT ); Sun, 13 Dec 2020 06:02:58 -0500 Received: from mail-wr1-x444.google.com (mail-wr1-x444.google.com [IPv6:2a00:1450:4864:20::444]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 84A65C0613CF for ; Sun, 13 Dec 2020 03:02:17 -0800 (PST) Received: by mail-wr1-x444.google.com with SMTP id a12so13494684wrv.8 for ; Sun, 13 Dec 2020 03:02:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=REbzHnQpX8ZFKZ/7cj+SuCzQJd+wUlVVcehKx4VnzHw=; b=wmpcNnjg2dS8LcAP80ETORu4VS/cSH0a2uYfQFZB4DAGPSzGiwyXzxmSwmql+X5UJG wImax83S0MM3N+3OTuHqU7tCO/hmCFpUJjKwlY6gqs/nHrtCM1PFSoyocgeKFUHZU5Kk JW4WWaUjshyfXNADsEZ6O3qu5R16AqnJyVxw1XIy6R32gCg8EF11ZhngWICZBugHE0L0 xui8tThIXUKa+G9K6JOJPE+/za8lAq8NKozP30yZjtI0/hnLAvCJOrmBCi45yWhJTVYu XEoDBbHRjyfWde/GQSAg57Cc7Q5buuLrmGDIlJLYTcSFUGOnR45OvgiaBgg4iggClPE4 DwVA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=REbzHnQpX8ZFKZ/7cj+SuCzQJd+wUlVVcehKx4VnzHw=; b=iwMVe9ZNRB3X3lodOO1lQ1d4SkE9wEDZ48fzkJ523tBb5FZyByULxS4Vb51LZD8cSg qbOWvSIqwGjelQT1EyIGM2JQ1EHzWed/b4Y7hriPIYAcjiBeWR0fud8hg1qQ8043/xU6 SoHdePaeukMcJXu85rsk/jlGXKKPlqqNeyVePj0vvRcx2l9eAeCqU1IeAzO6ws8lstE9 2KNomKA0jaqrie8Rx6vWEon4NlYzT0OPjWOlC+KqZECC/EtYn3uC56Jf0reAssgurXWn 4LVCkzR+YkCWRPcuJcjmXvYBn4pYLHQHnfIvOKBZdPz0a2kFTQuzRmCZj89wE39NmORi ZW2w== X-Gm-Message-State: AOAM532ShBNc8Gfy8a3EzHVNEl8vLbqoPW12sj+v8QVHCa449nll2r9c 1fk5q1Y1AHiu22C4s0nzAhZ1KQ== X-Google-Smtp-Source: ABdhPJxcw7hP/Ce0Jao2cU3RgX+XzikJNgWR3P9pEAhLQV4Mjf1BjcxPh4lxYN/ZIwgVwJGyRHrdqg== X-Received: by 2002:a5d:4349:: with SMTP id u9mr22841145wrr.319.1607857335973; Sun, 13 Dec 2020 03:02:15 -0800 (PST) Received: from ?IPv6:2a01:e34:ed2f:f020:ed37:d2d0:f316:c7b3? ([2a01:e34:ed2f:f020:ed37:d2d0:f316:c7b3]) by smtp.googlemail.com with ESMTPSA id q17sm25043907wrr.53.2020.12.13.03.02.14 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 13 Dec 2020 03:02:15 -0800 (PST) Subject: Re: [PATCH] thermal/core: Make 'forced_passive' as obsolete candidate To: Matthew Garrett Cc: rui.zhang@intel.com, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, amitk@kernel.org, Matthew Garrett References: <20201208153046.297456-1-daniel.lezcano@linaro.org> <20201212035012.GA11926@codon.org.uk> <20015331-955b-756f-3dce-4eb78e473704@linaro.org> <20201212200806.GA19048@codon.org.uk> <6105a8e5-7590-5ba1-5f2b-aa24bf286150@linaro.org> <20201213011105.GA21385@codon.org.uk> From: Daniel Lezcano Message-ID: Date: Sun, 13 Dec 2020 12:02:13 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20201213011105.GA21385@codon.org.uk> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 13/12/2020 02:11, Matthew Garrett wrote: > On Sun, Dec 13, 2020 at 12:39:26AM +0100, Daniel Lezcano wrote: >> On 12/12/2020 21:08, Matthew Garrett wrote: >>> Anything that provides a trip point that has no active notifications and >>> doesn't provide any information that tells the kernel to poll it. >> >> I'm not able to create a setup as you describe working correctly with >> the forced passive trip point. >> >> The forced passive trip can not be detected as there is no comparison >> with the defined temperature in the thermal_zone_device_update() function. > > The logic seems to be in the step_wise thermal governor. I'm not sure > why it would be used in thermal_zone_device_update() - the entire point > is that we don't get updates from the device? The thermal_zone_device_update() loops the trip points: for (count = 0; count < tz->trips; count++) handle_thermal_trip(tz, count); As the 'forced_passive' is not in this loop (because it was moved in the step_wise governor), the temperature crossing is never detected and the 'forced_passive' logic in the governor is never called. That is something I realized when answering to your comment. >> If my analysis is correct, this 'feature' is broken since years, more >> than 8 years to be exact and nobody complained. > > I've no problem with it being removed if there are no users, but in that > case the justification should be rewritten - ACPI table updates aren't a > complete replacement for the functionality offered (and can't be used if > the lockdown LSM is being used in any case). Yes, I understand your point. Given it is not working since years, I think we can just drop the feature and change the reason of the removal in the log, instead of ACPI table updates, just say it is no longer used. Does it sound fine ? -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog