<div dir="ltr">On 29 August 2013 15:56, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="im">On 08/26/2013 03:12 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
"ff" is for "fixed function".  This frees up the name "gs" to refer to<br>
user-defined geometry shaders.<br>
---<br>
  src/mesa/drivers/dri/i965/brw_<u></u>context.h      |  14 +--<br>
  src/mesa/drivers/dri/i965/brw_<u></u>gs.c           |  42 ++++----<br>
  src/mesa/drivers/dri/i965/brw_<u></u>gs.h           |  19 ++--<br>
  src/mesa/drivers/dri/i965/brw_<u></u>gs_emit.c      | 150 ++++++++++++++-------------<br>
  src/mesa/drivers/dri/i965/brw_<u></u>gs_state.c     |  17 +--<br>
  src/mesa/drivers/dri/i965/brw_<u></u>misc_state.c   |  10 +-<br>
  src/mesa/drivers/dri/i965/brw_<u></u>state.h        |   2 +-<br>
  src/mesa/drivers/dri/i965/brw_<u></u>state_dump.c   |   4 +-<br>
  src/mesa/drivers/dri/i965/brw_<u></u>state_upload.c |   8 +-<br>
  src/mesa/drivers/dri/i965/<u></u>gen6_gs_state.c    |  10 +-<br>
  src/mesa/drivers/dri/i965/<u></u>gen6_sol.c         |  12 +--<br>
  src/mesa/drivers/dri/i965/<u></u>gen6_urb.c         |   8 +-<br>
  src/mesa/drivers/dri/i965/<u></u>gen7_disable.c     |   2 +-<br>
  src/mesa/drivers/dri/i965/<u></u>gen7_urb.c         |   4 +-<br>
  14 files changed, 156 insertions(+), 146 deletions(-)<br>
</blockquote>
<br>
<br>
<br>
</div><div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
@@ -208,7 +209,7 @@ static void brw_gs_emit_vue(struct brw_gs_compile *c,<br>
   * the allocated URB entry (which will be needed by the URB_WRITE meesage that<br>
   * follows).<br>
   */<br>
-static void brw_gs_ff_sync(struct brw_gs_compile *c, int num_prim)<br>
+static void brw_ff_gs_ff_sync(struct brw_ff_gs_compile *c, int num_prim)<br>
  {<br>
     struct brw_compile *p = &c->func;<br>
<br>
</blockquote>
<br></div>
Looks like you double-edited this hunk. "brw_ff_gs_ff".<br></blockquote><div><br></div><div>Sadly, no.  The first "ff" is the "ff" I'm adding.  The second "ff" is part of "ff_sync", which is the name of a URB message (see Graphics BSpec: 3D-Media-GPGPU Engine > Shared Functions > URB > URB Messages [DevSNB] > FF_SYNC Messages).  It's a rather unfortunate coincidence that makes the name look like a mistake :(<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Fix that, and patch 1 is<br>
Reviewed-by: Chad Versace <<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>><br>
</blockquote></div><br></div></div>