Re: broken suspend in .2.6.25-rc3 on T61p (was Re: new regressionin 2.6.25-rc3: no keyboard/lid acpi events on thinkpad T61p)

[Pavel Machek]

Hi!

> On Monday, 25 of February 2008, Michael S. Tsirkin wrote:
> > On Mon, Feb 25, 2008 at 9:46 PM, Andrew Morton
> > <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
> > > On Mon, 25 Feb 2008 21:19:24 +0200 "Michael S. Tsirkin" <m.s.tsirkin@xxxxxxxxx> wrote:
> > >
> > >  > On my T61p, 2.6.25-rc2 seems to get acpi events from keypresses
> > >  > such as Fn-F4 and lid open/close, prints them in /var/log/acpid
> > >  > and reacts accordingly (my acpi scripts suspend on lid close and Fn-F4).
> > >
> > >  You mean suspend-to-ram works correctly on your t61p?
> > >
> > >  Mine suspends, then five seconds later magically resumes itself and the
> > >  screen is all black.
> > 
> > Sorry, have not noticed what you were asking about.
> > Yes, rc2 seems to suspend/resume fine.
> > 
> > And after reverting
> > 
> >      revert commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2.
> 
> commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2
> Author: Pavel Machek <pavel@xxxxxx>
> Date:   Thu Feb 21 13:56:55 2008 +0100
> 
>     power_state: get rid of write-only variable in SATA

This is pretty unlikely to be it. Can you double check that this patch
really breaks something?

> >      revert commit 208c70a45624400fafd7511b96bc426bf01f8f5e.
> 
> commit 208c70a45624400fafd7511b96bc426bf01f8f5e
> Author: Alexey Starikovskiy <astarikovskiy@xxxxxxx>
> Date:   Thu Feb 14 15:58:47 2008 -0500
> 
>     ACPI: EC: Use proper handle for boot EC
> 
> > r3 does, too.
> 
> Please, _please_ always add commit subjects to your reports.  Also, please
> include the names of the authors of the commits that turn out to break things
> and send CCs to them.
> 
> It won't hurt to send CCs to the people who signed those commits off, too.

Hmm, as EC is the piece of hw that does the wakeups, yes, EC might be
responsible for autowaking.
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/