linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
@ 2010-11-04 16:06 Maciej Rutecki
  2010-11-05  0:27 ` Zhang Rui
  0 siblings, 1 reply; 11+ messages in thread
From: Maciej Rutecki @ 2010-11-04 16:06 UTC (permalink / raw)
  To: linux-kernel, linux-acpi, rui.zhang, len.brown, Rafael J. Wysocki

1. 
Last known good kernel: 2.6.36
Bad kernel: 2.6.37-rc1
===============================

2. Problem:
When I resume computer from s2ram/s2disk and current CPU temp is below lowest 
trip point, and -- from unknown reason -- fan start in maximum speed it 
doesn't turn off (it should).

It doesn't happen always but only when fan starts work and any of trip points 
aren't exceed. It seems that system only check temperature when exceed trip 
point.
===============================

3. /sys/cass/thermal/ info:

I have:
maciek@gumis:/sys/class/thermal$ ls
cooling_device0  cooling_device2  cooling_device4  cooling_device6  
thermal_zone1  thermal_zone3
cooling_device1  cooling_device3  cooling_device5  thermal_zone0    
thermal_zone2  thermal_zone4

maciek@gumis:/sys/class/thermal$ cat cooling_device*/type 
LCD
Processor
Processor
Fan
Fan
Fan
Fan

===============================

Colling device from 3 to 6 are inform what is current fan speed, for example:
cooling_device3 = 0
cooling_device4 = 0
cooling_device5 = 0
cooling_device6 = 0 
Means that fan is off.

cooling_device3 = 0
cooling_device4 = 0
cooling_device5 = 0
cooling_device6 = 1
Means fan works on 40% of full speed (1st trip point is exceed).

cooling_device3 = 1
cooling_device4 = 1
cooling_device5 = 1
cooling_device6 = 1
Means fan works on full speed (all (4) trip points is exceed).
===============================

5. State when fan works on full speed after resume:
maciek@gumis:/sys/class/thermal$ cat cooling_device*/cur_state
0
0
0
1
1
1
1
(Runs on full speed)

maciek@gumis:/sys/class/thermal$ cat thermal_zone0/trip_point_*
256000
critical
78000
active
70000
active
60000
active
50000
active

(It's OK for current CPU temp)

"sensors" command shows:
maciek@gumis:/sys/class/thermal$ sensors
acpitz-virtual-0
Adapter: Virtual device
temp1:       +37.0°C  (crit = +256.0°C)                  
temp2:       +42.0°C  (crit = +90.0°C)                  
temp3:       +33.0°C  (crit = +105.0°C)                  
temp4:       +20.0°C  (crit = +90.0°C)                  
temp5:      +100.0°C  (crit = +110.0°C)

"temp1" in this model of HP notebook is only for debug purposes (I think).
"temp2" shows CPU temp (fan speed depends on it)
"temp5" shows fan speed in % of maximum speed.
===============================

6. I run this command, to "hot" the processor:
"while true; do echo -n; done"
and cancel it, when I see, that first trip point was exceed, and fan starts 
working on lower speed.
===============================

7. When it done I saw:
maciek@gumis:/sys/class/thermal$ cat cooling_device*/cur_state
0
0
0
0
0
0
1

Fan works on 40% of full speed. It's OK.

maciek@gumis:/sys/class/thermal$ cat thermal_zone0/trip_point_*
256000
critical
78000
active
70000
active
60000
active
45000
active

Last trip point was changed from 50000 to 45000 because trip point was exceed 
(it's OK, fan stops working below 45*C or starts working on 60% speed, above 
60*C).

"sensors" show:
maciek@gumis:/sys/class/thermal$ sensors
acpitz-virtual-0
Adapter: Virtual device
temp1:       +50.0°C  (crit = +256.0°C)                  
temp2:       +48.0°C  (crit = +90.0°C)                  
temp3:       +34.0°C  (crit = +105.0°C)                  
temp4:       +20.0°C  (crit = +90.0°C)                  
temp5:       +40.0°C  (crit = +110.0°C)
===============================

Dmesg:
http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/dmesg-2.6.37-rc1.txt

acpidump:
http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/acpidump.dat

Config:
http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/config-2.6.37-rc1

Regards
-- 
Maciej Rutecki
http://www.maciek.unixy.pl

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-04 16:06 [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk Maciej Rutecki
@ 2010-11-05  0:27 ` Zhang Rui
  2010-11-05 19:49   ` Maciej Rutecki
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang Rui @ 2010-11-05  0:27 UTC (permalink / raw)
  To: maciej.rutecki; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

On Fri, 2010-11-05 at 00:06 +0800, Maciej Rutecki wrote:
> 1. 
> Last known good kernel: 2.6.36
> Bad kernel: 2.6.37-rc1
> ===============================
> 
> 2. Problem:
> When I resume computer from s2ram/s2disk and current CPU temp is below lowest 
> trip point, and -- from unknown reason -- fan start in maximum speed it 
> doesn't turn off (it should).
> 
could you please attach the output of "grep . /sys/class/thermal/*/*"
when this happens?

thanks,
rui

> It doesn't happen always but only when fan starts work and any of trip points 
> aren't exceed. It seems that system only check temperature when exceed trip 
> point.
> ===============================
> 
> 3. /sys/cass/thermal/ info:
> 
> I have:
> maciek@gumis:/sys/class/thermal$ ls
> cooling_device0  cooling_device2  cooling_device4  cooling_device6  
> thermal_zone1  thermal_zone3
> cooling_device1  cooling_device3  cooling_device5  thermal_zone0    
> thermal_zone2  thermal_zone4
> 
> maciek@gumis:/sys/class/thermal$ cat cooling_device*/type 
> LCD
> Processor
> Processor
> Fan
> Fan
> Fan
> Fan
> 
> ===============================
> 
> Colling device from 3 to 6 are inform what is current fan speed, for example:
> cooling_device3 = 0
> cooling_device4 = 0
> cooling_device5 = 0
> cooling_device6 = 0 
> Means that fan is off.
> 
> cooling_device3 = 0
> cooling_device4 = 0
> cooling_device5 = 0
> cooling_device6 = 1
> Means fan works on 40% of full speed (1st trip point is exceed).
> 
> cooling_device3 = 1
> cooling_device4 = 1
> cooling_device5 = 1
> cooling_device6 = 1
> Means fan works on full speed (all (4) trip points is exceed).
> ===============================
> 
> 5. State when fan works on full speed after resume:
> maciek@gumis:/sys/class/thermal$ cat cooling_device*/cur_state
> 0
> 0
> 0
> 1
> 1
> 1
> 1
> (Runs on full speed)
> 
> maciek@gumis:/sys/class/thermal$ cat thermal_zone0/trip_point_*
> 256000
> critical
> 78000
> active
> 70000
> active
> 60000
> active
> 50000
> active
> 
> (It's OK for current CPU temp)
> 
> "sensors" command shows:
> maciek@gumis:/sys/class/thermal$ sensors
> acpitz-virtual-0
> Adapter: Virtual device
> temp1:       +37.0°C  (crit = +256.0°C)                  
> temp2:       +42.0°C  (crit = +90.0°C)                  
> temp3:       +33.0°C  (crit = +105.0°C)                  
> temp4:       +20.0°C  (crit = +90.0°C)                  
> temp5:      +100.0°C  (crit = +110.0°C)
> 
> "temp1" in this model of HP notebook is only for debug purposes (I think).
> "temp2" shows CPU temp (fan speed depends on it)
> "temp5" shows fan speed in % of maximum speed.
> ===============================
> 
> 6. I run this command, to "hot" the processor:
> "while true; do echo -n; done"
> and cancel it, when I see, that first trip point was exceed, and fan starts 
> working on lower speed.
> ===============================
> 
> 7. When it done I saw:
> maciek@gumis:/sys/class/thermal$ cat cooling_device*/cur_state
> 0
> 0
> 0
> 0
> 0
> 0
> 1
> 
> Fan works on 40% of full speed. It's OK.
> 
> maciek@gumis:/sys/class/thermal$ cat thermal_zone0/trip_point_*
> 256000
> critical
> 78000
> active
> 70000
> active
> 60000
> active
> 45000
> active
> 
> Last trip point was changed from 50000 to 45000 because trip point was exceed 
> (it's OK, fan stops working below 45*C or starts working on 60% speed, above 
> 60*C).
> 
> "sensors" show:
> maciek@gumis:/sys/class/thermal$ sensors
> acpitz-virtual-0
> Adapter: Virtual device
> temp1:       +50.0°C  (crit = +256.0°C)                  
> temp2:       +48.0°C  (crit = +90.0°C)                  
> temp3:       +34.0°C  (crit = +105.0°C)                  
> temp4:       +20.0°C  (crit = +90.0°C)                  
> temp5:       +40.0°C  (crit = +110.0°C)
> ===============================
> 
> Dmesg:
> http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/dmesg-2.6.37-rc1.txt
> 
> acpidump:
> http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/acpidump.dat
> 
> Config:
> http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/config-2.6.37-rc1
> 
> Regards



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-05  0:27 ` Zhang Rui
@ 2010-11-05 19:49   ` Maciej Rutecki
  2010-11-08  6:58     ` Zhang Rui
  0 siblings, 1 reply; 11+ messages in thread
