<div dir="ltr">On 29 August 2013 19:37, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.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="im">On 08/26/2013 03:12 PM, Paul Berry wrote:<br>
<br>
</div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+   /* VS always requires at least 32 URB entries */<br>
</blockquote>
<br></div>
Haswell GT2 and GT3 require at least 64 URB entries.</blockquote><div><br></div><div>Good catch.  Thank you double checking me on that.<br><br></div><div>I'll make a follow-up patch that adds brw->urb.min_vs_entries, populates it correctly at context creation time, and uses it when programming the URB. <br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+   unsigned vs_chunks =<br>
+      ALIGN(32 * vs_entry_size_bytes, chunk_size_bytes) / chunk_size_bytes;<br>
+   unsigned vs_wants =<br>
+      ALIGN(brw->urb.max_vs_entries * vs_entry_size_bytes,<br>
+            chunk_size_bytes) / chunk_size_bytes - vs_chunks;<br>
</blockquote>
</div></div></blockquote></div><br></div></div>