On 24 July 2012 11:24, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On 07/18/2012 08:15 AM, Paul Berry wrote:<br>
> On Gen6, multisampled null render targets don't seem to work<br>
> properly--they cause the GPU to hang.  So, as a workaround, we render<br>
> into a dummy color buffer.<br>
><br>
> Fortunately this situation (multisampled rendering without a color<br>
> buffer) is rare, and we don't have to waste too much memory, because<br>
> we can give the workaround buffer a very small pitch.<br>
><br>
> Fixes piglit test "EXT_framebuffer_multisample/no-color {2,4}<br>
> depth-computed *" on Gen6.<br>
<br>
<br>
</div><div>> -   surf[0] = (BRW_SURFACE_NULL << BRW_SURFACE_TYPE_SHIFT |<br>
> +   if (fb->Visual.samples > 0) {<br>
> +      /* On Gen6, null render targets seem to cause GPU hangs when<br>
> +       * multisampling.  So work around this problem by rendering into dummy<br>
> +       * color buffer.<br>
> +       *<br>
> +       * To decrease the amount of memory needed by the workaround buffer, we<br>
> +       * set its pitch to 128 bytes (the width of a Y tile).  This means that<br>
> +       * the amount of memory needed for the workaround buffer is<br>
> +       * (width_in_tiles + height_in_tiles - 1) tiles.<br>
<br>
</div>It took me several minutes to see why this was the lower bound of needed memory.<br></blockquote><div><br>Yeah, it's kind of tricky reasoning.  Let me know if you can think of a comment that would have made it easier to follow.<br>
 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
Reviewed-by: Chad Versace <<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>><br>
<br>
I'm off to the dentist now, so I might not be able today to respond to your<br>
comments on my msaa patch series.<br>
</blockquote></div><br>No problem.  Good luck, and thanks again.<br>