linux-kernel - 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)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080225212615.GA2659@elf.ucw.cz>
Date:	Mon, 25 Feb 2008 22:26:15 +0100
From:	Pavel Machek <pavel@....cz>
To:	"Rafael J. Wysocki" <rjw@...k.pl>
Cc:	"Michael S. Tsirkin" <m.s.tsirkin@...il.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>,
	Alexey Starikovskiy <astarikovskiy@...e.de>,
	Len Brown <len.brown@...el.com>, linux-acpi@...r.kernel.org,
	Jeff Garzik <jeff@...zik.org>
Subject: 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)

Hi!

> On Monday, 25 of February 2008, Michael S. Tsirkin wrote:
> > On Mon, Feb 25, 2008 at 9:46 PM, Andrew Morton
> > <akpm@...ux-foundation.org> wrote:
> > > On Mon, 25 Feb 2008 21:19:24 +0200 "Michael S. Tsirkin" <m.s.tsirkin@...il.com> 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@....cz>
> 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@...e.de>
> 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/