From: Maciej Rutecki @ 2010-11-05 19:49 UTC (permalink / raw)
  To: Zhang Rui; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

On piątek, 5 listopada 2010 o 01:27:34 Zhang Rui wrote:
> On Fri, 2010-11-05 at 00:06 +0800, Maciej Rutecki wrote:
> > 1.
> > Last known good kernel: 2.6.36
> > Bad kernel: 2.6.37-rc1
> > ===============================
> > 
> > 2. Problem:
> > When I resume computer from s2ram/s2disk and current CPU temp is below
> > lowest trip point, and -- from unknown reason -- fan start in maximum
> > speed it doesn't turn off (it should).
> 
> could you please attach the output of "grep . /sys/class/thermal/*/*"
> when this happens?
>

Thanks for the answer.

maciek@gumis:~$ grep . /sys/class/thermal/*/*
/sys/class/thermal/cooling_device0/cur_state:0
/sys/class/thermal/cooling_device0/max_state:10
/sys/class/thermal/cooling_device0/type:LCD
/sys/class/thermal/cooling_device1/cur_state:0
/sys/class/thermal/cooling_device1/max_state:10
/sys/class/thermal/cooling_device1/type:Processor
/sys/class/thermal/cooling_device2/cur_state:0
/sys/class/thermal/cooling_device2/max_state:10
/sys/class/thermal/cooling_device2/type:Processor
/sys/class/thermal/cooling_device3/cur_state:1
/sys/class/thermal/cooling_device3/max_state:1
/sys/class/thermal/cooling_device3/type:Fan
/sys/class/thermal/cooling_device4/cur_state:1
/sys/class/thermal/cooling_device4/max_state:1
/sys/class/thermal/cooling_device4/type:Fan
/sys/class/thermal/cooling_device5/cur_state:1
/sys/class/thermal/cooling_device5/max_state:1
/sys/class/thermal/cooling_device5/type:Fan
/sys/class/thermal/cooling_device6/cur_state:1
/sys/class/thermal/cooling_device6/max_state:1
/sys/class/thermal/cooling_device6/type:Fan
/sys/class/thermal/thermal_zone0/cdev0_trip_point:1
/sys/class/thermal/thermal_zone0/cdev1_trip_point:2
/sys/class/thermal/thermal_zone0/cdev2_trip_point:3
/sys/class/thermal/thermal_zone0/cdev3_trip_point:4
/sys/class/thermal/thermal_zone0/mode:enabled
/sys/class/thermal/thermal_zone0/passive:0
/sys/class/thermal/thermal_zone0/temp:27000
/sys/class/thermal/thermal_zone0/trip_point_0_temp:256000
/sys/class/thermal/thermal_zone0/trip_point_0_type:critical
/sys/class/thermal/thermal_zone0/trip_point_1_temp:78000
/sys/class/thermal/thermal_zone0/trip_point_1_type:active
/sys/class/thermal/thermal_zone0/trip_point_2_temp:70000
/sys/class/thermal/thermal_zone0/trip_point_2_type:active
/sys/class/thermal/thermal_zone0/trip_point_3_temp:60000
/sys/class/thermal/thermal_zone0/trip_point_3_type:active
/sys/class/thermal/thermal_zone0/trip_point_4_temp:50000
/sys/class/thermal/thermal_zone0/trip_point_4_type:active
/sys/class/thermal/thermal_zone0/type:acpitz
/sys/class/thermal/thermal_zone1/cdev0_trip_point:1
/sys/class/thermal/thermal_zone1/cdev1_trip_point:1
/sys/class/thermal/thermal_zone1/mode:enabled
/sys/class/thermal/thermal_zone1/temp:29000
/sys/class/thermal/thermal_zone1/trip_point_0_temp:90000
/sys/class/thermal/thermal_zone1/trip_point_0_type:critical
/sys/class/thermal/thermal_zone1/trip_point_1_temp:87000
/sys/class/thermal/thermal_zone1/trip_point_1_type:passive
/sys/class/thermal/thermal_zone1/type:acpitz
/sys/class/thermal/thermal_zone2/mode:enabled
/sys/class/thermal/thermal_zone2/passive:0
/sys/class/thermal/thermal_zone2/temp:23000
/sys/class/thermal/thermal_zone2/trip_point_0_temp:105000
/sys/class/thermal/thermal_zone2/trip_point_0_type:critical
/sys/class/thermal/thermal_zone2/type:acpitz
/sys/class/thermal/thermal_zone3/cdev0_trip_point:1
/sys/class/thermal/thermal_zone3/cdev1_trip_point:1
/sys/class/thermal/thermal_zone3/mode:enabled
/sys/class/thermal/thermal_zone3/temp:20000
/sys/class/thermal/thermal_zone3/trip_point_0_temp:90000
/sys/class/thermal/thermal_zone3/trip_point_0_type:critical
/sys/class/thermal/thermal_zone3/trip_point_1_temp:60000
/sys/class/thermal/thermal_zone3/trip_point_1_type:passive
/sys/class/thermal/thermal_zone3/type:acpitz
/sys/class/thermal/thermal_zone4/mode:enabled
/sys/class/thermal/thermal_zone4/passive:0
/sys/class/thermal/thermal_zone4/temp:100000
/sys/class/thermal/thermal_zone4/trip_point_0_temp:110000
/sys/class/thermal/thermal_zone4/trip_point_0_type:critical
/sys/class/thermal/thermal_zone4/type:acpitz

Regards
-- 
Maciej Rutecki
http://www.maciek.unixy.pl

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-05 19:49   ` Maciej Rutecki
@ 2010-11-08  6:58     ` Zhang Rui
  2010-11-08 17:35       ` Maciej Rutecki
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang Rui @ 2010-11-08  6:58 UTC (permalink / raw)
  To: maciej.rutecki; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

IMO, the only one that may cause this regression is commit
c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
some app may enable polling via procfs and update the thermal zone
status.

could you please attach the output of "grep . /proc/acpi/thermal/*/*" in
2.6.36 kernel?

