<div class="gmail_quote">On Sun, Apr 11, 2010 at 8:12 PM, Jerome Glisse <span dir="ltr">&lt;<a href="mailto:glisse@freedesktop.org" target="_blank">glisse@freedesktop.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div>On Sun, Apr 11, 2010 at 06:39:05AM +0200, Marek Olšák wrote:<br>
&gt; Hi devs,<br>
&gt;<br>
&gt; The first attached patch fixes the calculation of mipmapped 3D texture sizes<br>
&gt; in the CS checker, the 3rd dimension (depth) should be minified too. This<br>
&gt; should probably go to 2.6.34.<br>
&gt;<br>
&gt; The second patch adds 2 new regs:<br>
&gt; - VAP_ALT_NUM_VERTICES, along with an update to the CS checker.<br>
&gt; - VAP_INDEX_OFFSET, I don&#39;t think this one needs to be tracked, because we<br>
&gt; have min/max vertex index clamping, but I might be wrong.<br>
&gt;<br>
<br>
</div>I need to recheck but iirc VAP_INDEX_OFFSET doesn&#39;t count in the<br>
min max clipping ie we need to track it.</blockquote><div><br>GL places index clamping prior to adding the index offset and if hw follows this rule, there is nothing sane to be done. VAP_INDEX_OFFSET is completely orthogonal to VAP_VF_MAX_VTX_INDX. The
latter contains a minimum allowed buffer-object size divided by a
stride, having nothing to do with actual indices. To support index offsets, we would have to check content of every index buffer, which I am not a fan of. Opinions?<br><br>-Marek<br>
</div></div>