On 11 January 2012 12:23, Eric Anholt <span dir="ltr">&lt;<a href="mailto:eric@anholt.net">eric@anholt.net</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="im">On Wed, 11 Jan 2012 12:06:37 -0800, Paul Berry &lt;<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>&gt; wrote:<br>
&gt; We always access pull constant buffers using the message types &quot;OWord<br>
&gt; Block Read&quot; or &quot;OWord Dual Block Read&quot;.  According to the Sandy Bridge<br>
&gt; PRM, Vol 4 Part 1, pages 214 and 218, when using these messages:<br>
&gt;<br>
&gt;     &quot;the surface pitch is ignored, the surface is treated as a<br>
&gt;     1-dimensional surface.  An element size (pitch) of 16 bytes is<br>
&gt;     used to determine the size of the buffer for out-of-bounds<br>
&gt;     checking if using the surface state model.&quot;<br>
&gt;<br>
&gt; Previously we were setting the pitch for pull constant buffers to the<br>
&gt; size of the whole constant buffer--this made no sense and would have<br>
&gt; led to incorrect behavior if it were not for the fact that the pitch<br>
&gt; is ignored.<br>
&gt;<br>
&gt; For clarity, this patch sets the pitch for pull constant buffers to 16<br>
&gt; bytes, consistent with the hardware&#39;s behavior.<br>
<br>
</div>I&#39;d prefer to see the 15 written as (16 - 1) just to clarify that it&#39;s a<br>
value-minus-one field, similarly to how we do the<br>
OUT_BATCH(_3DSTATE_CONSTANT_PS &lt;&lt; 16 | (7 - 2)); for the length field of<br>
packets.<br>
</blockquote></div><br>Ok, I&#39;ll do that.  Is that a &quot;reviewed-by&quot; with this change?<br>