thanks,
rui

On Sat, 2010-11-06 at 03:49 +0800, Maciej Rutecki wrote:
> On piątek, 5 listopada 2010 o 01:27:34 Zhang Rui wrote:
> > On Fri, 2010-11-05 at 00:06 +0800, Maciej Rutecki wrote:
> > > 1.
> > > Last known good kernel: 2.6.36
> > > Bad kernel: 2.6.37-rc1
> > > ===============================
> > > 
> > > 2. Problem:
> > > When I resume computer from s2ram/s2disk and current CPU temp is below
> > > lowest trip point, and -- from unknown reason -- fan start in maximum
> > > speed it doesn't turn off (it should).
> > 
> > could you please attach the output of "grep . /sys/class/thermal/*/*"
> > when this happens?
> >
> 
> Thanks for the answer.
> 
> maciek@gumis:~$ grep . /sys/class/thermal/*/*
> /sys/class/thermal/cooling_device0/cur_state:0
> /sys/class/thermal/cooling_device0/max_state:10
> /sys/class/thermal/cooling_device0/type:LCD
> /sys/class/thermal/cooling_device1/cur_state:0
> /sys/class/thermal/cooling_device1/max_state:10
> /sys/class/thermal/cooling_device1/type:Processor
> /sys/class/thermal/cooling_device2/cur_state:0
> /sys/class/thermal/cooling_device2/max_state:10
> /sys/class/thermal/cooling_device2/type:Processor
> /sys/class/thermal/cooling_device3/cur_state:1
> /sys/class/thermal/cooling_device3/max_state:1
> /sys/class/thermal/cooling_device3/type:Fan
> /sys/class/thermal/cooling_device4/cur_state:1
> /sys/class/thermal/cooling_device4/max_state:1
> /sys/class/thermal/cooling_device4/type:Fan
> /sys/class/thermal/cooling_device5/cur_state:1
> /sys/class/thermal/cooling_device5/max_state:1
> /sys/class/thermal/cooling_device5/type:Fan
> /sys/class/thermal/cooling_device6/cur_state:1
> /sys/class/thermal/cooling_device6/max_state:1
> /sys/class/thermal/cooling_device6/type:Fan
> /sys/class/thermal/thermal_zone0/cdev0_trip_point:1
> /sys/class/thermal/thermal_zone0/cdev1_trip_point:2
> /sys/class/thermal/thermal_zone0/cdev2_trip_point:3
> /sys/class/thermal/thermal_zone0/cdev3_trip_point:4
> /sys/class/thermal/thermal_zone0/mode:enabled
> /sys/class/thermal/thermal_zone0/passive:0
> /sys/class/thermal/thermal_zone0/temp:27000
> /sys/class/thermal/thermal_zone0/trip_point_0_temp:256000
> /sys/class/thermal/thermal_zone0/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone0/trip_point_1_temp:78000
> /sys/class/thermal/thermal_zone0/trip_point_1_type:active
> /sys/class/thermal/thermal_zone0/trip_point_2_temp:70000
> /sys/class/thermal/thermal_zone0/trip_point_2_type:active
> /sys/class/thermal/thermal_zone0/trip_point_3_temp:60000
> /sys/class/thermal/thermal_zone0/trip_point_3_type:active
> /sys/class/thermal/thermal_zone0/trip_point_4_temp:50000
> /sys/class/thermal/thermal_zone0/trip_point_4_type:active
> /sys/class/thermal/thermal_zone0/type:acpitz
> /sys/class/thermal/thermal_zone1/cdev0_trip_point:1
> /sys/class/thermal/thermal_zone1/cdev1_trip_point:1
> /sys/class/thermal/thermal_zone1/mode:enabled
> /sys/class/thermal/thermal_zone1/temp:29000
> /sys/class/thermal/thermal_zone1/trip_point_0_temp:90000
> /sys/class/thermal/thermal_zone1/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone1/trip_point_1_temp:87000
> /sys/class/thermal/thermal_zone1/trip_point_1_type:passive
> /sys/class/thermal/thermal_zone1/type:acpitz
> /sys/class/thermal/thermal_zone2/mode:enabled
> /sys/class/thermal/thermal_zone2/passive:0
> /sys/class/thermal/thermal_zone2/temp:23000
> /sys/class/thermal/thermal_zone2/trip_point_0_temp:105000
> /sys/class/thermal/thermal_zone2/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone2/type:acpitz
> /sys/class/thermal/thermal_zone3/cdev0_trip_point:1
> /sys/class/thermal/thermal_zone3/cdev1_trip_point:1
> /sys/class/thermal/thermal_zone3/mode:enabled
> /sys/class/thermal/thermal_zone3/temp:20000
> /sys/class/thermal/thermal_zone3/trip_point_0_temp:90000
> /sys/class/thermal/thermal_zone3/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone3/trip_point_1_temp:60000
> /sys/class/thermal/thermal_zone3/trip_point_1_type:passive
> /sys/class/thermal/thermal_zone3/type:acpitz
> /sys/class/thermal/thermal_zone4/mode:enabled
> /sys/class/thermal/thermal_zone4/passive:0
> /sys/class/thermal/thermal_zone4/temp:100000
> /sys/class/thermal/thermal_zone4/trip_point_0_temp:110000
> /sys/class/thermal/thermal_zone4/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone4/type:acpitz
> 
> Regards
> -- 
> Maciej Rutecki
> http://www.maciek.unixy.pl



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-08  6:58     ` Zhang Rui
@ 2010-11-08 17:35       ` Maciej Rutecki
  2010-11-15  3:16         ` Zhang Rui
  0 siblings, 1 reply; 11+ messages in thread
From: Maciej Rutecki @ 2010-11-08 17:35 UTC (permalink / raw)
  To: Zhang Rui; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

On poniedziałek, 8 listopada 2010 o 07:58:52 Zhang Rui wrote:
> IMO, the only one that may cause this regression is commit
> c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
> some app may enable polling via procfs and update the thermal zone
> status.

I revert this commit from 2.6.37-rc1. Still the same:
maciek@gumis:~$ grep . /sys/class/thermal/*/*
/sys/class/thermal/cooling_device0/cur_state:0
/sys/class/thermal/cooling_device0/max_state:10
/sys/class/thermal/cooling_device0/type:LCD
/sys/class/thermal/cooling_device1/cur_state:0
/sys/class/thermal/cooling_device1/max_state:10
/sys/class/thermal/cooling_device1/type:Processor
/sys/class/thermal/cooling_device2/cur_state:0
/sys/class/thermal/cooling_device2/max_state:10
/sys/class/thermal/cooling_device2/type:Processor
/sys/class/thermal/cooling_device3/cur_state:1
/sys/class/thermal/cooling_device3/max_state:1
/sys/class/thermal/cooling_device3/type:Fan
/sys/class/thermal/cooling_device4/cur_state:1
/sys/class/thermal/cooling_device4/max_state:1
/sys/class/thermal/cooling_device4/type:Fan
/sys/class/thermal/cooling_device5/cur_state:1
/sys/class/thermal/cooling_device5/max_state:1
/sys/class/thermal/cooling_device5/type:Fan
/sys/class/thermal/cooling_device6/cur_state:1
/sys/class/thermal/cooling_device6/max_state:1
/sys/class/thermal/cooling_device6/type:Fan
/sys/class/thermal/thermal_zone0/cdev0_trip_point:1
/sys/class/thermal/thermal_zone0/cdev1_trip_point:2
/sys/class/thermal/thermal_zone0/cdev2_trip_point:3
/sys/class/thermal/thermal_zone0/cdev3_trip_point:4
/sys/class/thermal/thermal_zone0/mode:enabled
/sys/class/thermal/thermal_zone0/passive:0
/sys/class/thermal/thermal_zone0/temp:37000
/sys/class/thermal/thermal_zone0/trip_point_0_temp:256000
/sys/class/thermal/thermal_zone0/trip_point_0_type:critical
/sys/class/thermal/thermal_zone0/trip_point_1_temp:78000
/sys/class/thermal/thermal_zone0/trip_point_1_type:active
/sys/class/thermal/thermal_zone0/trip_point_2_temp:70000
/sys/class/thermal/thermal_zone0/trip_point_2_type:active
/sys/class/thermal/thermal_zone0/trip_point_3_temp:60000
/sys/class/thermal/thermal_zone0/trip_point_3_type:active
/sys/class/thermal/thermal_zone0/trip_point_4_temp:50000
/sys/class/thermal/thermal_zone0/trip_point_4_type:active
/sys/class/thermal/thermal_zone0/type:acpitz
/sys/class/thermal/thermal_zone1/cdev0_trip_point:1
/sys/class/thermal/thermal_zone1/cdev1_trip_point:1
/sys/class/thermal/thermal_zone1/mode:enabled
/sys/class/thermal/thermal_zone1/temp:36000
/sys/class/thermal/thermal_zone1/trip_point_0_temp:90000
/sys/class/thermal/thermal_zone1/trip_point_0_type:critical
/sys/class/thermal/thermal_zone1/trip_point_1_temp:87000
/sys/class/thermal/thermal_zone1/trip_point_1_type:passive
/sys/class/thermal/thermal_zone1/type:acpitz
/sys/class/thermal/thermal_zone2/mode:enabled
/sys/class/thermal/thermal_zone2/passive:0
/sys/class/thermal/thermal_zone2/temp:33000
/sys/class/thermal/thermal_zone2/trip_point_0_temp:105000
/sys/class/thermal/thermal_zone2/trip_point_0_type:critical
/sys/class/thermal/thermal_zone2/type:acpitz
/sys/class/thermal/thermal_zone3/cdev0_trip_point:1
/sys/class/thermal/thermal_zone3/cdev1_trip_point:1
/sys/class/thermal/thermal_zone3/mode:enabled
/sys/class/thermal/thermal_zone3/temp:30800
/sys/class/thermal/thermal_zone3/trip_point_0_temp:90000
/sys/class/thermal/thermal_zone3/trip_point_0_type:critical
/sys/class/thermal/thermal_zone3/trip_point_1_temp:60000
/sys/class/thermal/thermal_zone3/trip_point_1_type:passive
/sys/class/thermal/thermal_zone3/type:acpitz
/sys/class/thermal/thermal_zone4/mode:enabled
/sys/class/thermal/thermal_zone4/passive:0
/sys/class/thermal/thermal_zone4/temp:100000
/sys/class/thermal/thermal_zone4/trip_point_0_temp:110000
/sys/class/thermal/thermal_zone4/trip_point_0_type:critical
/sys/class/thermal/thermal_zone4/type:acpitz

