Date: Wed, 5 Nov 2008 14:24:01 -0200
From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: ibm-acpi-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org
Subject: Re: [ibm-acpi-devel] Fan level 7 after resume wit 2.6.28-rc3
Message-ID: <20081105162401.GF14931@khazad-dum.debian.net>
References: <20081105073306.GA3132@x61> <20081105074712.GA2816@x61> <20081105122632.GC14931@khazad-dum.debian.net> <20081105130852.GB30758@dose.home.local>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20081105130852.GB30758@dose.home.local>
User-Agent: Mutt/1.5.18 (2008-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2211
Lines: 55

On Wed, 05 Nov 2008, Tino Keitel wrote:
> On Wed, Nov 05, 2008 at 10:26:32 -0200, Henrique de Moraes Holschuh wrote:
> > On Wed, 05 Nov 2008, Tino Keitel wrote:
> > > On Wed, Nov 05, 2008 at 08:33:08 +0100, Tino Keitel wrote:
> > > > I tried 2.6.28-rc3 (git a75952b) on my ThinkPad X61s, and now the fan
> > > > runs at full speed after resume from suspend:
> > > 
> > > I forgot to look into the kernel logs for more info. Well, I found this
> > > line:
> > > 
> > > thinkpad_acpi: restoring fan level to 0x07
> > > 
> > > So I guess that according to
> > > 
> > > ACPI: thinkpad-acpi: attempt to preserve fan state on resume
> > > 
> > > the attempt failed. :-)
> > 
> > Indeed.  Please send the full kernel output of thinkpad-acpi, showing boot,
> > suspend and resume messages from the driver, please?
> 
> See http://tikei.de/kernel.log
> 
> > Also, please tell me what is the contents of /proc/acpi/ibm/fan (and if
> 
> See the first mail. I don't touch any fan settings, everything is at
> the default value.
> 
> > possible, the contents of /proc/acpi/ibm/ecdump) before you issue the
> > suspend command.
> 
> See http://tikei.de/ecdump (however, this was taken with 2.6.27.4).

It should not have set the fan to level 7, so it means we will need to add
debugging hints to the driver.  I don't have my T43 with me right now, or I
could try to reproduce the issue.

Are you confortable with adding some printk's to kernel source?  If so, try
to printk the interesting bits in fan_suspend and fan_resume on
drivers/misc/thinkpad_acpi.c.

Otherwise, I can provide you with a patch with the debug printks later.  But
either way, you'd need to recompile your kernel to get the changed
thinkpad-acpi module to test.  I hope that is not a problem?

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/