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 Tue, 26 Feb 2008 19:16:11 +0100 Pavel Machek <[EMAIL PROTECTED]> wrote: > On Tue 2008-02-26 13:10:01, Dave Jones wrote: > > On Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: > > > > > > if by 'custom' you mean the solution everyone agreed to work > > > > toward at the power management summit several years ago > > > > (hal/pm-utils) then, yes. > > > > > > I must have been on different summit... I believe it is bad to tie > > > s2ram to hal, because it makes testing on minimal system hard. > > > > > > Anyway, what is the "default" way to trigger s2ram for Andrew? Perhaps > > > Fedora already has his machine whitelisted... > > > > There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. > > Remember that? The cross-distro package everyone agreed was a good idea > > so that every distro didn't have their own magic utility ? > > Well, we have cross-distro package, it is at suspend.sf.net , and it > can bring up video - which is kind of important. (It is single binary, > so it can be pagelocked -- which is important for s2disk). > > Plus it does not depend on HAL. Meanwhile, my computer continues to not work. First thing we need to do is to work out why it won't stay suspended? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 07:16:11PM +0100, Pavel Machek wrote: > On Tue 2008-02-26 13:10:01, Dave Jones wrote: > > On Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: > > > > > > if by 'custom' you mean the solution everyone agreed to work > > > > toward at the power management summit several years ago > > > > (hal/pm-utils) then, yes. > > > > > > I must have been on different summit... I believe it is bad to tie > > > s2ram to hal, because it makes testing on minimal system hard. > > > > > > Anyway, what is the "default" way to trigger s2ram for Andrew? Perhaps > > > Fedora already has his machine whitelisted... > > > > There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. > > Remember that? The cross-distro package everyone agreed was a good idea > > so that every distro didn't have their own magic utility ? > > Well, we have cross-distro package, it is at suspend.sf.net , and it > can bring up video - which is kind of important. (It is single binary, > so it can be pagelocked -- which is important for s2disk). > > Plus it does not depend on HAL. Neither does pm-utils. Once again for the hard of thinking.. The mechanism belongs in pm-utils. HAL is just a fancy wrapper around that. Don't want/like hal? fine, a smaller wrapper around pm-suspend and friends is trivial (or even unnecessary if you're happy with running pm-suspend by hand) Dave -- http://www.codemonkey.org.uk -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue 2008-02-26 13:10:01, Dave Jones wrote: > On Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: > > > > if by 'custom' you mean the solution everyone agreed to work > > > toward at the power management summit several years ago > > > (hal/pm-utils) then, yes. > > > > I must have been on different summit... I believe it is bad to tie > > s2ram to hal, because it makes testing on minimal system hard. > > > > Anyway, what is the "default" way to trigger s2ram for Andrew? Perhaps > > Fedora already has his machine whitelisted... > > There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. > Remember that? The cross-distro package everyone agreed was a good idea > so that every distro didn't have their own magic utility ? Well, we have cross-distro package, it is at suspend.sf.net , and it can bring up video - which is kind of important. (It is single binary, so it can be pagelocked -- which is important for s2disk). Plus it does not depend on HAL. 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: > > if by 'custom' you mean the solution everyone agreed to work > > toward at the power management summit several years ago > > (hal/pm-utils) then, yes. > > I must have been on different summit... I believe it is bad to tie > s2ram to hal, because it makes testing on minimal system hard. > > Anyway, what is the "default" way to trigger s2ram for Andrew? Perhaps > Fedora already has his machine whitelisted... There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. Remember that? The cross-distro package everyone agreed was a good idea so that every distro didn't have their own magic utility ? sigh, I give up. Dave -- http://www.codemonkey.org.uk -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue 2008-02-26 12:46:13, Dave Jones wrote: > On Tue, Feb 26, 2008 at 11:11:17AM +0100, Pavel Machek wrote: > > Hi! > > > > Andrew is trying to get s2ram to work on Fedora: > > > > > > > Please try s2ram, there's good chance it will just work. > > > > > > > > configure: error: Required libx86 was not found > > > > > > apt-get install libx86-dev? > > > > > > Alternatively, can you post dmidecode? Thinkpads usually work with > > > acpi_sleep=s3_bios,s3_mode ; and I can look up whitelist manually. > > > > ...unfortunately, it does not ship s2ram by default, and does not even > > carry its dependencies. > > > > Is there some custom mechanism to get suspend-to-ram to work on > > Fedora? > > if by 'custom' you mean the solution everyone agreed to work > toward at the power management summit several years ago > (hal/pm-utils) then, yes. I must have been on different summit... I believe it is bad to tie s2ram to hal, because it makes testing on minimal system hard. Anyway, what is the "default" way to trigger s2ram for Andrew? Perhaps Fedora already has his machine whitelisted... 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 11:11:17AM +0100, Pavel Machek wrote: > Hi! > > Andrew is trying to get s2ram to work on Fedora: > > > > > Please try s2ram, there's good chance it will just work. > > > > > > configure: error: Required libx86 was not found > > > > apt-get install libx86-dev? > > > > Alternatively, can you post dmidecode? Thinkpads usually work with > > acpi_sleep=s3_bios,s3_mode ; and I can look up whitelist manually. > > ...unfortunately, it does not ship s2ram by default, and does not even > carry its dependencies. > > Is there some custom mechanism to get suspend-to-ram to work on > Fedora? if by 'custom' you mean the solution everyone agreed to work toward at the power management summit several years ago (hal/pm-utils) then, yes. Dave -- http://www.codemonkey.org.uk -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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! Andrew is trying to get s2ram to work on Fedora: > > > Please try s2ram, there's good chance it will just work. > > > > configure: error: Required libx86 was not found > > apt-get install libx86-dev? > > Alternatively, can you post dmidecode? Thinkpads usually work with > acpi_sleep=s3_bios,s3_mode ; and I can look up whitelist manually. ...unfortunately, it does not ship s2ram by default, and does not even carry its dependencies. Is there some custom mechanism to get suspend-to-ram to work on Fedora? If not, would it be possible to start shipping s2ram from suspend.sf.net? (Unlike s2disk, this will not eat filesystems, I promise :-). 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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! Andrew is trying to get s2ram to work on Fedora: Please try s2ram, there's good chance it will just work. configure: error: Required libx86 was not found apt-get install libx86-dev? Alternatively, can you post dmidecode? Thinkpads usually work with acpi_sleep=s3_bios,s3_mode ; and I can look up whitelist manually. ...unfortunately, it does not ship s2ram by default, and does not even carry its dependencies. Is there some custom mechanism to get suspend-to-ram to work on Fedora? If not, would it be possible to start shipping s2ram from suspend.sf.net? (Unlike s2disk, this will not eat filesystems, I promise :-). 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 11:11:17AM +0100, Pavel Machek wrote: Hi! Andrew is trying to get s2ram to work on Fedora: Please try s2ram, there's good chance it will just work. configure: error: Required libx86 was not found apt-get install libx86-dev? Alternatively, can you post dmidecode? Thinkpads usually work with acpi_sleep=s3_bios,s3_mode ; and I can look up whitelist manually. ...unfortunately, it does not ship s2ram by default, and does not even carry its dependencies. Is there some custom mechanism to get suspend-to-ram to work on Fedora? if by 'custom' you mean the solution everyone agreed to work toward at the power management summit several years ago (hal/pm-utils) then, yes. Dave -- http://www.codemonkey.org.uk -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue 2008-02-26 12:46:13, Dave Jones wrote: On Tue, Feb 26, 2008 at 11:11:17AM +0100, Pavel Machek wrote: Hi! Andrew is trying to get s2ram to work on Fedora: Please try s2ram, there's good chance it will just work. configure: error: Required libx86 was not found apt-get install libx86-dev? Alternatively, can you post dmidecode? Thinkpads usually work with acpi_sleep=s3_bios,s3_mode ; and I can look up whitelist manually. ...unfortunately, it does not ship s2ram by default, and does not even carry its dependencies. Is there some custom mechanism to get suspend-to-ram to work on Fedora? if by 'custom' you mean the solution everyone agreed to work toward at the power management summit several years ago (hal/pm-utils) then, yes. I must have been on different summit... I believe it is bad to tie s2ram to hal, because it makes testing on minimal system hard. Anyway, what is the default way to trigger s2ram for Andrew? Perhaps Fedora already has his machine whitelisted... 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue 2008-02-26 13:10:01, Dave Jones wrote: On Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: if by 'custom' you mean the solution everyone agreed to work toward at the power management summit several years ago (hal/pm-utils) then, yes. I must have been on different summit... I believe it is bad to tie s2ram to hal, because it makes testing on minimal system hard. Anyway, what is the default way to trigger s2ram for Andrew? Perhaps Fedora already has his machine whitelisted... There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. Remember that? The cross-distro package everyone agreed was a good idea so that every distro didn't have their own magic utility ? Well, we have cross-distro package, it is at suspend.sf.net , and it can bring up video - which is kind of important. (It is single binary, so it can be pagelocked -- which is important for s2disk). Plus it does not depend on HAL. 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: if by 'custom' you mean the solution everyone agreed to work toward at the power management summit several years ago (hal/pm-utils) then, yes. I must have been on different summit... I believe it is bad to tie s2ram to hal, because it makes testing on minimal system hard. Anyway, what is the default way to trigger s2ram for Andrew? Perhaps Fedora already has his machine whitelisted... There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. Remember that? The cross-distro package everyone agreed was a good idea so that every distro didn't have their own magic utility ? sigh, I give up. Dave -- http://www.codemonkey.org.uk -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 07:16:11PM +0100, Pavel Machek wrote: On Tue 2008-02-26 13:10:01, Dave Jones wrote: On Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: if by 'custom' you mean the solution everyone agreed to work toward at the power management summit several years ago (hal/pm-utils) then, yes. I must have been on different summit... I believe it is bad to tie s2ram to hal, because it makes testing on minimal system hard. Anyway, what is the default way to trigger s2ram for Andrew? Perhaps Fedora already has his machine whitelisted... There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. Remember that? The cross-distro package everyone agreed was a good idea so that every distro didn't have their own magic utility ? Well, we have cross-distro package, it is at suspend.sf.net , and it can bring up video - which is kind of important. (It is single binary, so it can be pagelocked -- which is important for s2disk). Plus it does not depend on HAL. Neither does pm-utils. Once again for the hard of thinking.. The mechanism belongs in pm-utils. HAL is just a fancy wrapper around that. Don't want/like hal? fine, a smaller wrapper around pm-suspend and friends is trivial (or even unnecessary if you're happy with running pm-suspend by hand) Dave -- http://www.codemonkey.org.uk -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, 26 Feb 2008 19:16:11 +0100 Pavel Machek [EMAIL PROTECTED] wrote: On Tue 2008-02-26 13:10:01, Dave Jones wrote: On Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote: if by 'custom' you mean the solution everyone agreed to work toward at the power management summit several years ago (hal/pm-utils) then, yes. I must have been on different summit... I believe it is bad to tie s2ram to hal, because it makes testing on minimal system hard. Anyway, what is the default way to trigger s2ram for Andrew? Perhaps Fedora already has his machine whitelisted... There is no s2ram. pm-suspend uses the white/black-lists in pm-utils. Remember that? The cross-distro package everyone agreed was a good idea so that every distro didn't have their own magic utility ? Well, we have cross-distro package, it is at suspend.sf.net , and it can bring up video - which is kind of important. (It is single binary, so it can be pagelocked -- which is important for s2disk). Plus it does not depend on HAL. Meanwhile, my computer continues to not work. First thing we need to do is to work out why it won't stay suspended? -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 12:53 AM, Andrew Morton <[EMAIL PROTECTED]> wrote: > > On Tue, 26 Feb 2008 00:48:12 +0200 "Michael S. Tsirkin" <[EMAIL PROTECTED]> > wrote: > > > On Tue, Feb 26, 2008 at 12:39 AM, Andrew Morton > > <[EMAIL PROTECTED]> wrote: > > > On Tue, 26 Feb 2008 00:36:54 +0200 "Michael S. Tsirkin" <[EMAIL > PROTECTED]> wrote: > > > > > > > > Hmm, mystery partly solved... as you guessed it, this piece of code > > > > > was not in my tree. > > > > > > > > > > (still, how can this cause autoresume after 5 seconds is a mystery > to > > > > > me). > > > > > > > > > > > > > > > > Pavel > > > > > > > > Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] > > > > > > And earlier - I think 2.6.23 does it as well. > > > > But that one at least resumes fine, does it not? > > Nope, the resume-after-five-seconds and black-screen-after-resume have > always been there (I've only had the thing a few months). > > I thought the restoring of the screen after resume is handled by the X > server? I'm using the nv.o driver. Perhaps nvidia's driver handles it > right, dunno. > > Oh, I have the ATI thingy. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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! > > > > Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] > > > > > > And earlier - I think 2.6.23 does it as well. > > > > But that one at least resumes fine, does it not? > > Nope, the resume-after-five-seconds and black-screen-after-resume have > always been there (I've only had the thing a few months). > > I thought the restoring of the screen after resume is handled by the X > server? I'm using the nv.o driver. Perhaps nvidia's driver handles it > right, dunno. Aha, so you do not have s2ram from suspend.sf.net installed, do you? Restoring the screen is done by either a) kernel/bios (acpi_sleep=..., or better s2ram -f -a X ) b) vbetool c) X . s2ram should detect your machine, and automatically set acpi_sleep and/or perform vbetool magic. Please try s2ram, there's good chance it will just work. 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, 26 Feb 2008 00:48:12 +0200 "Michael S. Tsirkin" <[EMAIL PROTECTED]> wrote: > On Tue, Feb 26, 2008 at 12:39 AM, Andrew Morton > <[EMAIL PROTECTED]> wrote: > > On Tue, 26 Feb 2008 00:36:54 +0200 "Michael S. Tsirkin" <[EMAIL PROTECTED]> > > wrote: > > > > > > Hmm, mystery partly solved... as you guessed it, this piece of code > > > > was not in my tree. > > > > > > > > (still, how can this cause autoresume after 5 seconds is a mystery to > > > > me). > > > > > > > > > > > > Pavel > > > > > > Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] > > > > And earlier - I think 2.6.23 does it as well. > > But that one at least resumes fine, does it not? Nope, the resume-after-five-seconds and black-screen-after-resume have always been there (I've only had the thing a few months). I thought the restoring of the screen after resume is handled by the X server? I'm using the nv.o driver. Perhaps nvidia's driver handles it right, dunno. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 12:39 AM, Andrew Morton <[EMAIL PROTECTED]> wrote: > On Tue, 26 Feb 2008 00:36:54 +0200 "Michael S. Tsirkin" <[EMAIL PROTECTED]> > wrote: > > > > Hmm, mystery partly solved... as you guessed it, this piece of code > > > was not in my tree. > > > > > > (still, how can this cause autoresume after 5 seconds is a mystery to > > > me). > > > > > > > > > Pavel > > > > Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] > > And earlier - I think 2.6.23 does it as well. But that one at least resumes fine, does it not? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, 26 Feb 2008 00:36:54 +0200 "Michael S. Tsirkin" <[EMAIL PROTECTED]> wrote: > > Hmm, mystery partly solved... as you guessed it, this piece of code > > was not in my tree. > > > > (still, how can this cause autoresume after 5 seconds is a mystery to > > me). > > > > > > Pavel > > Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] And earlier - I think 2.6.23 does it as well. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 12:20 AM, Pavel Machek <[EMAIL PROTECTED]> wrote:
>
> Hi!
>
> > > > > > commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2
> > > > > > Author: Pavel Machek <[EMAIL PROTECTED]>
> > > > > > 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.
> > >
> > > Hmm, I guess that should teach me about "simple cleanups".
> > >
> > > Do you use any of:
> > >
> > > ata/sata_inic162x.c
> > > ata/sata_nv.c
> > > ata/sata_sil24.c
> > >
> > > by chance?
> >
> > I don't think so.
> > Here are the only 3 ata modules I have built:
> > drivers/ata/ahci.ko
> > drivers/ata/ata_piix.ko
> > drivers/ata/libata.ko
> >
> >
> > ahci.c seems to look at power_state.
> >
> > static int ahci_pci_device_resume(struct pci_dev *pdev)
> > {
> > struct ata_host *host = dev_get_drvdata(>dev);
> > int rc;
> >
> > rc = ata_pci_device_do_resume(pdev);
> > if (rc)
> > return rc;
> >
> > if (pdev->dev.power.power_state.event == PM_EVENT_SUSPEND) {
> > rc = ahci_reset_controller(host);
> > if (rc)
> > return rc;
> >
> > ahci_init_controller(host);
> > }
>
> > Right?
>
> Hmm, mystery partly solved... as you guessed it, this piece of code
> was not in my tree.
>
> (still, how can this cause autoresume after 5 seconds is a mystery to
> me).
>
>
> Pavel
Maybe it doesn't. Andrew saw the autoresume on -rc[2,3], I didn't.
For me, it causes resume to fail.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
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 2008-02-25 16:32:38, Jeff Garzik wrote: > Pavel Machek wrote: >>> commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 >>> Author: Pavel Machek <[EMAIL PROTECTED]> >>> 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? > > Quote... > > After reverting 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 > on top of 2.6.25-rc3 the kernel again resumes from suspend to > ram. > > Seems pretty clear to me. Yep, that patch was crappy. I developed it on machine with SCSI powersave patches applied, and did not realize this code changed. Sorry. 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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!
> > > > > commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2
> > > > > Author: Pavel Machek <[EMAIL PROTECTED]>
> > > > > 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.
> >
> > Hmm, I guess that should teach me about "simple cleanups".
> >
> > Do you use any of:
> >
> > ata/sata_inic162x.c
> > ata/sata_nv.c
> > ata/sata_sil24.c
> >
> > by chance?
>
> I don't think so.
> Here are the only 3 ata modules I have built:
> drivers/ata/ahci.ko
> drivers/ata/ata_piix.ko
> drivers/ata/libata.ko
>
>
> ahci.c seems to look at power_state.
>
> static int ahci_pci_device_resume(struct pci_dev *pdev)
> {
> struct ata_host *host = dev_get_drvdata(>dev);
> int rc;
>
> rc = ata_pci_device_do_resume(pdev);
> if (rc)
> return rc;
>
> if (pdev->dev.power.power_state.event == PM_EVENT_SUSPEND) {
> rc = ahci_reset_controller(host);
> if (rc)
> return rc;
>
> ahci_init_controller(host);
> }
> Right?
Hmm, mystery partly solved... as you guessed it, this piece of code
was not in my tree.
(still, how can this cause autoresume after 5 seconds is a mystery to
me).
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 [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
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:50 PM, Pavel Machek <[EMAIL PROTECTED]> wrote:
> Hi!
>
>
> > > > commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2
> > > > Author: Pavel Machek <[EMAIL PROTECTED]>
> > > > 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.
>
> Hmm, I guess that should teach me about "simple cleanups".
>
> Do you use any of:
>
> ata/sata_inic162x.c
> ata/sata_nv.c
> ata/sata_sil24.c
>
> by chance?
I don't think so.
Here are the only 3 ata modules I have built:
drivers/ata/ahci.ko
drivers/ata/ata_piix.ko
drivers/ata/libata.ko
ahci.c seems to look at power_state.
static int ahci_pci_device_resume(struct pci_dev *pdev)
{
struct ata_host *host = dev_get_drvdata(>dev);
int rc;
rc = ata_pci_device_do_resume(pdev);
if (rc)
return rc;
if (pdev->dev.power.power_state.event == PM_EVENT_SUSPEND) {
rc = ahci_reset_controller(host);
if (rc)
return rc;
ahci_init_controller(host);
}
ata_host_resume(host);
return 0;
}
Right?
> (Ok, the patch is very safe to revert, it was "cleanup", it fixes
> nothing).
>
>
> Pavel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
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! > > > commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 > > > Author: Pavel Machek <[EMAIL PROTECTED]> > > > 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. Hmm, I guess that should teach me about "simple cleanups". Do you use any of: ata/sata_inic162x.c ata/sata_nv.c ata/sata_sil24.c by chance? (Ok, the patch is very safe to revert, it was "cleanup", it fixes nothing). 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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)
Pavel Machek wrote: commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 Author: Pavel Machek <[EMAIL PROTECTED]> 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? Quote... After reverting 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 on top of 2.6.25-rc3 the kernel again resumes from suspend to ram. Seems pretty clear to me. Jeff -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 <[EMAIL PROTECTED]> wrote: > Hi! > > > > On Monday, 25 of February 2008, Michael S. Tsirkin wrote: > > > On Mon, Feb 25, 2008 at 9:46 PM, Andrew Morton > > > <[EMAIL PROTECTED]> wrote: > > > > On Mon, 25 Feb 2008 21:19:24 +0200 "Michael S. Tsirkin" <[EMAIL > PROTECTED]> 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 <[EMAIL PROTECTED]> > > 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 <[EMAIL PROTECTED]> > > 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 > > <[EMAIL PROTECTED]> wrote: > > > On Mon, 25 Feb 2008 21:19:24 +0200 "Michael S. Tsirkin" <[EMAIL > > > PROTECTED]> 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 <[EMAIL PROTECTED]> > 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 <[EMAIL PROTECTED]> > 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Monday, 25 of February 2008, Michael S. Tsirkin wrote: > On Mon, Feb 25, 2008 at 9:46 PM, Andrew Morton > <[EMAIL PROTECTED]> wrote: > > On Mon, 25 Feb 2008 21:19:24 +0200 "Michael S. Tsirkin" <[EMAIL PROTECTED]> > > 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 <[EMAIL PROTECTED]> Date: Thu Feb 21 13:56:55 2008 +0100 power_state: get rid of write-only variable in SATA > revert commit 208c70a45624400fafd7511b96bc426bf01f8f5e. commit 208c70a45624400fafd7511b96bc426bf01f8f5e Author: Alexey Starikovskiy <[EMAIL PROTECTED]> 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. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Monday, 25 of February 2008, Michael S. Tsirkin wrote: On Mon, Feb 25, 2008 at 9:46 PM, Andrew Morton [EMAIL PROTECTED] wrote: On Mon, 25 Feb 2008 21:19:24 +0200 Michael S. Tsirkin [EMAIL PROTECTED] 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 [EMAIL PROTECTED] Date: Thu Feb 21 13:56:55 2008 +0100 power_state: get rid of write-only variable in SATA revert commit 208c70a45624400fafd7511b96bc426bf01f8f5e. commit 208c70a45624400fafd7511b96bc426bf01f8f5e Author: Alexey Starikovskiy [EMAIL PROTECTED] 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. Thanks, Rafael -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 [EMAIL PROTECTED] wrote: On Mon, 25 Feb 2008 21:19:24 +0200 Michael S. Tsirkin [EMAIL PROTECTED] 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 [EMAIL PROTECTED] 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 [EMAIL PROTECTED] 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 [EMAIL PROTECTED] wrote: Hi! On Monday, 25 of February 2008, Michael S. Tsirkin wrote: On Mon, Feb 25, 2008 at 9:46 PM, Andrew Morton [EMAIL PROTECTED] wrote: On Mon, 25 Feb 2008 21:19:24 +0200 Michael S. Tsirkin [EMAIL PROTECTED] 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 [EMAIL PROTECTED] 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 [EMAIL PROTECTED] 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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)
Pavel Machek wrote: commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 Author: Pavel Machek [EMAIL PROTECTED] 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? Quote... After reverting 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 on top of 2.6.25-rc3 the kernel again resumes from suspend to ram. Seems pretty clear to me. Jeff -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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! commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 Author: Pavel Machek [EMAIL PROTECTED] 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. Hmm, I guess that should teach me about simple cleanups. Do you use any of: ata/sata_inic162x.c ata/sata_nv.c ata/sata_sil24.c by chance? (Ok, the patch is very safe to revert, it was cleanup, it fixes nothing). 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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:50 PM, Pavel Machek [EMAIL PROTECTED] wrote:
Hi!
commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2
Author: Pavel Machek [EMAIL PROTECTED]
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.
Hmm, I guess that should teach me about simple cleanups.
Do you use any of:
ata/sata_inic162x.c
ata/sata_nv.c
ata/sata_sil24.c
by chance?
I don't think so.
Here are the only 3 ata modules I have built:
drivers/ata/ahci.ko
drivers/ata/ata_piix.ko
drivers/ata/libata.ko
ahci.c seems to look at power_state.
static int ahci_pci_device_resume(struct pci_dev *pdev)
{
struct ata_host *host = dev_get_drvdata(pdev-dev);
int rc;
rc = ata_pci_device_do_resume(pdev);
if (rc)
return rc;
if (pdev-dev.power.power_state.event == PM_EVENT_SUSPEND) {
rc = ahci_reset_controller(host);
if (rc)
return rc;
ahci_init_controller(host);
}
ata_host_resume(host);
return 0;
}
Right?
(Ok, the patch is very safe to revert, it was cleanup, it fixes
nothing).
Pavel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
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!
commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2
Author: Pavel Machek [EMAIL PROTECTED]
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.
Hmm, I guess that should teach me about simple cleanups.
Do you use any of:
ata/sata_inic162x.c
ata/sata_nv.c
ata/sata_sil24.c
by chance?
I don't think so.
Here are the only 3 ata modules I have built:
drivers/ata/ahci.ko
drivers/ata/ata_piix.ko
drivers/ata/libata.ko
ahci.c seems to look at power_state.
static int ahci_pci_device_resume(struct pci_dev *pdev)
{
struct ata_host *host = dev_get_drvdata(pdev-dev);
int rc;
rc = ata_pci_device_do_resume(pdev);
if (rc)
return rc;
if (pdev-dev.power.power_state.event == PM_EVENT_SUSPEND) {
rc = ahci_reset_controller(host);
if (rc)
return rc;
ahci_init_controller(host);
}
Right?
Hmm, mystery partly solved... as you guessed it, this piece of code
was not in my tree.
(still, how can this cause autoresume after 5 seconds is a mystery to
me).
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 [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
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 2008-02-25 16:32:38, Jeff Garzik wrote: Pavel Machek wrote: commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 Author: Pavel Machek [EMAIL PROTECTED] 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? Quote... After reverting 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 on top of 2.6.25-rc3 the kernel again resumes from suspend to ram. Seems pretty clear to me. Yep, that patch was crappy. I developed it on machine with SCSI powersave patches applied, and did not realize this code changed. Sorry. 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 12:20 AM, Pavel Machek [EMAIL PROTECTED] wrote:
Hi!
commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2
Author: Pavel Machek [EMAIL PROTECTED]
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.
Hmm, I guess that should teach me about simple cleanups.
Do you use any of:
ata/sata_inic162x.c
ata/sata_nv.c
ata/sata_sil24.c
by chance?
I don't think so.
Here are the only 3 ata modules I have built:
drivers/ata/ahci.ko
drivers/ata/ata_piix.ko
drivers/ata/libata.ko
ahci.c seems to look at power_state.
static int ahci_pci_device_resume(struct pci_dev *pdev)
{
struct ata_host *host = dev_get_drvdata(pdev-dev);
int rc;
rc = ata_pci_device_do_resume(pdev);
if (rc)
return rc;
if (pdev-dev.power.power_state.event == PM_EVENT_SUSPEND) {
rc = ahci_reset_controller(host);
if (rc)
return rc;
ahci_init_controller(host);
}
Right?
Hmm, mystery partly solved... as you guessed it, this piece of code
was not in my tree.
(still, how can this cause autoresume after 5 seconds is a mystery to
me).
Pavel
Maybe it doesn't. Andrew saw the autoresume on -rc[2,3], I didn't.
For me, it causes resume to fail.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
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 Tue, 26 Feb 2008 00:36:54 +0200 Michael S. Tsirkin [EMAIL PROTECTED] wrote: Hmm, mystery partly solved... as you guessed it, this piece of code was not in my tree. (still, how can this cause autoresume after 5 seconds is a mystery to me). Pavel Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] And earlier - I think 2.6.23 does it as well. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, 26 Feb 2008 00:48:12 +0200 Michael S. Tsirkin [EMAIL PROTECTED] wrote: On Tue, Feb 26, 2008 at 12:39 AM, Andrew Morton [EMAIL PROTECTED] wrote: On Tue, 26 Feb 2008 00:36:54 +0200 Michael S. Tsirkin [EMAIL PROTECTED] wrote: Hmm, mystery partly solved... as you guessed it, this piece of code was not in my tree. (still, how can this cause autoresume after 5 seconds is a mystery to me). Pavel Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] And earlier - I think 2.6.23 does it as well. But that one at least resumes fine, does it not? Nope, the resume-after-five-seconds and black-screen-after-resume have always been there (I've only had the thing a few months). I thought the restoring of the screen after resume is handled by the X server? I'm using the nv.o driver. Perhaps nvidia's driver handles it right, dunno. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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! Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] And earlier - I think 2.6.23 does it as well. But that one at least resumes fine, does it not? Nope, the resume-after-five-seconds and black-screen-after-resume have always been there (I've only had the thing a few months). I thought the restoring of the screen after resume is handled by the X server? I'm using the nv.o driver. Perhaps nvidia's driver handles it right, dunno. Aha, so you do not have s2ram from suspend.sf.net installed, do you? Restoring the screen is done by either a) kernel/bios (acpi_sleep=..., or better s2ram -f -a X ) b) vbetool c) X . s2ram should detect your machine, and automatically set acpi_sleep and/or perform vbetool magic. Please try s2ram, there's good chance it will just work. 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 12:53 AM, Andrew Morton [EMAIL PROTECTED] wrote: On Tue, 26 Feb 2008 00:48:12 +0200 Michael S. Tsirkin [EMAIL PROTECTED] wrote: On Tue, Feb 26, 2008 at 12:39 AM, Andrew Morton [EMAIL PROTECTED] wrote: On Tue, 26 Feb 2008 00:36:54 +0200 Michael S. Tsirkin [EMAIL PROTECTED] wrote: Hmm, mystery partly solved... as you guessed it, this piece of code was not in my tree. (still, how can this cause autoresume after 5 seconds is a mystery to me). Pavel Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] And earlier - I think 2.6.23 does it as well. But that one at least resumes fine, does it not? Nope, the resume-after-five-seconds and black-screen-after-resume have always been there (I've only had the thing a few months). I thought the restoring of the screen after resume is handled by the X server? I'm using the nv.o driver. Perhaps nvidia's driver handles it right, dunno. Oh, I have the ATI thingy. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
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 Tue, Feb 26, 2008 at 12:39 AM, Andrew Morton [EMAIL PROTECTED] wrote: On Tue, 26 Feb 2008 00:36:54 +0200 Michael S. Tsirkin [EMAIL PROTECTED] wrote: Hmm, mystery partly solved... as you guessed it, this piece of code was not in my tree. (still, how can this cause autoresume after 5 seconds is a mystery to me). Pavel Maybe it doesn't. Andrew saw the autoresume on -rc[2,3] And earlier - I think 2.6.23 does it as well. But that one at least resumes fine, does it not? -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