maciek@gumis:~$ grep . /proc/acpi/thermal_zone/*/*
/proc/acpi/thermal_zone/TZ0/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ0/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ0/state:state:                   active[0]
/proc/acpi/thermal_zone/TZ0/temperature:temperature:             37 C
/proc/acpi/thermal_zone/TZ0/trip_points:critical (S5):           256 C
/proc/acpi/thermal_zone/TZ0/trip_points:passive (forced):<not set>
/proc/acpi/thermal_zone/TZ0/trip_points:active[0]:               78 C: 
devices=C325 
/proc/acpi/thermal_zone/TZ0/trip_points:active[1]:               70 C: 
devices=C326 
/proc/acpi/thermal_zone/TZ0/trip_points:active[2]:               60 C: 
devices=C327 
/proc/acpi/thermal_zone/TZ0/trip_points:active[3]:               50 C: 
devices=C328 
/proc/acpi/thermal_zone/TZ1/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ1/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ1/state:state:                   ok
/proc/acpi/thermal_zone/TZ1/temperature:temperature:             35 C
/proc/acpi/thermal_zone/TZ1/trip_points:critical (S5):           90 C
/proc/acpi/thermal_zone/TZ1/trip_points:passive:                 87 C: tc1=1 
tc2=2 tsp=300 devices=CPU0 CPU1 
/proc/acpi/thermal_zone/TZ2/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ2/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ2/state:state:                   ok
/proc/acpi/thermal_zone/TZ2/temperature:temperature:             33 C
/proc/acpi/thermal_zone/TZ2/trip_points:critical (S5):           105 C
/proc/acpi/thermal_zone/TZ2/trip_points:passive (forced):<not set>
/proc/acpi/thermal_zone/TZ3/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ3/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ3/state:state:                   ok
/proc/acpi/thermal_zone/TZ3/temperature:temperature:             31 C
/proc/acpi/thermal_zone/TZ3/trip_points:critical (S5):           90 C
/proc/acpi/thermal_zone/TZ3/trip_points:passive:                 60 C: tc1=1 
tc2=2 tsp=300 devices=CPU0 CPU1 
/proc/acpi/thermal_zone/TZ4/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ4/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ4/state:state:                   ok
/proc/acpi/thermal_zone/TZ4/temperature:temperature:             100 C
/proc/acpi/thermal_zone/TZ4/trip_points:critical (S5):           110 C
/proc/acpi/thermal_zone/TZ4/trip_points:passive (forced):<not set>

> 
> could you please attach the output of "grep . /proc/acpi/thermal/*/*" in
> 2.6.36 kernel?
> 
2.6.36 after suspend to ram (CPU temp was above 1st trip point):

