<div dir="ltr">On 28 August 2013 20:16, 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 class="im">On 08/26/2013 03:12 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Dword 2 of all 3DSTATE commands is the same, so there's no need tohave<br>
</blockquote>
<br></div>
Well, not -all- 3DSTATE commands...just these :)<br>
<br>
It's weird that you decided to share the bits for 3DSTATE_VS, 3DSTATE_GS, and 3DSTATE_WM on SNB, but not GEN7_PS_* for 3DSTATE_PS on IVB.  If you're going to do WM, you might as well do PS too...<div class="im"><br>

<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
separate defines for it.  This will allow us to unify some of the<br>
state setup code between VS and GS.<br>
---<br>
  src/mesa/drivers/dri/i965/brw_<u></u>defines.h   | 30 +++++++++---------------------<br>
  src/mesa/drivers/dri/i965/<u></u>gen6_blorp.cpp  |  2 +-<br>
  src/mesa/drivers/dri/i965/<u></u>gen6_gs_state.c |  6 +++---<br>
  src/mesa/drivers/dri/i965/<u></u>gen6_vs_state.c |  4 ++--<br>
  src/mesa/drivers/dri/i965/<u></u>gen6_wm_state.c |  4 ++--<br>
  src/mesa/drivers/dri/i965/<u></u>gen7_disable.c  |  4 ++--<br>
  src/mesa/drivers/dri/i965/<u></u>gen7_vs_state.c |  4 ++--<br>
  7 files changed, 21 insertions(+), 33 deletions(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/<u></u>brw_defines.h b/src/mesa/drivers/dri/i965/<u></u>brw_defines.h<br>
index ec6c854..d698757 100644<br>
--- a/src/mesa/drivers/dri/i965/<u></u>brw_defines.h<br>
+++ b/src/mesa/drivers/dri/i965/<u></u>brw_defines.h<br>
@@ -1303,14 +1303,16 @@ enum brw_message_target {<br>
<br>
  #define _3DSTATE_SCISSOR_STATE_<u></u>POINTERS               0x780f /* GEN6+ */<br>
<br>
-#define _3DSTATE_VS                            0x7810 /* GEN6+ */<br>
+/* Common to _3DSTATE_{VS,GS} */<br>
</blockquote>
<br></div>
No mention of WM here.  Maybe:<br>
/* Common to 3DSTATE_{VS,GS,PS|WM} */</blockquote><div><br></div><div>Both of these are good points.  Fixed.<br></div></div></div></div>