From: Jesse Barnes <jesse.barnes@intel.com>
To: "Jeff Chua" <jeff.chua.linux@gmail.com>
Cc: "Romano Giannetti" <romano@dea.icai.upcomillas.es>,
"Linus Torvalds" <torvalds@linux-foundation.org>,
"suspend-devel List" <suspend-devel@lists.sourceforge.net>,
"Dave Airlie" <airlied@linux.ie>, "Greg KH" <gregkh@suse.de>,
lkml <linux-kernel@vger.kernel.org>,
"Rafael J. Wysocki" <rjw@sisk.pl>,
linux-acpi@vger.kernel.org
Subject: Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off aftersuspend-to-disk. Screen becomes green.
Date: Thu, 21 Feb 2008 17:02:01 -0800 [thread overview]
Message-ID: <200802211702.02306.jesse.barnes@intel.com> (raw)
In-Reply-To: <b6a2187b0802211652x304daa2ci36a76a76a2fcb82@mail.gmail.com>
On Thursday, February 21, 2008 4:52 pm Jeff Chua wrote:
> On Fri, Feb 22, 2008 at 8:46 AM, Jesse Barnes <jesse.barnes@intel.com>
wrote:
> > Your s2ram script is doing your STD also? Seems counterintuitive.
> > Anyway, some machines also re-POST the GPU on resume from S3; maybe yours
> > is doing that.
>
> It's s2ram to do STR, not STD. Sorry for the confusion. But the key
> point is there's no GREEN for STR. Mr Green only appear with STD.
Ah, ok that makes sense.
So typically, what you'd see at suspend time is this ugly call chain:
1) user requests suspend or hibernate
2) kernel kicks users off VT
3) X calls LeaveVT function of X driver
4) X driver restores whatever video state it felt like saving when it started
up
5) kernel calls suspend methods
6) machine goes to sleep
then on resume:
1) user requests wakeup
2) kernel calls resume methods
3) X takes back the VT, calling driver EnterVT methods
4) X driver EnterVT routine runs, doing whatever it wants
5) you're back to where you were (on a good day anyway)
So, on your machine, I suspect your firmware is doing enough that X doesn't
have to save/restore full video state around Enter/Leave VT (the same
functions called at VT switch time when you press ctl-alt-fx), otherwise
you'd be missing things like your backlight or text consoles.
So the advantage of the kernel suspend/resume hooks for the DRM layer is that
the kernel video drivers can do full state save/restore (which X usually
doesn't do, and isn't really designed to do), so that if your platform
*doesn't* do it all, you'll still end up with a usable machine in the end.
The fact that you'd started running into problems since we merged this just
means your platform was taking care of it for you (lucky you) and that we
have some bugs in the hibernate code that we're just discovering.
Jesse
next prev parent reply other threads:[~2008-02-22 1:16 UTC|newest]
Thread overview: 111+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-20 17:17 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green Jeff Chua
2008-02-20 17:19 ` Jeff Chua
2008-02-20 17:28 ` Linus Torvalds
2008-02-20 17:37 ` Jeff Chua
2008-02-20 17:52 ` Linus Torvalds
2008-02-20 18:02 ` Jeff Chua
2008-02-21 19:43 ` [Suspend-devel] 2.6.25-rc2 System no longer powers off aftersuspend-to-disk. " Romano Giannetti
2008-02-21 21:02 ` Jesse Barnes
2008-02-22 0:20 ` Jeff Chua
2008-02-22 0:23 ` Jesse Barnes
2008-02-22 0:42 ` Jeff Chua
2008-02-22 0:46 ` Jesse Barnes
2008-02-22 0:52 ` Jeff Chua
2008-02-22 1:02 ` Jesse Barnes [this message]
2008-02-22 1:27 ` Jeff Chua
2008-02-22 1:28 ` Linus Torvalds
2008-02-22 1:35 ` Jesse Barnes
2008-02-22 10:37 ` Matthew Garrett
2008-02-22 13:06 ` Ingo Molnar
2008-02-22 16:10 ` Rafael J. Wysocki
2008-02-22 16:50 ` Linus Torvalds
2008-02-22 18:01 ` Matthew Garrett
2008-02-23 11:17 ` Ingo Molnar
2008-02-22 0:31 ` Rafael J. Wysocki
2008-02-22 0:42 ` Jeff Chua
2008-02-22 1:01 ` Jeff Chua
2008-02-22 1:06 ` Rafael J. Wysocki
2008-02-22 0:46 ` Linus Torvalds
2008-02-22 0:54 ` Rafael J. Wysocki
2008-02-22 1:13 ` Jesse Barnes
2008-02-22 1:44 ` Jesse Barnes
2008-02-22 23:31 ` i915 hibernation patch (was: Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off aftersuspend-to-disk. Screen becomes green.) Rafael J. Wysocki
2008-02-23 1:00 ` Linus Torvalds
2008-02-23 1:55 ` Rafael J. Wysocki
2008-02-23 2:07 ` Linus Torvalds
2008-02-23 4:36 ` Jeff Chua
2008-02-23 18:13 ` [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...)) Rafael J. Wysocki
2008-02-23 18:43 ` Linus Torvalds
2008-02-24 8:29 ` Pavel Machek
2008-02-24 15:21 ` Jeff Chua
2008-02-24 4:16 ` Mirco Tischler
2008-02-24 11:11 ` Rafael J. Wysocki
2008-02-24 11:25 ` Rafael J. Wysocki
2008-02-23 11:17 ` i915 hibernation patch (was: Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off aftersuspend-to-disk. Screen becomes green.) Pavel Machek
2008-02-20 17:54 ` 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green Jeff Chua
2008-02-20 18:37 ` Linus Torvalds
2008-02-18 6:31 ` [Suspend-devel] " Pavel Machek
2008-02-20 18:49 ` Jeff Chua
2008-02-20 19:25 ` Matthew Garrett
2008-02-20 18:57 ` Jesse Barnes
2008-02-20 17:50 ` Jesse Barnes
2008-02-20 18:29 ` Jeff Chua
2008-02-20 18:53 ` Jesse Barnes
2008-02-20 19:10 ` Jeff Chua
2008-02-20 19:18 ` Jesse Barnes
2008-02-20 20:09 ` Jesse Barnes
2008-02-20 20:14 ` Rafael J. Wysocki
2008-02-20 20:29 ` Linus Torvalds
2008-02-20 20:33 ` [Suspend-devel] " Pablo Sanchez
2008-02-20 20:41 ` Jesse Barnes
2008-02-20 21:13 ` Linus Torvalds
2008-02-20 21:44 ` Jesse Barnes
2008-02-20 22:22 ` Linus Torvalds
2008-02-21 8:30 ` david
2008-02-22 16:56 ` Mark Lord
2008-02-22 17:02 ` Rafael J. Wysocki
2008-02-22 17:32 ` Mark Lord
2008-02-22 17:44 ` Rafael J. Wysocki
2008-02-22 19:23 ` david
2008-02-22 23:16 ` Rafael J. Wysocki
2008-02-20 22:36 ` Rafael J. Wysocki
2008-02-20 23:13 ` Linus Torvalds
2008-02-20 23:35 ` Rafael J. Wysocki
2008-02-21 0:00 ` Linus Torvalds
2008-02-21 0:13 ` Rafael J. Wysocki
2008-02-21 0:25 ` Linus Torvalds
2008-02-21 0:59 ` Rafael J. Wysocki
2008-02-22 16:54 ` Mark Lord
2008-02-20 22:45 ` Nigel Cunningham
2008-02-21 0:13 ` Matthew Garrett
2008-02-21 0:40 ` Nigel Cunningham
2008-02-21 0:46 ` Greg KH
2008-02-21 1:17 ` Nigel Cunningham
2008-02-21 4:43 ` Greg KH
2008-02-21 6:05 ` Nigel Cunningham
2008-02-21 6:37 ` Greg KH
2008-02-21 1:10 ` Matthew Garrett
2008-02-21 1:25 ` Nigel Cunningham
2008-02-20 20:45 ` [Suspend-devel] " Rafael J. Wysocki
2008-02-20 21:26 ` Alexey Starikovskiy
2008-02-20 21:37 ` Jesse Barnes
2008-02-21 0:35 ` Jeff Chua
2008-02-21 0:39 ` Jesse Barnes
2008-02-21 1:19 ` Jeff Chua
2008-02-21 1:21 ` Jesse Barnes
2008-02-21 1:49 ` Jeff Chua
2008-02-21 2:00 ` Jeff Chua
2008-02-21 16:27 ` [Suspend-devel] " Rafael J. Wysocki
2008-02-21 18:34 ` Jesse Barnes
2008-02-21 20:30 ` Rafael J. Wysocki
2008-02-21 22:11 ` Rafael J. Wysocki
2008-02-21 23:45 ` Jesse Barnes
2008-02-22 0:28 ` Rafael J. Wysocki
2008-02-22 0:48 ` Jeff Chua
2008-02-20 22:32 ` Jesse Barnes
2008-02-20 23:03 ` Jesse Barnes
2008-02-20 23:34 ` Jesse Barnes
2008-02-20 23:49 ` Rafael J. Wysocki
2008-02-21 0:17 ` Jesse Barnes
2008-02-21 1:07 ` [Suspend-devel] " Rafael J. Wysocki
2008-02-20 18:47 ` Mark Lord
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200802211702.02306.jesse.barnes@intel.com \
--to=jesse.barnes@intel.com \
--cc=airlied@linux.ie \
--cc=gregkh@suse.de \
--cc=jeff.chua.linux@gmail.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rjw@sisk.pl \
--cc=romano@dea.icai.upcomillas.es \
--cc=suspend-devel@lists.sourceforge.net \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox