All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ken Moffat <zarniwhoop@ntlworld.com>
To: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Dave Airlie <airlied@redhat.com>,
	linux-kernel@vger.kernel.org,
	Dan Carpenter <dan.carpenter@oracle.com>,
	Alex Deucher <alexdeucher@gmail.com>
Subject: Re: Resume from suspend broken in 3.15. (bisected)
Date: Thu, 29 May 2014 00:49:27 +0100	[thread overview]
Message-ID: <20140528234927.GA18638@milliways> (raw)
In-Reply-To: <20140528172521.GA9945@milliways>

On Wed, May 28, 2014 at 06:25:21PM +0100, Ken Moffat wrote:
> Hi Daniel,
> 

 [ correcting details, confirming that reverting this does fix the
problem, adding Cc:s ]

>  I've only started full testing of 3.15 on one of my machines now
> that -rc7 has been released (this one had two issues in the radeon
> code, second was fixed in rc7).  Unfortunately, suspend to RAM
> (pm-suspend), or rather the wake-up, is broken on this box [ my
> other two boxes are fine in rc7 ].
> 
> Bisection identified one of your commits -
> 
> commit 25f397a429dfa43f22c278d0119a60a343aa568f
> Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> Date:   Fri Jul 19 18:57:11 2013 +0200
> 
>     drm/crtc-helper: explicit DPMS on after modeset
>     
>     Atm the crtc helper implementation of set_config has really
>     inconsisten semantics: If just an fb update is good enough, dpms state
>     will be left as-is, but if we do a full modeset we force everything to
>     dpms on.
>     
>     This change has already been applied to the i915 modeset code in
>     
> ('git show' stops at that point)

 update : I've no idea what was going on there, nor for the problem
with attempting to revert it.  I've now gone back into git,
extracted the full commit to a file with 'git show', and then used
git apply -R to revert it from 3.15-rc7.  That version wakes up from
suspend to RAM, 3.15-rc7 itself did not.

 Maybe I was still in git log when I thought I was on the command
line.  Anyway, snipping git's view of my failed attempt to revert
it, and adding Dan and Alex who were CC'd on the commit.

> 
>  The processor is an AMD A4 APU.  The symptoms after this commit are
> that pm-suspend works (I invoke it from my keyboard's sleep key),
> i.e. the power LED on the case goes out and the monitor goes black
> and reports no signal.  When I press a key to resume, the power LED
> comes on but the screen stays black.  Magic-SysRQ does not work and
> I have to use the case switch to reboot.  That results in filesystem
> errors which fsck fixes and warns about (probably, just a sign of
> unclean shutdown).
> 
>  The only thing in the log are a couple of messages from EXT4 at the
> end of putting the box to sleep -
> May 28 16:19:02 bluesbreaker kernel: [   39.440859] EXT4-fs (sda10): re-mounted. Opts: commit=0
> May 28 16:19:02 bluesbreaker kernel: [   39.592318] EXT4-fs (sda13): re-mounted. Opts: commit=0
> May 28 16:20:05 bluesbreaker syslogd 1.5.0: restart.
> 
>  What can I do to help debug this ?
> 
ĸen
-- 
Nanny Ogg usually went to bed early. After all, she was an old lady.
Sometimes she went to bed as early as 6 a.m.

  reply	other threads:[~2014-05-28 23:49 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-28 17:25 Resume from suspend broken in 3.15. (bisected) Ken Moffat
2014-05-28 23:49 ` Ken Moffat [this message]
2014-05-29  0:26   ` Alex Deucher
2014-05-29  6:03     ` Dan Carpenter
2014-05-29 20:47       ` Alex Deucher
2014-05-29 20:57         ` Daniel Vetter
2014-05-29 23:21           ` Ken Moffat
2014-06-07 12:01 ` Pavel Machek
2014-06-08  9:50   ` Daniel Vetter

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=20140528234927.GA18638@milliways \
    --to=zarniwhoop@ntlworld.com \
    --cc=airlied@redhat.com \
    --cc=alexdeucher@gmail.com \
    --cc=dan.carpenter@oracle.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=linux-kernel@vger.kernel.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.