maciek@gumis:~$ grep . /proc/acpi/fan/*/*
/proc/acpi/fan/C325/state:status:                  off
/proc/acpi/fan/C326/state:status:                  off
/proc/acpi/fan/C327/state:status:                  off
/proc/acpi/fan/C328/state:status:                  on

maciek@gumis:~$ grep . /proc/acpi/thermal_zone/*/*
/proc/acpi/thermal_zone/TZ0/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ0/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ0/state:state:                   active[0]
/proc/acpi/thermal_zone/TZ0/temperature:temperature:             50 C
/proc/acpi/thermal_zone/TZ0/trip_points:critical (S5):           256 C
/proc/acpi/thermal_zone/TZ0/trip_points:passive (forced):<not set>
/proc/acpi/thermal_zone/TZ0/trip_points:active[0]:               78 C: 
devices=C325 
/proc/acpi/thermal_zone/TZ0/trip_points:active[1]:               70 C: 
devices=C326 
/proc/acpi/thermal_zone/TZ0/trip_points:active[2]:               60 C: 
devices=C327 
/proc/acpi/thermal_zone/TZ0/trip_points:active[3]:               45 C: 
devices=C328 
/proc/acpi/thermal_zone/TZ1/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ1/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ1/state:state:                   ok
/proc/acpi/thermal_zone/TZ1/temperature:temperature:             47 C
/proc/acpi/thermal_zone/TZ1/trip_points:critical (S5):           90 C
/proc/acpi/thermal_zone/TZ1/trip_points:passive:                 87 C: tc1=1 
tc2=2 tsp=300 devices=CPU0 CPU1 
/proc/acpi/thermal_zone/TZ2/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ2/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ2/state:state:                   ok
/proc/acpi/thermal_zone/TZ2/temperature:temperature:             40 C
/proc/acpi/thermal_zone/TZ2/trip_points:critical (S5):           105 C
/proc/acpi/thermal_zone/TZ2/trip_points:passive (forced):<not set>
/proc/acpi/thermal_zone/TZ3/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ3/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ3/state:state:                   ok
/proc/acpi/thermal_zone/TZ3/temperature:temperature:             28 C
/proc/acpi/thermal_zone/TZ3/trip_points:critical (S5):           90 C
/proc/acpi/thermal_zone/TZ3/trip_points:passive:                 60 C: tc1=1 
tc2=2 tsp=300 devices=CPU0 CPU1 
/proc/acpi/thermal_zone/TZ4/cooling_mode:<setting not supported>
/proc/acpi/thermal_zone/TZ4/polling_frequency:<polling disabled>
/proc/acpi/thermal_zone/TZ4/state:state:                   ok
/proc/acpi/thermal_zone/TZ4/temperature:temperature:             40 C
/proc/acpi/thermal_zone/TZ4/trip_points:critical (S5):           110 C
/proc/acpi/thermal_zone/TZ4/trip_points:passive (forced):<not set>

