linux-kernel - Re: regression in linux 3.7 - fan speed at 100% after suspend/resume at 100%

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]

Date:	Wed, 05 Dec 2012 08:30:56 +0100
From:	Roberto Oppedisano <roberto.oppedisano@...il.com>
To:	Zhang Rui <rui.zhang@...el.com>
CC:	LKML <linux-kernel@...r.kernel.org>,
	"Rafael J. Wysocki" <rjw@...k.pl>, eduardo.valentin@...com
Subject: Re: regression in linux 3.7 - fan speed at 100% after suspend/resume
 at 100%

Hello Rui

Il 04/12/2012 15:11, Zhang Rui ha scritto:
> Hi, Roberto,
>
> please attach the acpidump output.

attached.

> On Tue, 2012-12-04 at 13:57 +0100, Roberto Oppedisano wrote:
>> Hello,
>>       with recent kernels after a suspend/resume cycle on my laptop (HP
>> 6730b) the fans stays at full speed.
> does the fan keep running at full speed after resume?

yes, the fan never stops and runs always at full speed.


> please attach the output of "grep . /sys/class/thermal/*/*" before
> suspend and when the fan is running at full speed.
> please attach the same stuff in a good kernel.

See attached files.
3.6.9 is the good kernel where the fan behaves correctly.
3.7.0+git is the bad one.

Kind regards
R

>
>
> thanks,
> rui
>
>
>> I've bisected down to this commit:
>>
>> ce119f83257aae29b84a5bfad0669e8348437b18 is the first bad commit
>> commit ce119f83257aae29b84a5bfad0669e8348437b18
>> Author: Zhang Rui <rui.zhang@...el.com>
>> Date:   Wed Jun 27 14:13:04 2012 +0800
>>
>>       Thermal: Introduce simple arbitrator for setting device cooling state
>>
>>       This fixes the problem that a cooling device may be referenced by
>>       by multiple trip points in multiple thermal zones.
>>
>>       With this patch, we have two stages for updating a thermal zone,
>>       1. check if a thermal_instance needs to be updated or not
>>       2. update the cooling device, based on the target cooling state
>>          of all its instances.
>>
>>       Note that, currently, the cooling device is set to the deepest
>>       cooling state required.
>>
>>       Signed-off-by: Zhang Rui <rui.zhang@...el.com>
>>       Reviewed-by: Rafael J. Wysocki <rjw@...k.pl>
>>       Reviewed-by: Eduardo Valentin <eduardo.valentin@...com>
>>
>> :040000 040000 8469b057ee5b33bdb264624db35c951d0ecdb66b
>> d7288437cff3c7437e59144c46c04c78669a597b M      drivers
>> :040000 040000 397296161d55c825f07501470130928d226497cc
>> f715fd1770dd33fd333c5fa339e740537b2a916f M      include
>>
>> I tried to revert it but there are rejects so I cold not try current
>> kernel with simply this commit removed.
>>
>> As a side note when the fan is at full speed a see an anomaly on thermal
>> indicator #4
>>
>> $ grep 0 /sys/devices/virtual/thermal/thermal_zone*/temp
>> /sys/devices/virtual/thermal/thermal_zone0/temp:59000
>> /sys/devices/virtual/thermal/thermal_zone1/temp:63000
>> /sys/devices/virtual/thermal/thermal_zone2/temp:48000
>> /sys/devices/virtual/thermal/thermal_zone3/temp:27900
>> /sys/devices/virtual/thermal/thermal_zone4/temp:100000
>>
>> while the others seem to report normal value. Attached the output of
>> dmesg on this laptop.
>>
>> More info/testing on request.
>>
>> Kind regards
>> R
>
>


Download attachment "acpidump.good.bz2" of type "application/octet-stream" (116095 bytes)

View attachment "systhermal-3.6.9-aftersuspend.txt" of type "text/plain" (3962 bytes)

View attachment "systhermal-3.7.0+git-presuspend.txt" of type "text/plain" (3962 bytes)

View attachment "systhermal-3.7.0+git-aftersuspend.txt" of type "text/plain" (3967 bytes)

View attachment "systhermal-3.6.9-presuspend.txt" of type "text/plain" (3962 bytes)