Skip to contentSkip to conversation

Fan noise after suspend to ram/disk

REGRESSION Maciej Rutecki 3 participants 11 messages Nov 4, 2010 – Nov 16, 2010 1 message not in the archive

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

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

Signature · 2 lines
Maciej Rutecki
http://www.maciek.unixy.pl
reviewer
#1 lore raw
Quoted 10 lines · Maciej Rutecki

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

Quoted 133 lines

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

Quoted 7 lines

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

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

#2 lore raw
Quoted 6 lines · On piątek, 5 listopada 2010 o 01:27:34 Zhang Rui wrote:
Quoted 5 lines · Maciej Rutecki

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

Quoted 8 lines
Quoted 4 lines

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

Signature · 6 lines
Maciej Rutecki
http://www.maciek.unixy.pl
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
reviewer
#3 lore raw

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

Quoted 89 lines · Maciej Rutecki
Quoted 6 lines · On piątek, 5 listopada 2010 o 01:27:34 Zhang Rui wrote:
Quoted 5 lines · Maciej Rutecki

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

Quoted 8 lines
Quoted 4 lines

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
Quoted 4 lines

Regards

Signature · 2 lines
Maciej Rutecki
http://www.maciek.unixy.pl

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

#4 lore raw
Quoted 4 lines · 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>
Quoted 4 lines

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

Signature · 6 lines
Maciej Rutecki
http://www.maciek.unixy.pl
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
reviewer
#5 lore raw
Quoted 7 lines · Maciej Rutecki
Quoted 4 lines · 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

Quoted 167 lines
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>
Quoted 4 lines

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>
Quoted 1 line

Regards

---
 drivers/thermal/thermal_sys.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/thermal/thermal_sys.c +3 -1
Index: 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

A message that is not in the archive (sent off-list or not yet received): [email protected]

reviewer
#7 lore raw
Quoted 13 lines · Maciej Rutecki
Quoted 11 lines · On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
Quoted 7 lines · Maciej Rutecki
Quoted 4 lines · 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.

Quoted 8 lines

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

Quoted 1 line

Regards

---
 drivers/acpi/bus.c |    5 +++++
 1 file changed, 5 insertions(+)
drivers/acpi/bus.c +5 -0
Index: 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;
 
#8 lore raw
Quoted 21 lines · On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
Quoted 12 lines · Maciej Rutecki
Quoted 10 lines · On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
Quoted 7 lines · Maciej Rutecki
Quoted 4 lines · 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.

Quoted 7 lines

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.

Quoted 5 lines

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.

Quoted 1 line

Please attach the acpidump output of this laptop as well.

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

Quoted 4 lines

thanks,
rui

Regards

Signature · 6 lines
Maciej Rutecki
http://www.maciek.unixy.pl
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
#9 lore raw
In reply to Zhang Rui
Quoted 21 lines · On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
Quoted 12 lines · Maciej Rutecki
Quoted 10 lines · On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
Quoted 7 lines · Maciej Rutecki
Quoted 4 lines · 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.

Quoted 7 lines

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.

Quoted 5 lines

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

Signature · 6 lines
Maciej Rutecki
http://www.maciek.unixy.pl
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
#10 lore raw
In reply to Zhang Rui
Quoted 28 lines · On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
Quoted 12 lines · Maciej Rutecki
Quoted 10 lines · On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
Quoted 7 lines · Maciej Rutecki
Quoted 4 lines · 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.

Quoted 7 lines

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.

Quoted 4 lines

thanks,
rui

Quoted 1 line

Regards

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

Thanks

Signature · 2 lines
Maciej Rutecki
http://www.maciek.unixy.pl
#11 lore raw
Quoted 29 lines · Maciej Rutecki
Quoted 28 lines · On wtorek, 16 listopada 2010 o 02:26:15 Zhang Rui wrote:
Quoted 12 lines · Maciej Rutecki
Quoted 10 lines · On poniedziałek, 15 listopada 2010 o 04:16:35 Zhang Rui wrote:
Quoted 7 lines · Maciej Rutecki
Quoted 4 lines · 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.

Quoted 7 lines

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.

Quoted 7 lines
Quoted 4 lines

thanks,
rui

Quoted 1 line

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
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html