<div dir="ltr">On 20 August 2013 11:30, Paul Berry <span dir="ltr"><<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.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">From: Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>><br>
<br>
All but two of the piglit GLSL 1.50/uniform_buffer tests work, and<br>
maxuniformblocksize and referenced-by-shader work.<br>
<br>
v2 (Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>>): Account for Ken's recent<br>
binding table re-work.  Use brw->vec4_gs.bind_bo_offset instead of<br>
brw->gs.bind_bo_offset.<br>
---<br>
 src/mesa/drivers/dri/i965/Makefile.sources       |   1 +<br>
 src/mesa/drivers/dri/i965/brw_context.h          |  17 ++-<br>
 src/mesa/drivers/dri/i965/brw_gs_emit.c          |   2 +-<br>
 src/mesa/drivers/dri/i965/brw_gs_surface_state.c | 182 +++++++++++++++++++++++<br>
 src/mesa/drivers/dri/i965/brw_state.h            |   3 +<br>
 src/mesa/drivers/dri/i965/brw_state_upload.c     |   3 +<br>
 src/mesa/drivers/dri/i965/gen6_sol.c             |   6 +-<br>
 7 files changed, 208 insertions(+), 6 deletions(-)<br>
 create mode 100644 src/mesa/drivers/dri/i965/brw_gs_surface_state.c<br></blockquote><div><br></div><div>From our in-person code review yesterday:<br><br></div><div>It became clear that I don't understand this patch as well as I should.  I need to spend more time reviewing it myself.<br>
<br>The ascii art tables in brw_context.h (around line 607, just above SURF_INDEX_DRAW()) should be updated.<br></div></div></div></div>