<div dir="ltr">On 22 May 2013 12:21, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 05/21/2013 04:52 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/src/mesa/drivers/dri/i915/i830_vtbl.c b/src/mesa/drivers/dri/i915/i830_vtbl.c<br>
index b30f45e..a35f58b 100644<br>
--- a/src/mesa/drivers/dri/i915/<u></u>i830_vtbl.c<br>
+++ b/src/mesa/drivers/dri/i915/<u></u>i830_vtbl.c<br>
@@ -760,7 +760,13 @@ i830_update_draw_buffer(struct intel_context *intel)<br>
<br>
         for (i = 0; i < fb->_NumColorDrawBuffers; i++) {<br>
             irb = intel_renderbuffer(fb->_<u></u>ColorDrawBuffers[i]);<br>
-           colorRegions[i] = (irb && irb->mt) ? irb->mt->region : NULL;<br>
+           if (irb && irb->mt) {<br>
+              colorRegions[i] =<br>
+                 intel_miptree_get_region(<u></u>intel, irb->mt,<br>
+                                          INTEL_MIPTREE_ACCESS_RENDER);<br>
+           } else {<br>
+              colorRegions[i] = NULL;<br>
+           }<br>
</blockquote>
<br>
This idiom appears several places.  Would it be better to pass irb and let intel_miptree_get_region return NULL?  Or have a wrapper function? Just thinking out loud...<br></blockquote><div><br></div><div>I'm not convinced it would be a benefit.  But considering the discussion I just had with Eric, Ken, and Chad (see my other reply with the same subject line from a few minutes ago), the question is probably moot.<br>
</div></div></div></div>