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: <8f53421d0802251331n31b37db8gb644dc9aa2d553d4@mail.gmail.com>
Date:	Mon, 25 Feb 2008 23:31:39 +0200
From:	"Michael S. Tsirkin" <m.s.tsirkin@...il.com>
To:	"Pavel Machek" <pavel@....cz>
Cc:	"Rafael J. Wysocki" <rjw@...k.pl>,
	"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)

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

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

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/