Regards
-- 
Maciej Rutecki
http://www.maciek.unixy.pl

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-08 17:35       ` Maciej Rutecki
@ 2010-11-15  3:16         ` Zhang Rui
       [not found]           ` <201011152139.46261.maciej.rutecki@gmail.com>
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang Rui @ 2010-11-15  3:16 UTC (permalink / raw)
  To: maciej.rutecki; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

[-- Attachment #1: Type: text/plain, Size: 9805 bytes --]

On Tue, 2010-11-09 at 01:35 +0800, Maciej Rutecki wrote:
> On poniedziałek, 8 listopada 2010 o 07:58:52 Zhang Rui wrote:
> > IMO, the only one that may cause this regression is commit
> > c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
> > some app may enable polling via procfs and update the thermal zone
> > status.
> 
> I revert this commit from 2.6.37-rc1. Still the same:

could you please apply the patch attached, on top of 2.6.37-rc1, and
then attach the dmesg output if the fan is noise after suspend/resume.

thanks,
rui
> maciek@gumis:~$ grep . /sys/class/thermal/*/*
> /sys/class/thermal/cooling_device0/cur_state:0
> /sys/class/thermal/cooling_device0/max_state:10
> /sys/class/thermal/cooling_device0/type:LCD
> /sys/class/thermal/cooling_device1/cur_state:0
> /sys/class/thermal/cooling_device1/max_state:10
> /sys/class/thermal/cooling_device1/type:Processor
> /sys/class/thermal/cooling_device2/cur_state:0
> /sys/class/thermal/cooling_device2/max_state:10
> /sys/class/thermal/cooling_device2/type:Processor
> /sys/class/thermal/cooling_device3/cur_state:1
> /sys/class/thermal/cooling_device3/max_state:1
> /sys/class/thermal/cooling_device3/type:Fan
> /sys/class/thermal/cooling_device4/cur_state:1
> /sys/class/thermal/cooling_device4/max_state:1
> /sys/class/thermal/cooling_device4/type:Fan
> /sys/class/thermal/cooling_device5/cur_state:1
> /sys/class/thermal/cooling_device5/max_state:1
> /sys/class/thermal/cooling_device5/type:Fan
> /sys/class/thermal/cooling_device6/cur_state:1
> /sys/class/thermal/cooling_device6/max_state:1
> /sys/class/thermal/cooling_device6/type:Fan
> /sys/class/thermal/thermal_zone0/cdev0_trip_point:1
> /sys/class/thermal/thermal_zone0/cdev1_trip_point:2
> /sys/class/thermal/thermal_zone0/cdev2_trip_point:3
> /sys/class/thermal/thermal_zone0/cdev3_trip_point:4
> /sys/class/thermal/thermal_zone0/mode:enabled
> /sys/class/thermal/thermal_zone0/passive:0
> /sys/class/thermal/thermal_zone0/temp:37000
> /sys/class/thermal/thermal_zone0/trip_point_0_temp:256000
> /sys/class/thermal/thermal_zone0/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone0/trip_point_1_temp:78000
> /sys/class/thermal/thermal_zone0/trip_point_1_type:active
> /sys/class/thermal/thermal_zone0/trip_point_2_temp:70000
> /sys/class/thermal/thermal_zone0/trip_point_2_type:active
> /sys/class/thermal/thermal_zone0/trip_point_3_temp:60000
> /sys/class/thermal/thermal_zone0/trip_point_3_type:active
> /sys/class/thermal/thermal_zone0/trip_point_4_temp:50000
> /sys/class/thermal/thermal_zone0/trip_point_4_type:active
> /sys/class/thermal/thermal_zone0/type:acpitz
> /sys/class/thermal/thermal_zone1/cdev0_trip_point:1
> /sys/class/thermal/thermal_zone1/cdev1_trip_point:1
> /sys/class/thermal/thermal_zone1/mode:enabled
> /sys/class/thermal/thermal_zone1/temp:36000
> /sys/class/thermal/thermal_zone1/trip_point_0_temp:90000
> /sys/class/thermal/thermal_zone1/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone1/trip_point_1_temp:87000
> /sys/class/thermal/thermal_zone1/trip_point_1_type:passive
> /sys/class/thermal/thermal_zone1/type:acpitz
> /sys/class/thermal/thermal_zone2/mode:enabled
> /sys/class/thermal/thermal_zone2/passive:0
> /sys/class/thermal/thermal_zone2/temp:33000
> /sys/class/thermal/thermal_zone2/trip_point_0_temp:105000
> /sys/class/thermal/thermal_zone2/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone2/type:acpitz
> /sys/class/thermal/thermal_zone3/cdev0_trip_point:1
> /sys/class/thermal/thermal_zone3/cdev1_trip_point:1
> /sys/class/thermal/thermal_zone3/mode:enabled
> /sys/class/thermal/thermal_zone3/temp:30800
> /sys/class/thermal/thermal_zone3/trip_point_0_temp:90000
> /sys/class/thermal/thermal_zone3/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone3/trip_point_1_temp:60000
> /sys/class/thermal/thermal_zone3/trip_point_1_type:passive
> /sys/class/thermal/thermal_zone3/type:acpitz
> /sys/class/thermal/thermal_zone4/mode:enabled
> /sys/class/thermal/thermal_zone4/passive:0
> /sys/class/thermal/thermal_zone4/temp:100000
> /sys/class/thermal/thermal_zone4/trip_point_0_temp:110000
> /sys/class/thermal/thermal_zone4/trip_point_0_type:critical
> /sys/class/thermal/thermal_zone4/type:acpitz
> 
> maciek@gumis:~$ grep . /proc/acpi/thermal_zone/*/*
> /proc/acpi/thermal_zone/TZ0/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ0/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ0/state:state:                   active[0]
> /proc/acpi/thermal_zone/TZ0/temperature:temperature:             37 C
> /proc/acpi/thermal_zone/TZ0/trip_points:critical (S5):           256 C
> /proc/acpi/thermal_zone/TZ0/trip_points:passive (forced):<not set>
> /proc/acpi/thermal_zone/TZ0/trip_points:active[0]:               78 C: 
> devices=C325 
> /proc/acpi/thermal_zone/TZ0/trip_points:active[1]:               70 C: 
> devices=C326 
> /proc/acpi/thermal_zone/TZ0/trip_points:active[2]:               60 C: 
> devices=C327 
> /proc/acpi/thermal_zone/TZ0/trip_points:active[3]:               50 C: 
> devices=C328 
> /proc/acpi/thermal_zone/TZ1/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ1/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ1/state:state:                   ok
> /proc/acpi/thermal_zone/TZ1/temperature:temperature:             35 C
> /proc/acpi/thermal_zone/TZ1/trip_points:critical (S5):           90 C
> /proc/acpi/thermal_zone/TZ1/trip_points:passive:                 87 C: tc1=1 
> tc2=2 tsp=300 devices=CPU0 CPU1 
> /proc/acpi/thermal_zone/TZ2/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ2/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ2/state:state:                   ok
> /proc/acpi/thermal_zone/TZ2/temperature:temperature:             33 C
> /proc/acpi/thermal_zone/TZ2/trip_points:critical (S5):           105 C
> /proc/acpi/thermal_zone/TZ2/trip_points:passive (forced):<not set>
> /proc/acpi/thermal_zone/TZ3/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ3/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ3/state:state:                   ok
> /proc/acpi/thermal_zone/TZ3/temperature:temperature:             31 C
> /proc/acpi/thermal_zone/TZ3/trip_points:critical (S5):           90 C
> /proc/acpi/thermal_zone/TZ3/trip_points:passive:                 60 C: tc1=1 
> tc2=2 tsp=300 devices=CPU0 CPU1 
> /proc/acpi/thermal_zone/TZ4/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ4/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ4/state:state:                   ok
> /proc/acpi/thermal_zone/TZ4/temperature:temperature:             100 C
> /proc/acpi/thermal_zone/TZ4/trip_points:critical (S5):           110 C
> /proc/acpi/thermal_zone/TZ4/trip_points:passive (forced):<not set>
> 
> > 
> > could you please attach the output of "grep . /proc/acpi/thermal/*/*" in
> > 2.6.36 kernel?
> > 
> 2.6.36 after suspend to ram (CPU temp was above 1st trip point):
> 
> maciek@gumis:~$ grep . /proc/acpi/fan/*/*
> /proc/acpi/fan/C325/state:status:                  off
> /proc/acpi/fan/C326/state:status:                  off
> /proc/acpi/fan/C327/state:status:                  off
> /proc/acpi/fan/C328/state:status:                  on
> 
> maciek@gumis:~$ grep . /proc/acpi/thermal_zone/*/*
> /proc/acpi/thermal_zone/TZ0/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ0/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ0/state:state:                   active[0]
> /proc/acpi/thermal_zone/TZ0/temperature:temperature:             50 C
> /proc/acpi/thermal_zone/TZ0/trip_points:critical (S5):           256 C
> /proc/acpi/thermal_zone/TZ0/trip_points:passive (forced):<not set>
> /proc/acpi/thermal_zone/TZ0/trip_points:active[0]:               78 C: 
> devices=C325 
> /proc/acpi/thermal_zone/TZ0/trip_points:active[1]:               70 C: 
> devices=C326 
> /proc/acpi/thermal_zone/TZ0/trip_points:active[2]:               60 C: 
> devices=C327 
> /proc/acpi/thermal_zone/TZ0/trip_points:active[3]:               45 C: 
> devices=C328 
> /proc/acpi/thermal_zone/TZ1/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ1/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ1/state:state:                   ok
> /proc/acpi/thermal_zone/TZ1/temperature:temperature:             47 C
> /proc/acpi/thermal_zone/TZ1/trip_points:critical (S5):           90 C
> /proc/acpi/thermal_zone/TZ1/trip_points:passive:                 87 C: tc1=1 
> tc2=2 tsp=300 devices=CPU0 CPU1 
> /proc/acpi/thermal_zone/TZ2/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ2/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ2/state:state:                   ok
> /proc/acpi/thermal_zone/TZ2/temperature:temperature:             40 C
> /proc/acpi/thermal_zone/TZ2/trip_points:critical (S5):           105 C
> /proc/acpi/thermal_zone/TZ2/trip_points:passive (forced):<not set>
> /proc/acpi/thermal_zone/TZ3/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ3/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ3/state:state:                   ok
> /proc/acpi/thermal_zone/TZ3/temperature:temperature:             28 C
> /proc/acpi/thermal_zone/TZ3/trip_points:critical (S5):           90 C
> /proc/acpi/thermal_zone/TZ3/trip_points:passive:                 60 C: tc1=1 
> tc2=2 tsp=300 devices=CPU0 CPU1 
> /proc/acpi/thermal_zone/TZ4/cooling_mode:<setting not supported>
> /proc/acpi/thermal_zone/TZ4/polling_frequency:<polling disabled>
> /proc/acpi/thermal_zone/TZ4/state:state:                   ok
> /proc/acpi/thermal_zone/TZ4/temperature:temperature:             40 C
> /proc/acpi/thermal_zone/TZ4/trip_points:critical (S5):           110 C
> /proc/acpi/thermal_zone/TZ4/trip_points:passive (forced):<not set>
> 
> Regards


[-- Attachment #2: patch-thermal-regression-debug --]
[-- Type: text/x-vhdl, Size: 878 bytes --]

---
 drivers/thermal/thermal_sys.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Index: linux-2.6/drivers/thermal/thermal_sys.c
===================================================================
--- linux-2.6.orig/drivers/thermal/thermal_sys.c
+++ linux-2.6/drivers/thermal/thermal_sys.c
@@ -966,6 +966,8 @@ void thermal_zone_device_update(struct t
 		       "%d\n", tz->id);
 		goto leave;
 	}
+	printk("Rui: Current temperature: %d\n", temp);
+	dump_stack();
 
 	for (count = 0; count < tz->trips; count++) {
 		tz->ops->get_trip_type(tz, count, &trip_type);
@@ -997,7 +999,7 @@ void thermal_zone_device_update(struct t
 					continue;
 
 				cdev = instance->cdev;
-
+				printk("Rui: Active : cdev %s%d, temperature %lu, trip_temp %lu\n", cdev->type, cdev->id, temp, trip_temp);
 				if (temp >= trip_temp)
 					cdev->ops->set_cur_state(cdev, 1);
 				else

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
       [not found]           ` <201011152139.46261.maciej.rutecki@gmail.com>
