<div dir="ltr">On 14 August 2013 11:46, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</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"><div class="HOEnZb"><div class="h5">On Wed, Aug 14, 2013 at 10:25 AM, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote:<br>

> On 9 August 2013 19:01, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
>> -   src.brw_surfaceformat = dst.brw_surfaceformat;<br>
>> +   if (src.num_samples > 1)<br>
>> +      src.brw_surfaceformat = dst.brw_surfaceformat;<br>
>> +   else<br>
>> +      dst.brw_surfaceformat = src.brw_surfaceformat;<br>
><br>
><br>
> I thought we decided that it was a bug to ever set src.brw_surfaceformat =<br>
> dst.brw_surfaceformat, and we should do dst.brw_surfaceformat =<br>
> src.brw_surfaceformat always.  Am I remembering wrong?<br>
</div></div><div></div>As suggested by Ian, I modified a piglit test and checked the<br>
behavior on proprietary NVIDIA OpenGL 4.3 drivers. NVIDIA<br>
still uses destination format to decide the type of averaging<br>
during resolve. During our offline discussion you mentioned<br>
how NVIDIA's approach produce slightly better looking images<br>
as compared to using the source buffer format for avearging the<br>
samples. Also, OpenGL specification doesn't seem to be strict<br>
about it. So, I decided to use an if-else to take care of averaging<br>
format in single sample scaled blits. I don't have any strong<br>
opinions about it and I'm willing to post a patch to use<br>
dst.brw_surfaceformat = src.brw_surfaceformat for all cases.<br>
Please let me know if you're in favor of fixing it.<br></blockquote><div><br></div><div>Ah, ok.  In that case I think you made the right decision.  Would you mind putting a comment above the if statement documenting your reasoning?  It's a really subtle decision and I'd hate for someone to come along in the future and try to "fix" it without understanding the history.<br>
<br>Thanks! </div></div></div></div>