On 9 July 2012 13:35, 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 class="im">> @@ -192,6 +192,12 @@ struct brw_blorp_blit_prog_key<br>
>      */<br>
>     unsigned tex_samples;<br>
><br>
> +   /* If tex_samples > 0, whether or not the GPU pipeline will be configured<br>
> +    * to read from it as though it were an interleaved MSAA layout.  False if<br>
> +    * tex_samples == 0.<br>
> +    */<br>
> +   bool tex_interleaved;<br>
<br>
</div>There's a confusing sentence fragment here.<br></blockquote><div><br>Oh, wow, that's a pretty bad garden path sentence.  I'll fix it to "If tex_samples > 0, this boolean indicates whether or not the GPU pipeline will be configured...".<br>
 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="im">> +<br>
>     /* Actual number of samples per pixel in the source image. */<br>
>     unsigned src_samples;<br>
><br>
> @@ -205,6 +211,12 @@ struct brw_blorp_blit_prog_key<br>
>      */<br>
>     unsigned rt_samples;<br>
><br>
> +   /* If rt_samples > 0, whether or not the GPU pipeline will be configured<br>
> +    * to write to it as though it were an interleaved MSAA layout.  False if<br>
> +    * rt_samples == 0.<br>
> +    */<br>
> +   bool rt_interleaved;<br>
<br>
</div>Here too.<br>
<br>
But you kill these comments in the next patch, so whatever.<br>
<br>
Reviewed-by: Chad Versace <<a href="mailto:chad.versace@linux.intel.com">chad.versace@linux.intel.com</a>><br>
</blockquote></div><br>