<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:jljusten@gmail.com" title="Jordan Justen <jljusten@gmail.com>"> <span class="fn">Jordan Justen</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - [G965] piglit.spec.!opengl 1_2.tex3d-maxsize spins forever"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108630">bug 108630</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;">CC</td>
<td>
</td>
<td>kenneth@whitecape.org
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [G965] piglit.spec.!opengl 1_2.tex3d-maxsize spins forever"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108630#c3">Comment # 3</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [G965] piglit.spec.!opengl 1_2.tex3d-maxsize spins forever"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108630">bug 108630</a>
from <span class="vcard"><a class="email" href="mailto:jljusten@gmail.com" title="Jordan Justen <jljusten@gmail.com>"> <span class="fn">Jordan Justen</span></a>
</span></b>
<pre>Andrii,
I found the issue. It is not specific to g965. In the blorp code, on a
retry, the check_aperture_failed_once variable will be reset back to
false since we clear things before the retry. This small patch added
to your change fixes it.
Another idea could be to preset check_aperture_failed_once before
the retry label.
Can you re-submit the patch with this, or a similar change?
diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c
b/src/mesa/drivers/dri/i965/genX_blorp_exec.c
index fd9ce93c6c..7b0f77af25 100644
--- a/src/mesa/drivers/dri/i965/genX_blorp_exec.c
+++ b/src/mesa/drivers/dri/i965/genX_blorp_exec.c
@@ -309,7 +309,7 @@ retry:
intel_batchbuffer_require_space(brw, 1400);
brw_require_statebuffer_space(brw, 600);
intel_batchbuffer_save_state(brw);
- check_aperture_failed_once = intel_batchbuffer_saved_state_is_empty(brw);
+ check_aperture_failed_once |= intel_batchbuffer_saved_state_is_empty(brw);
brw->batch.no_wrap = true;
#if GEN_GEN == 6</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>