All of lore.kernel.org
 help / color / mirror / Atom feed
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: Tue, 19 Feb 2008 17:00:11 -0800	[thread overview]
Message-ID: <200802191700.12050.jesse.barnes@intel.com> (raw)
In-Reply-To: <b6a2187b0802191653q4dfaa32cl96b9278e97606ef0@mail.gmail.com>

I found the same poweroff issue on my T61.  It turned out to be related to the 
C state code disabling interrupts when it shouldn't iirc.  Booting 
with 'idle=poll' seems to work around the problem.

The "green screen" problem should be fixed (see the DRM git tree for details).

Jesse

On Tuesday, February 19, 2008 4:53 pm Jeff Chua wrote:
> On Feb 16, 2008 5:00 AM, Greg KH <gregkh@suse.de> wrote:
> > > Also, I've tried CONFIG_DETECT_SOFTLOCKUP=n, but this doesn't fix it
> > > either.
> >
> > Ok, this looks to be something else.
> >
> > > Here's the last dmesg after suspend-to-disk and hang there...
> > >
> > > CPU 1 is now offline
> > > SMP alternatives: switching to UP code
> > > PM: Syncing filesystems ... done.
> > > Freezing user space processes ... (elapsed 0.00 seconds) done.
> > > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done.
> > > PM: Shrinking memory...  ^H-^Hdone (0 pages freed)
> > > PM: Freed 0 kbytes in 0.10 seconds (0.00 MB/s)
> > > ACPI: Preparing to enter system sleep state S4
> > > Suspending console(s)
> > >
> > > [ ... it just hangs here ... press power-switch does the job, and
> > > system is able to resume upon powering on ]
> >
> > Wait, this is a suspend-to-disk issue.  Totally different than the "will
> > not power off" issue.
> >
> > Can you start a new thread on this, and add the suspend people to it?
>
> I bisected down this one commit that causes the problem with
> suspend-to-disk on Lenovo X60s (i945 chipset).
>
> commit ba8bbcf6ff4650712f64c0ef61139c73898e2165
> Author: Jesse Barnes <jesse.barnes@intel.com>
> Date:   Thu Nov 22 14:14:14 2007 +1000
>
>     i915: add suspend/resume support
>
>     Add suspend/resume support to the i915 driver.  Moves some of the
>     initialization into the driver load routine, and fixes up places where
> we assumed no dev_private existed in some of the cleanup paths.  This
> allows us to suspend/resume properly even if X isn't running.
>
>     Signed-off-by: Dave Airlie <airlied@linux.ie>
>
>
> There where problem reverting the some i915 files with the latest
> linux git pull, so I copied those i915*.{h,c} prior to this commit,
> and problem went away.
>
>
> Suspend-to-ram, suspend-to-disk all working now.
>
>
> Thanks,
> Jeff.



-------------------------------------------------------------------------
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: 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>,
	"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: Tue, 19 Feb 2008 17:00:11 -0800	[thread overview]
Message-ID: <200802191700.12050.jesse.barnes@intel.com> (raw)
In-Reply-To: <b6a2187b0802191653q4dfaa32cl96b9278e97606ef0@mail.gmail.com>

I found the same poweroff issue on my T61.  It turned out to be related to the 
C state code disabling interrupts when it shouldn't iirc.  Booting 
with 'idle=poll' seems to work around the problem.

The "green screen" problem should be fixed (see the DRM git tree for details).

Jesse

On Tuesday, February 19, 2008 4:53 pm Jeff Chua wrote:
> On Feb 16, 2008 5:00 AM, Greg KH <gregkh@suse.de> wrote:
> > > Also, I've tried CONFIG_DETECT_SOFTLOCKUP=n, but this doesn't fix it
> > > either.
> >
> > Ok, this looks to be something else.
> >
> > > Here's the last dmesg after suspend-to-disk and hang there...
> > >
> > > CPU 1 is now offline
> > > SMP alternatives: switching to UP code
> > > PM: Syncing filesystems ... done.
> > > Freezing user space processes ... (elapsed 0.00 seconds) done.
> > > Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done.
> > > PM: Shrinking memory...  ^H-^Hdone (0 pages freed)
> > > PM: Freed 0 kbytes in 0.10 seconds (0.00 MB/s)
> > > ACPI: Preparing to enter system sleep state S4
> > > Suspending console(s)
> > >
> > > [ ... it just hangs here ... press power-switch does the job, and
> > > system is able to resume upon powering on ]
> >
> > Wait, this is a suspend-to-disk issue.  Totally different than the "will
> > not power off" issue.
> >
> > Can you start a new thread on this, and add the suspend people to it?
>
> I bisected down this one commit that causes the problem with
> suspend-to-disk on Lenovo X60s (i945 chipset).
>
> commit ba8bbcf6ff4650712f64c0ef61139c73898e2165
> Author: Jesse Barnes <jesse.barnes@intel.com>
> Date:   Thu Nov 22 14:14:14 2007 +1000
>
>     i915: add suspend/resume support
>
>     Add suspend/resume support to the i915 driver.  Moves some of the
>     initialization into the driver load routine, and fixes up places where
> we assumed no dev_private existed in some of the cleanup paths.  This
> allows us to suspend/resume properly even if X isn't running.
>
>     Signed-off-by: Dave Airlie <airlied@linux.ie>
>
>
> There where problem reverting the some i915 files with the latest
> linux git pull, so I copied those i915*.{h,c} prior to this commit,
> and problem went away.
>
>
> Suspend-to-ram, suspend-to-disk all working now.
>
>
> Thanks,
> Jeff.



  reply	other threads:[~2008-02-20  1:00 UTC|newest]

Thread overview: 102+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-20  0:53 2.6.25-rc2 System no longer powers off after suspend-to-disk. Screen becomes green Jeff Chua
2008-02-20  0:53 ` Jeff Chua
2008-02-20  1:00 ` Jesse Barnes [this message]
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
  -- strict thread matches above, loose matches on Subject: below --
2008-02-20 17:17 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-20 17:54   ` 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-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
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: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 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-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-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=200802191700.12050.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.