<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [HSW-M GT3e bisected]Blackscreen after booting with HDMI only"
href="https://bugs.freedesktop.org/show_bug.cgi?id=76151#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [HSW-M GT3e bisected]Blackscreen after booting with HDMI only"
href="https://bugs.freedesktop.org/show_bug.cgi?id=76151">bug 76151</a>
from <span class="vcard"><a class="email" href="mailto:daniel@ffwll.ch" title="Daniel Vetter <daniel@ffwll.ch>"> <span class="fn">Daniel Vetter</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=76151#c5">comment #5</a>)
<span class="quote">> Ugh, that interleaves the lifetime of the pm vs forcewake ref.
>
> reg read: -> deferred forcewake.
> gen6_gt_get: -> pm_get, forcewake++
> gen6_gt_put: -> --forcewake, pm_put
>
> device is now asleep
>
> forcewake_timer: boom, invalid reg access.</span >
You can only do reg I/O if you already have a runtime pm reference. But yeah
I've missed that we don't actually call the full machinery on suspend ...
Both approaches have the ugly issue though that the del_timer_sync will leak
the forcewake reference. So I think we need to call the timer function if we've
actually deactivated it ... fun stuff.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>