All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nigel Cunningham <ncunningham@crca.org.au>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Pedro Ribeiro <pedrib@gmail.com>,
	linux-kernel@vger.kernel.org, dri-devel@lists.sourceforge.net
Subject: Re: 2.6.33-rc8 regression on i915: resume from hibernate locks up every 2nd time
Date: Thu, 18 Feb 2010 09:41:31 +1100	[thread overview]
Message-ID: <4B7C709B.2060809@crca.org.au> (raw)
In-Reply-To: <201002172320.57491.rjw@sisk.pl>

Hi Rafael.

Rafael J. Wysocki wrote:
> On Wednesday 17 February 2010, Pedro Ribeiro wrote:
>> 2010/2/17 Rafael J. Wysocki <rjw@sisk.pl>:
>>> On Wednesday 17 February 2010, Rafael J. Wysocki wrote:
>>>> On Wednesday 17 February 2010, Pedro Ribeiro wrote:
>>>>> Hi,
>>>>>
>>>>> commit 84b79f8d2882b0a84330c04839ed4d3cefd2ff77 (drm/i915: Fix crash
>>>>> while aborting hibernation) introduced two new issues which were not
>>>>> present in 2.6.33-rc7:
>>>>>
>>>>> - every second resume from hibernate results in a blank screen
>>>>> - the annoying flash at the end of atomic copy/restore during the
>>>>> hibernate process is back (present in kernels < 2.6.33)
>>>>>
>>>>> The first issue is serious, the second is just an annoyance.
>>>> The second one is an expected price of fixing the aborted hibernation
>>>> regression.
>>>>
>>>> The first one shouldn't happen, though.
>>>>
>>>> I'll see if I can reproduce that locally.
>>> No, I can't.
>>>
>>> Is the driver compiled directly into the kernel or modular?
>> The driver is modular.
>> And sorry, I forgot to tell you I'm using TuxOnIce, it certainly makes
>> a difference.
> 
> It shouldn't in fact, although I'm not sure.

You're right. It shouldn't. I'm using exactly the same driver model
calls in exactly the same order, and not changing driver code at all.

Regards,

Nigel

  reply	other threads:[~2010-02-17 22:39 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-17 18:47 2.6.33-rc8 regression on i915: resume from hibernate locks up every 2nd time Pedro Ribeiro
2010-02-17 20:11 ` Rafael J. Wysocki
2010-02-17 20:52   ` Rafael J. Wysocki
2010-02-17 22:10     ` Pedro Ribeiro
2010-02-17 22:20       ` Rafael J. Wysocki
2010-02-17 22:41         ` Nigel Cunningham [this message]
2010-02-18  0:02         ` Tino Keitel
2010-02-18  0:29         ` Pedro Ribeiro
2010-02-18 22:06           ` [PATCH] i915 / PM: Fix hibernate regression caused by suspend/resume splitting Rafael J. Wysocki
2010-02-18 22:06           ` Rafael J. Wysocki
2010-02-19  6:39             ` Tino Keitel
2010-02-19  6:39             ` Tino Keitel
2010-02-22 15:47             ` Linus Torvalds
2010-02-22 16:32               ` Jesse Barnes
2010-02-22 16:32               ` Jesse Barnes
2010-02-22 16:36               ` Eric Anholt
2010-02-22 16:36               ` Eric Anholt
2010-02-22 15:47             ` Linus Torvalds

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=4B7C709B.2060809@crca.org.au \
    --to=ncunningham@crca.org.au \
    --cc=dri-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pedrib@gmail.com \
    --cc=rjw@sisk.pl \
    /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.