<div dir="ltr">On 5 January 2013 01:07, Chris Forbes <span dir="ltr"><<a href="mailto:chrisf@ijw.co.nz" target="_blank">chrisf@ijw.co.nz</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">




These cover most of the behavior specified by ARB_texture_multisample.<br>
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org" target="_blank">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</blockquote></div><br></div><div class="gmail_extra">This series looks like a great start.  I've sent out some comments on individual patches, mostly concerning places where I think we need to generalize the tests (e.g. to support sample counts other than 4).<br>



<br></div><div class="gmail_extra">In addition, I think we should add the following tests:<br></div><div class="gmail_extra"><br>(1) A test to verify that the sample positions returned by SAMPLE_POSITION actually match those used for rasterization.  For example, the test could loop over all sample numbers, query SAMPLE_POSITION for that sample number, then render a very tiny quad (say, 1/100 of a pixel in size) centered at that SAMPLE_POSITION, and verify using texelFetch() that only the one sample was affected.  I think it would be sufficient for this test to only run when TEXTURE_FIXED_SAMPLE_LOCATIONS is true.<br>


<br></div><div class="gmail_extra">(2) A test to verify that rendering works properly for both very small (e.g 16x16) and large textures.  Rationale: on i965, we normally turn off tiling for very small textures/renderbuffers.  However, for multisampled textures/renderbuffers, tiling is required; we need to make sure the implementation gets this right, otherwise the results of rendering to a small multisampled buffer will be scrambled.<br>

<br></div><div class="gmail_extra">(3) A test to verify that the limits advertised by MAX_COLOR_TEXTURE_SAMPLES, MAX_DEPTH_TEXTURE_SAMPLES, and MAX_INTEGER_SAMPLES are actually respected by the hardware.  (For example, query MAX_COLOR_TEXTURE_SAMPLES, then try to create a multisampled color texture with that number of samples, and verify that it succeeds).<br>

<br></div><div class="gmail_extra">It would be ok with me if you wanted to do these additional tests as a follow-up series, rather than insert them into the existing series.<br><br>Thanks again for working on this, Chris--it seems like we're getting close to some code we can commit to master.<br>


</div></div>