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

From: Michael S. Tsirkin
Date: Mon Feb 25 2008 - 16:32:05 EST


On Mon, Feb 25, 2008 at 11:26 PM, Pavel Machek <pavel@xxxxxx> wrote:
> 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?

I did and it seems to: just reverting
559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 fixes resume for me.
635adc28087ced0c843d2ecb6d4ae474d0e611cd which is
559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2~1
also resumes fine.

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

As far as I can tell, nope, reverting
208c70a45624400fafd7511b96bc426bf01f8f5e fixes acpi
keyboard/lid events but does fix resume from suspend to ram for me.
--
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/