linux-kernel - Re: [RESEND PATCH v5 3/5] acerhdf: Use bang-bang thermal governor

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <cone.1417677666.882029.18692.1000@mups>
Date:	Thu, 04 Dec 2014 08:21:06 +0100
From:	Peter Feuerer <peter@...e.net>
To:	Darren Hart <dvhart@...radead.org>
Cc:	LKML <linux-kernel@...r.kernel.org>,
	platform-driver-x86@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>,
	Zhang Rui <rui.zhang@...el.com>, Andreas Mohr <andi@...as.de>,
	Javi Merino <javi.merino@....com>
Subject: Re: [RESEND PATCH v5 3/5] acerhdf: Use bang-bang thermal governor

Hi again,

Peter Feuerer writes:

> Hi Darren,
> 
> thank you very much for your reply.
> 
> 
> Darren Hart writes:
> 
>> On Fri, Nov 28, 2014 at 03:20:50PM +0100, Peter Feuerer wrote:
>>> acerhdf has been doing an on-off fan control using hysteresis by
>>> post-manipulating the outcome of thermal subsystem trip point handling.
>>> This patch enables acerhdf to use the bang-bang governor, which is
>>> intended for on-off controlled fans.
>>> 
>>> Cc: platform-driver-x86@...r.kernel.org
>>> Cc: Darren Hart <dvhart@...radead.org>
>>> Cc: Andrew Morton <akpm@...ux-foundation.org>
>>> CC: Zhang Rui <rui.zhang@...el.com>
>>> Cc: Andreas Mohr <andi@...as.de>
>>> Cc: Javi Merino <javi.merino@....com>
>>> Acked-and-tested-by: Borislav Petkov <bp@...e.de>
>>> Signed-off-by: Peter Feuerer <peter@...e.net>
>>> ---
>>>  drivers/platform/x86/Kconfig   |  3 ++-
>>>  drivers/platform/x86/acerhdf.c | 36 +++++++++++++++++++++++++++++++-----
>>>  2 files changed, 33 insertions(+), 6 deletions(-)
>>> 
>>> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
>>> index a2eabe6..c173266 100644
>>> --- a/drivers/platform/x86/Kconfig
>>> +++ b/drivers/platform/x86/Kconfig
>>> @@ -38,7 +38,8 @@ config ACER_WMI
>>>  
>>>  config ACERHDF
>>>  	tristate "Acer Aspire One temperature and fan driver"
>>> -	depends on THERMAL && ACPI
>>> +	select THERMAL_GOV_BANG_BANG
>> 
>> So we use select sparingly as it does implicit things.
>> 
>> I checked the THERMAL_GOV_BANG_BANG Kconfig entry, and the help says acerhdf
>> already depends on it (which it doesn't appear to). Any particular reason to add
>> select here instead of adding it as a depends.
>> 
>> Why did you drop THERMAL?
> 
> I had it like this in my first version of patches:
> + depends on THERMAL && ACPI && THERMAL_GOV_BANG_BANG
> 
> But after some discussion with lkml community we ended up with the select 
> line and dropped THERMAL dependency, as it is implied by 
> THEMAL_GOV_BANG_BANG.  I'm not so experienced with Kconfig, so I must rely 
> on the statements of the community in this case.

Just found the link about this discussion I had with Rui and Boris:
http://linux-kernel.2935.n7.nabble.com/PATCH-0-4-acerhdf-thermal-adding-new-models-and-appropriate-governor-tp848572p908256.html

[...]

-- 
kind regards,
--peter;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/