<div dir="ltr">On 9 September 2013 16:31, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</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><div>On 09/09/2013 08:20 AM, Paul Berry wrote:<br>
> +const struct brw_tracked_state gen7_gs_state = {<br>
> +   .dirty = {<br>
> +      .mesa  = _NEW_PROGRAM_CONSTANTS,<br>
<br>
</div></div>I'm pretty sure you need _NEW_TRANSFORM here.  The atom earlier in this<br>
patch will upload new push constant data if _NEW_TRANSFORM is set, so<br>
you'll want to emit 3DSTATE_CONSTANT_GS to point to the new constant<br>
data as well.<br>
<br>
Otherwise, this looks good.<br></blockquote><div><br></div><div>Good point.  Fixed, thanks!<br></div></div></div></div>