<br><br><div class="gmail_quote">On 9 February 2011 09:40, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On Wed, 9 Feb 2011 00:27:52 +0000, Pedro Ribeiro <<a href="mailto:pedrib@gmail.com">pedrib@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> I've been having unreliable operation when hibernating ever since 2.6.37-rc5<br>
> (the first 2.6.37 I tried).<br>
> With 2.6.36 I never had a problem.<br>
<br>
</div>Wrong, you just never saw it.<br>
<br>
commit 9334ef755f060e251f3f395caeda1a58b6834ea3<br>
Author: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>><br>
Date: Fri Jan 28 11:53:03 2011 +0000<br>
<br>
drm: Don't switch fb when disabling an output<br>
<br>
In drm_crtc_helper_set_config, we call drm_crtc_helper_set_mode which<br>
may return early and do no operation if the crtc is to be disabled. In<br>
this case we merrily swap to the new fb, discarding the old_fb believing<br>
that it has been cleaned up. However, due to the early return, the<br>
old_fb was not presented to the backend for correct reaping, and nor was<br>
the new one - which is about to be reaped via the<br>
drm_helper_disable_unused_functions(), leading to incorrect refcounting<br>
of the pinned objects.<br>
<br>
Bugzilla: <a href="https://bugzilla.kernel.org/show_bug.cgi?id=27722" target="_blank">https://bugzilla.kernel.org/show_bug.cgi?id=27722</a><br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=29857" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=29857</a><br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=29230" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=29230</a><br>
Tested-by: Takashi Iwai <<a href="mailto:tiwai@suse.de">tiwai@suse.de</a>><br>
Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>><br>
<font color="#888888"><br>
--<br>
Chris Wilson, Intel Open Source Technology Centre<br>
</font></blockquote></div><br>Thanks for the prompt reply. <br><br>I assume this will be included in stable?<br><br>For the moment I'll just apply the patch to my build.<br><br>Regards,<br>Pedro<br>