@ 2010-11-16  1:26             ` Zhang Rui
  2010-11-16 15:46               ` Maciej Rutecki
                                 ` (2 more replies)
  0 siblings, 3 replies; 11+ messages in thread
From: Zhang Rui @ 2010-11-16  1:26 UTC (permalink / raw)
  To: maciej.rutecki; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

[-- Attachment #1: Type: text/plain, Size: 1262 bytes --]

On Tue, 2010-11-16 at 04:39 +0800, Maciej Rutecki wrote:
> On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
> > On Tue, 2010-11-09 at 01:35 +0800, Maciej Rutecki wrote:
> > > On poniedziałek, 8 listopada 2010 o 07:58:52 Zhang Rui wrote:
> > > > IMO, the only one that may cause this regression is commit
> > > > c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
> > > > some app may enable polling via procfs and update the thermal zone
> > > > status.
> > > 
> > > I revert this commit from 2.6.37-rc1. Still the same:
> > could you please apply the patch attached, on top of 2.6.37-rc1, and
> > then attach the dmesg output if the fan is noise after suspend/resume.
> > 
> 
> Dmesg in attachement.
> 
> Starting suspend to ram from:
> [  104.602674] PM: Syncing filesystems ... done.
> resume OK, but next:
> [  144.298104] PM: Syncing filesystems ... done.
> after resume fan starts on full speed.
> 
it seems that it's not the thermal driver that changes the Fan speed.
Please apply the debug patch attached, rebuild with CONFIG_ACPI_DEBUG
set and reboot with "acpi.debug_level=0x004
acpi.debug_layer=0x04A10000", and then attach the dmesg output after the
same test.
Please attach the acpidump output of this laptop as well.

thanks,
rui

> Regards


[-- Attachment #2: patch-thermal-debug-2 --]
[-- Type: text/x-vhdl, Size: 596 bytes --]

---
 drivers/acpi/bus.c |    5 +++++
 1 file changed, 5 insertions(+)

Index: linux-2.6/drivers/acpi/bus.c
===================================================================
--- linux-2.6.orig/drivers/acpi/bus.c
+++ linux-2.6/drivers/acpi/bus.c
@@ -256,6 +256,11 @@ int acpi_bus_set_power(acpi_handle handl
 	if (result)
 		return result;
 
+	if (!strcmp(acpi_device_hid(device), "PNP0C0B")) {
+		printk("Rui: Change %s state from %d to %d\n", device->pnp.bus_id, device->power.state, state);
+		dump_stack();
+	}
+
 	if ((state < ACPI_STATE_D0) || (state > ACPI_STATE_D3))
 		return -EINVAL;
 

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-16  1:26             ` Zhang Rui
@ 2010-11-16 15:46               ` Maciej Rutecki
  2010-11-16 17:31               ` Maciej Rutecki
  2010-11-16 19:28               ` Maciej Rutecki
  2 siblings, 0 replies; 11+ messages in thread
From: Maciej Rutecki @ 2010-11-16 15:46 UTC (permalink / raw)
  To: Zhang Rui; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
> On Tue, 2010-11-16 at 04:39 +0800, Maciej Rutecki wrote:
> > On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
> > > On Tue, 2010-11-09 at 01:35 +0800, Maciej Rutecki wrote:
> > > > On poniedziałek, 8 listopada 2010 o 07:58:52 Zhang Rui wrote:
> > > > > IMO, the only one that may cause this regression is commit
> > > > > c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
> > > > > some app may enable polling via procfs and update the thermal zone
> > > > > status.
> > > > 
> > > > I revert this commit from 2.6.37-rc1. Still the same:
> > > could you please apply the patch attached, on top of 2.6.37-rc1, and
> > > then attach the dmesg output if the fan is noise after suspend/resume.
> > 
> > Dmesg in attachement.
> > 
> > Starting suspend to ram from:
> > [  104.602674] PM: Syncing filesystems ... done.
> > resume OK, but next:
> > [  144.298104] PM: Syncing filesystems ... done.
> > after resume fan starts on full speed.
> 
> it seems that it's not the thermal driver that changes the Fan speed.
> Please apply the debug patch attached, rebuild with CONFIG_ACPI_DEBUG
> set and reboot with "acpi.debug_level=0x004
> acpi.debug_layer=0x04A10000", and then attach the dmesg output after the
> same test.

I will test it today.

> Please attach the acpidump output of this laptop as well.

http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/acpidump.dat

> 
> thanks,
> rui
> 

Regards
-- 
Maciej Rutecki
http://www.maciek.unixy.pl

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-16  1:26             ` Zhang Rui
  2010-11-16 15:46               ` Maciej Rutecki
@ 2010-11-16 17:31               ` Maciej Rutecki
  2010-11-16 19:28               ` Maciej Rutecki
  2 siblings, 0 replies; 11+ messages in thread
From: Maciej Rutecki @ 2010-11-16 17:31 UTC (permalink / raw)
  To: Zhang Rui; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
> On Tue, 2010-11-16 at 04:39 +0800, Maciej Rutecki wrote:
> > On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
> > > On Tue, 2010-11-09 at 01:35 +0800, Maciej Rutecki wrote:
> > > > On poniedziałek, 8 listopada 2010 o 07:58:52 Zhang Rui wrote:
> > > > > IMO, the only one that may cause this regression is commit
> > > > > c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
> > > > > some app may enable polling via procfs and update the thermal zone
> > > > > status.
> > > > 
> > > > I revert this commit from 2.6.37-rc1. Still the same:
> > > could you please apply the patch attached, on top of 2.6.37-rc1, and
> > > then attach the dmesg output if the fan is noise after suspend/resume.
> > 
> > Dmesg in attachement.
> > 
> > Starting suspend to ram from:
> > [  104.602674] PM: Syncing filesystems ... done.
> > resume OK, but next:
> > [  144.298104] PM: Syncing filesystems ... done.
> > after resume fan starts on full speed.
> 
> it seems that it's not the thermal driver that changes the Fan speed.
> Please apply the debug patch attached, rebuild with CONFIG_ACPI_DEBUG
> set and reboot with "acpi.debug_level=0x004
> acpi.debug_layer=0x04A10000", and then attach the dmesg output after the
> same test.

2.6.37-rc1 + your latest patch, suspend to ram&resume:
http://unixy.pl/maciek/download/kernel/2.6.37-rc1/gumis/dmesg-debug-2.6.37-
rc1.txt.zip

Thanks
-- 
Maciej Rutecki
http://www.maciek.unixy.pl

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-16  1:26             ` Zhang Rui
  2010-11-16 15:46               ` Maciej Rutecki
  2010-11-16 17:31               ` Maciej Rutecki
