[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <50BEF830.6020206@gmail.com>
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)
Powered by blists - more mailing lists