From: Jesse Barnes <jesse.barnes@intel.com>
To: Jeff Chua <jeff.chua.linux@gmail.com>
Cc: 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,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Date: Wed, 20 Feb 2008 09:50:52 -0800 [thread overview]
Message-ID: <200802200950.53591.jesse.barnes@intel.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802210101050.3576@boston.corp.fedex.com>
On Wednesday, February 20, 2008 9:17 am Jeff Chua wrote:
> On Feb 20, 2008 2:19 PM, Jeff Chua
>
> > I'll try the "idle=poll" to see if that works and will try some printk
>
> I don't know what exactly the i915_suspend() and i915_resume() are
> supposed to do because it works better without them.
>
> After inserting "return 0;" right at the top of those two functions,
> suspend (and power-off properly), and resume (without green screen) works
> just fine.
>
> I would like to know what they're for.
They're for saving and restoring GPU state across suspend/resume. They're
particularly useful if your machine doesn't re-POST at resume time. In that
case your GPU may be totally uninitialized, so either the kernel or X has to
set it up for you (X only does that partially).
> Tested suspend-to-ram, and suspend-to-disk, both console and X on notebook
> internal LCD display, all works without these two functions.
>
> But, anyway, got down to just one line in i915_drv.c causing the hang
> during suspend. "pci_set_power_state(dev->pdev, PCI_D3hot);".
Interesting, which chipset do you have? AFAIK that shouldn't cause a hang.
> And green screen problem during resume is caused by i915_restore_vga(dev);
I know I fixed that problem in at least one configuration... Can you try:
# echo test > /sys/power/disk
# echo disk > /sys/power/state
and see if that also turns your screen green?
Also, getting a GPU register dump would be helpful. The intel_reg_dumper tool
is built as part of the xf86-video-driver build
(git://anongit.freedesktop.org/git/xorg/driver/xf86-video-intel), can you
pull that down and try it out?
Thanks,
Jesse
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
WARNING: multiple messages have this Message-ID (diff)
From: Jesse Barnes <jesse.barnes@intel.com>
To: Jeff Chua <jeff.chua.linux@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
lkml <linux-kernel@vger.kernel.org>,
Dave Airlie <airlied@linux.ie>, "Rafael J. Wysocki" <rjw@sisk.pl>,
linux-acpi@vger.kernel.org,
suspend-devel List <suspend-devel@lists.sourceforge.net>,
Greg KH <gregkh@suse.de>
Subject: Re: 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green.
Date: Wed, 20 Feb 2008 09:50:52 -0800 [thread overview]
Message-ID: <200802200950.53591.jesse.barnes@intel.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802210101050.3576@boston.corp.fedex.com>
On Wednesday, February 20, 2008 9:17 am Jeff Chua wrote:
> On Feb 20, 2008 2:19 PM, Jeff Chua
>
> > I'll try the "idle=poll" to see if that works and will try some printk
>
> I don't know what exactly the i915_suspend() and i915_resume() are
> supposed to do because it works better without them.
>
> After inserting "return 0;" right at the top of those two functions,
> suspend (and power-off properly), and resume (without green screen) works
> just fine.
>
> I would like to know what they're for.
They're for saving and restoring GPU state across suspend/resume. They're
particularly useful if your machine doesn't re-POST at resume time. In that
case your GPU may be totally uninitialized, so either the kernel or X has to
set it up for you (X only does that partially).
> Tested suspend-to-ram, and suspend-to-disk, both console and X on notebook
> internal LCD display, all works without these two functions.
>
> But, anyway, got down to just one line in i915_drv.c causing the hang
> during suspend. "pci_set_power_state(dev->pdev, PCI_D3hot);".
Interesting, which chipset do you have? AFAIK that shouldn't cause a hang.
> And green screen problem during resume is caused by i915_restore_vga(dev);
I know I fixed that problem in at least one configuration... Can you try:
# echo test > /sys/power/disk
# echo disk > /sys/power/state
and see if that also turns your screen green?
Also, getting a GPU register dump would be helpful. The intel_reg_dumper tool
is built as part of the xf86-video-driver build
(git://anongit.freedesktop.org/git/xorg/driver/xf86-video-intel), can you
pull that down and try it out?
Thanks,
Jesse
next prev parent reply other threads:[~2008-02-20 17:50 UTC|newest]
Thread overview: 186+ 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:19 ` Jeff Chua
2008-02-20 17:28 ` Linus Torvalds
2008-02-20 17:28 ` Linus Torvalds
2008-02-20 17:37 ` Jeff Chua
2008-02-20 17:52 ` Linus Torvalds
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 19:43 ` Romano Giannetti
2008-02-21 21:02 ` Jesse Barnes
2008-02-21 21:02 ` [Suspend-devel] " 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:42 ` [Suspend-devel] " Jeff Chua
2008-02-22 0:46 ` Jesse Barnes
2008-02-22 0:52 ` Jeff Chua
2008-02-22 0:52 ` [Suspend-devel] " Jeff Chua
2008-02-22 1:02 ` Jesse Barnes
2008-02-22 1:02 ` [Suspend-devel] " Jesse Barnes
2008-02-22 1:27 ` Jeff Chua
2008-02-22 1:27 ` [Suspend-devel] " Jeff Chua
2008-02-22 1:28 ` Linus Torvalds
2008-02-22 1:28 ` [Suspend-devel] " 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 0:42 ` [Suspend-devel] " Jeff Chua
2008-02-22 1:01 ` Jeff Chua
2008-02-22 1:01 ` [Suspend-devel] " Jeff Chua
2008-02-22 1:06 ` Rafael J. Wysocki
2008-02-22 1:06 ` [Suspend-devel] " Rafael J. Wysocki
2008-02-22 0:46 ` Linus Torvalds
2008-02-22 0:46 ` [Suspend-devel] " Linus Torvalds
2008-02-22 0:54 ` Rafael J. Wysocki
2008-02-22 0:54 ` [Suspend-devel] " 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:00 ` Linus Torvalds
2008-02-23 1:55 ` Rafael J. Wysocki
2008-02-23 2:07 ` Linus Torvalds
2008-02-23 2:07 ` Linus Torvalds
2008-02-23 4:36 ` Jeff Chua
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:13 ` Rafael J. Wysocki
2008-02-23 18:43 ` Linus Torvalds
2008-02-23 18:43 ` Linus Torvalds
2008-02-24 8:29 ` Pavel Machek
2008-02-24 8:29 ` Pavel Machek
2008-02-24 15:21 ` Jeff Chua
2008-02-24 15:21 ` Jeff Chua
2008-02-24 4:16 ` Mirco Tischler
2008-02-24 4:16 ` Mirco Tischler
2008-02-24 11:11 ` Rafael J. Wysocki
2008-02-24 11:11 ` Rafael J. Wysocki
2008-02-24 11:11 ` Rafael J. Wysocki
2008-02-24 11:25 ` Rafael J. Wysocki
2008-02-24 11:25 ` 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-23 11:17 ` Pavel Machek
2008-02-23 1:55 ` Rafael J. Wysocki
2008-02-22 23:31 ` Rafael J. Wysocki
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 17:54 ` Jeff Chua
2008-02-20 18:37 ` Linus Torvalds
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 19:25 ` Matthew Garrett
2008-02-20 18:57 ` Jesse Barnes
2008-02-20 17:50 ` Jesse Barnes [this message]
2008-02-20 17:50 ` Jesse Barnes
2008-02-20 18:29 ` Jeff Chua
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 19:18 ` Jesse Barnes
2008-02-20 20:09 ` 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:29 ` Linus Torvalds
2008-02-20 20:33 ` Pablo Sanchez
2008-02-20 20:33 ` [Suspend-devel] " Pablo Sanchez
2008-02-20 20:41 ` Jesse Barnes
2008-02-20 20:41 ` Jesse Barnes
2008-02-20 21:13 ` Linus Torvalds
2008-02-20 21:44 ` Jesse Barnes
2008-02-20 21:44 ` Jesse Barnes
2008-02-20 22:22 ` Linus Torvalds
2008-02-20 22:22 ` Linus Torvalds
2008-02-21 8:30 ` david
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 22:36 ` Rafael J. Wysocki
2008-02-20 23:13 ` Linus Torvalds
2008-02-20 23:35 ` Rafael J. Wysocki
2008-02-20 23:35 ` Rafael J. Wysocki
2008-02-21 0:00 ` Linus Torvalds
2008-02-21 0:00 ` Linus Torvalds
2008-02-21 0:13 ` Rafael J. Wysocki
2008-02-21 0:13 ` Rafael J. Wysocki
2008-02-21 0:25 ` Linus Torvalds
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-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 ` Rafael J. Wysocki
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:19 ` Jeff Chua
2008-02-21 1:21 ` Jesse Barnes
2008-02-21 1:49 ` Jeff Chua
2008-02-21 1:49 ` Jeff Chua
2008-02-21 2:00 ` Jeff Chua
2008-02-21 2:00 ` Jeff Chua
2008-02-21 16:27 ` [Suspend-devel] " Rafael J. Wysocki
2008-02-21 16:27 ` Rafael J. Wysocki
2008-02-21 18:34 ` Jesse Barnes
2008-02-21 18:34 ` Jesse Barnes
2008-02-21 20:30 ` Rafael J. Wysocki
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-21 23:45 ` Jesse Barnes
2008-02-22 0:28 ` Rafael J. Wysocki
2008-02-22 0:28 ` Rafael J. Wysocki
2008-02-22 0:48 ` Jeff Chua
2008-02-22 0:48 ` Jeff Chua
2008-02-21 22:11 ` Rafael J. Wysocki
2008-02-20 22:32 ` Jesse Barnes
2008-02-20 22:32 ` Jesse Barnes
2008-02-20 23:03 ` Jesse Barnes
2008-02-20 23:03 ` Jesse Barnes
2008-02-20 23:34 ` Jesse Barnes
2008-02-20 23:34 ` Jesse Barnes
2008-02-20 23:49 ` Rafael J. Wysocki
2008-02-20 23:49 ` Rafael J. Wysocki
2008-02-21 0:17 ` Jesse Barnes
2008-02-21 0:17 ` Jesse Barnes
2008-02-21 1:07 ` [Suspend-devel] " Rafael J. Wysocki
2008-02-20 18:47 ` Mark Lord
-- strict thread matches above, loose matches on Subject: below --
2008-02-20 0:53 Jeff Chua
2008-02-20 0:53 ` Jeff Chua
2008-02-20 1:00 ` Jesse Barnes
2008-02-20 1:00 ` Jesse Barnes
2008-02-20 1:06 ` Rafael J. Wysocki
2008-02-20 2:28 ` Linus Torvalds
2008-02-20 2:28 ` Linus Torvalds
2008-02-20 4:32 ` Jesse Barnes
2008-02-20 6:19 ` Jeff Chua
2008-02-20 6:19 ` Jeff Chua
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=200802200950.53591.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=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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.