@ 2010-11-16 19:28               ` Maciej Rutecki
  2010-11-16 22:35                 ` Rafael J. Wysocki
  2 siblings, 1 reply; 11+ messages in thread
From: Maciej Rutecki @ 2010-11-16 19:28 UTC (permalink / raw)
  To: Zhang Rui; +Cc: linux-kernel, linux-acpi, Brown, Len, Rafael J. Wysocki

On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
> On Tue, 2010-11-16 at 04:39 +0800, Maciej Rutecki wrote:
> > On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
> > > On Tue, 2010-11-09 at 01:35 +0800, Maciej Rutecki wrote:
> > > > On poniedziałek, 8 listopada 2010 o 07:58:52 Zhang Rui wrote:
> > > > > IMO, the only one that may cause this regression is commit
> > > > > c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
> > > > > some app may enable polling via procfs and update the thermal zone
> > > > > status.
> > > > 
> > > > I revert this commit from 2.6.37-rc1. Still the same:
> > > could you please apply the patch attached, on top of 2.6.37-rc1, and
> > > then attach the dmesg output if the fan is noise after suspend/resume.
> > 
> > Dmesg in attachement.
> > 
> > Starting suspend to ram from:
> > [  104.602674] PM: Syncing filesystems ... done.
> > resume OK, but next:
> > [  144.298104] PM: Syncing filesystems ... done.
> > after resume fan starts on full speed.
> 
> it seems that it's not the thermal driver that changes the Fan speed.
> Please apply the debug patch attached, rebuild with CONFIG_ACPI_DEBUG
> set and reboot with "acpi.debug_level=0x004
> acpi.debug_layer=0x04A10000", and then attach the dmesg output after the
> same test.
> Please attach the acpidump output of this laptop as well.
> 
> thanks,
> rui
> 
> > Regards

Revert 3e384ee6c687cb397581ee8f9440fc8220cfac80 seems solve problem:
https://bugzilla.kernel.org/show_bug.cgi?id=22932#c3

Thanks
-- 
Maciej Rutecki
http://www.maciek.unixy.pl

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk
  2010-11-16 19:28               ` Maciej Rutecki
@ 2010-11-16 22:35                 ` Rafael J. Wysocki
  0 siblings, 0 replies; 11+ messages in thread
From: Rafael J. Wysocki @ 2010-11-16 22:35 UTC (permalink / raw)
  To: maciej.rutecki; +Cc: Zhang Rui, linux-kernel, linux-acpi, Brown, Len

On Tuesday, November 16, 2010, Maciej Rutecki wrote:
> On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
> > On Tue, 2010-11-16 at 04:39 +0800, Maciej Rutecki wrote:
> > > On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
> > > > On Tue, 2010-11-09 at 01:35 +0800, Maciej Rutecki wrote:
> > > > > On poniedziałek, 8 listopada 2010 o 07:58:52 Zhang Rui wrote:
> > > > > > IMO, the only one that may cause this regression is commit
> > > > > > c57b62f5b1e6dd69ff8c96f6db7f86ea31c0e21f
> > > > > > some app may enable polling via procfs and update the thermal zone
> > > > > > status.
> > > > > 
> > > > > I revert this commit from 2.6.37-rc1. Still the same:
> > > > could you please apply the patch attached, on top of 2.6.37-rc1, and
> > > > then attach the dmesg output if the fan is noise after suspend/resume.
> > > 
> > > Dmesg in attachement.
> > > 
> > > Starting suspend to ram from:
> > > [  104.602674] PM: Syncing filesystems ... done.
> > > resume OK, but next:
> > > [  144.298104] PM: Syncing filesystems ... done.
> > > after resume fan starts on full speed.
> > 
> > it seems that it's not the thermal driver that changes the Fan speed.
> > Please apply the debug patch attached, rebuild with CONFIG_ACPI_DEBUG
> > set and reboot with "acpi.debug_level=0x004
> > acpi.debug_layer=0x04A10000", and then attach the dmesg output after the
> > same test.
> > Please attach the acpidump output of this laptop as well.
> > 
> > thanks,
> > rui
> > 
> > > Regards
> 
> Revert 3e384ee6c687cb397581ee8f9440fc8220cfac80 seems solve problem:
> https://bugzilla.kernel.org/show_bug.cgi?id=22932#c3

Please do the test I asked for in the bug entry.

Thanks,
Rafael

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2010-11-16 22:36 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-04 16:06 [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk Maciej Rutecki
2010-11-05  0:27 ` Zhang Rui
2010-11-05 19:49   ` Maciej Rutecki
2010-11-08  6:58     ` Zhang Rui
2010-11-08 17:35       ` Maciej Rutecki
2010-11-15  3:16         ` Zhang Rui
     [not found]           ` <201011152139.46261.maciej.rutecki@gmail.com>
2010-11-16  1:26             ` Zhang Rui
2010-11-16 15:46               ` Maciej Rutecki
2010-11-16 17:31               ` Maciej Rutecki
2010-11-16 19:28               ` Maciej Rutecki
2010-11-16 22:35                 ` Rafael J. Wysocki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).