<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:agd5f@yahoo.com" title="Alex Deucher <agd5f@yahoo.com>"> <span class="fn">Alex Deucher</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [bisected] kernel 3.7.0-rc1 breaks 6950 (CAYMAN)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=56139">bug 56139</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Attachment #68760 is obsolete</td>
<td>
</td>
<td>1
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [bisected] kernel 3.7.0-rc1 breaks 6950 (CAYMAN)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=56139#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [bisected] kernel 3.7.0-rc1 breaks 6950 (CAYMAN)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=56139">bug 56139</a>
from <span class="vcard"><a class="email" href="mailto:agd5f@yahoo.com" title="Alex Deucher <agd5f@yahoo.com>"> <span class="fn">Alex Deucher</span></a>
</span></b>
<pre>Created <span class=""><a href="attachment.cgi?id=69113" name="attach_69113" title="possible fix">attachment 69113</a> <a href="attachment.cgi?id=69113&action=edit" title="possible fix">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=56139&attachment=69113'>[review]</a>
possible fix
(In reply to <a href="show_bug.cgi?id=56139#c4">comment #4</a>)
<span class="quote">> the bug appeared. So it seems blanking the display controllers with for(i =
> 0; i < rdev->num_crtc; i++) is not equivalent to the code that it replaces.
> The original code first wrote in the EVERGREEN_CRTC_UPDATE_LOCK registers,
> before setting EVERGREEN_CRTC_CONTROL registers and writing again in the
> EVERGREEN_CRTC_UPDATE_LOCK registers. On the other hand, the new code
> doesn't write in the EVERGREEN_CRTC_UPDATE_LOCK neither before nor after
> setting EVERGREEN_CRTC_CONTROL.</span >
It should be equivalent. CRTC_UPDATE_LOCK turns off double buffering in the
crtc which makes register updates atomic. The new code waits for the frame
count to increase (the double buffered updates happen at vblank) so it should
be equivalent. That said, it shouldn't hurt to take the lock. Does this patch
help?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>