On Thu, Feb 9, 2012 at 11:02 AM, Ian Romanick <span dir="ltr">&lt;<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a>&gt;</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="HOEnZb"><div class="h5">On 02/09/2012 08:14 AM, Brian Paul wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Anuj,<br>
<br>
The patch you committed (15986d2), the first version you posted that tests:<br>
<br>
if (width &lt; 2 * border || width &gt; maxSize)<br>
return GL_FALSE;<br>
<br>
causes my copy of conform (mustpass.c / proxy texture test) to fail with<br>
swrast/softpipe/llvmpipe. It passes w/ NVIDIA&#39;s driver.<br>
<br>
I thought you were going to commit the patch that does:<br>
<br>
if (width &lt; 2 * border || width &gt; 2 * border + maxSize)<br>
return GL_FALSE;<br>
<br>
That allows conform to pass here.<br>
</blockquote>
<br></div></div>
Allow me to remove the egg from my face.  I dug a bit deeper into the test case that was previously failing and conform / mustpass.  What I said before (<a href="http://lists.freedesktop.org/archives/mesa-dev/2012-February/018562.html" target="_blank">http://lists.freedesktop.org/<u></u>archives/mesa-dev/2012-<u></u>February/018562.html</a>) was wrong.  You and Jose were correct.  Sorry.<br>


</blockquote></div><br>Intel oglconform test case (max_values negative.textureSize.textureCube) also passes with the patch i earlier posted as per Brian&#39;s comments:<a href="%20http://lists.freedesktop.org/archives/mesa-dev/2012-January/018410.html"> http://lists.freedesktop.org/archives/mesa-dev/2012-January/018410.html</a><br>

I&#39;ll make the required changes as per this patch.<br>