Re: [REGRESSION] [2.6.37-rc1] Fan noise after suspend to ram/disk

[Zhang Rui <rui.zhang@intel.com>]

[-- 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;