ilo: clean up draw and rectlist state emission
authorChia-I Wu <olvaffe@gmail.com>
Thu, 25 Sep 2014 05:22:19 +0000 (13:22 +0800)
committerChia-I Wu <olvaffe@gmail.com>
Fri, 26 Sep 2014 13:15:55 +0000 (21:15 +0800)
Add these new high-level functions

  ilo_render_get_draw_dynamic_states_len()
  ilo_render_emit_draw_dynamic_states()
  ilo_render_get_rectlist_dynamic_states_len()
  ilo_render_emit_rectlist_dynamic_states()
  ilo_render_get_draw_surface_states_len()
  ilo_render_emit_draw_surface_states()

for draw and rectlist state emission.  They are implemented in the new
ilo_render_dynamic.c and ilo_render_surface.c.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
src/gallium/drivers/ilo/Makefile.sources
src/gallium/drivers/ilo/ilo_render_dynamic.c [new file with mode: 0644]
src/gallium/drivers/ilo/ilo_render_gen.h
src/gallium/drivers/ilo/ilo_render_gen6.c
src/gallium/drivers/ilo/ilo_render_gen7.c
src/gallium/drivers/ilo/ilo_render_surface.c [new file with mode: 0644]

index 845b023..2bb81ba 100644 (file)
@@ -37,9 +37,11 @@ C_SOURCES := \
        ilo_render.c \
        ilo_render.h \
        ilo_render_gen.h \
+       ilo_render_dynamic.c \
        ilo_render_gen6.c \
        ilo_render_gen7.c \
        ilo_render_gen7.h \
+       ilo_render_surface.c \
        ilo_screen.c \
        ilo_screen.h \
        ilo_shader.c \
diff --git a/src/gallium/drivers/ilo/ilo_render_dynamic.c b/src/gallium/drivers/ilo/ilo_render_dynamic.c
new file mode 100644 (file)
index 0000000..6f530ac
--- /dev/null
@@ -0,0 +1,428 @@
+/*
+ * Mesa 3-D graphics library
+ *
+ * Copyright (C) 2012-2014 LunarG, Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included
+ * in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ *
+ * Authors:
+ *    Chia-I Wu <olv@lunarg.com>
+ */
+
+#include "ilo_common.h"
+#include "ilo_blitter.h"
+#include "ilo_builder_3d.h"
+#include "ilo_state.h"
+#include "ilo_render_gen.h"
+#include "ilo_render.h"
+
+#define DIRTY(state) (session->pipe_dirty & ILO_DIRTY_ ## state)
+
+static void
+gen6_emit_draw_dynamic_viewports(struct ilo_render *r,
+                                 const struct ilo_state_vector *vec,
+                                 struct gen6_draw_session *session)
+{
+   ILO_DEV_ASSERT(r->dev, 6, 6);
+
+   /* SF_VIEWPORT, CLIP_VIEWPORT, and CC_VIEWPORT */
+   if (DIRTY(VIEWPORT)) {
+      r->state.CLIP_VIEWPORT = gen6_CLIP_VIEWPORT(r->builder,
+            vec->viewport.cso, vec->viewport.count);
+
+      r->state.SF_VIEWPORT = gen6_SF_VIEWPORT(r->builder,
+            vec->viewport.cso, vec->viewport.count);
+
+      r->state.CC_VIEWPORT = gen6_CC_VIEWPORT(r->builder,
+            vec->viewport.cso, vec->viewport.count);
+
+      session->viewport_changed = true;
+   }
+}
+
+static void
+gen7_emit_draw_dynamic_viewports(struct ilo_render *r,
+                                 const struct ilo_state_vector *vec,
+                                 struct gen6_draw_session *session)
+{
+   ILO_DEV_ASSERT(r->dev, 7, 7.5);
+
+   /* SF_CLIP_VIEWPORT and CC_VIEWPORT */
+   if (DIRTY(VIEWPORT)) {
+      r->state.SF_CLIP_VIEWPORT = gen7_SF_CLIP_VIEWPORT(r->builder,
+            vec->viewport.cso, vec->viewport.count);
+
+      r->state.CC_VIEWPORT = gen6_CC_VIEWPORT(r->builder,
+            vec->viewport.cso, vec->viewport.count);
+
+      session->viewport_changed = true;
+   }
+}
+
+static void
+gen6_emit_draw_dynamic_scissors(struct ilo_render *r,
+                                const struct ilo_state_vector *vec,
+                                struct gen6_draw_session *session)
+{
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   /* SCISSOR_RECT */
+   if (DIRTY(SCISSOR) || DIRTY(VIEWPORT)) {
+      /* there should be as many scissors as there are viewports */
+      r->state.SCISSOR_RECT = gen6_SCISSOR_RECT(r->builder,
+            &vec->scissor, vec->viewport.count);
+
+      session->scissor_changed = true;
+   }
+}
+
+static void
+gen6_emit_draw_dynamic_cc(struct ilo_render *r,
+                          const struct ilo_state_vector *vec,
+                          struct gen6_draw_session *session)
+{
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   /* BLEND_STATE */
+   if (DIRTY(BLEND) || DIRTY(FB) || DIRTY(DSA)) {
+      r->state.BLEND_STATE = gen6_BLEND_STATE(r->builder,
+            vec->blend, &vec->fb, vec->dsa);
+
+      session->blend_changed = true;
+   }
+
+   /* COLOR_CALC_STATE */
+   if (DIRTY(DSA) || DIRTY(STENCIL_REF) || DIRTY(BLEND_COLOR)) {
+      r->state.COLOR_CALC_STATE =
+         gen6_COLOR_CALC_STATE(r->builder, &vec->stencil_ref,
+               vec->dsa->alpha_ref, &vec->blend_color);
+
+      session->cc_changed = true;
+   }
+
+   /* DEPTH_STENCIL_STATE */
+   if (DIRTY(DSA)) {
+      r->state.DEPTH_STENCIL_STATE =
+         gen6_DEPTH_STENCIL_STATE(r->builder, vec->dsa);
+
+      session->dsa_changed = true;
+   }
+}
+
+static void
+gen6_emit_draw_dynamic_samplers(struct ilo_render *r,
+                                const struct ilo_state_vector *vec,
+                                int shader_type,
+                                struct gen6_draw_session *session)
+{
+   const struct ilo_sampler_cso * const *samplers =
+      vec->sampler[shader_type].cso;
+   const struct pipe_sampler_view * const *views =
+      (const struct pipe_sampler_view **) vec->view[shader_type].states;
+   const int num_samplers = vec->sampler[shader_type].count;
+   const int num_views = vec->view[shader_type].count;
+   uint32_t *sampler_state, *border_color_state;
+   bool emit_border_color = false;
+   bool skip = false;
+
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   /* SAMPLER_BORDER_COLOR_STATE and SAMPLER_STATE */
+   switch (shader_type) {
+   case PIPE_SHADER_VERTEX:
+      if (DIRTY(SAMPLER_VS) || DIRTY(VIEW_VS)) {
+         sampler_state = &r->state.vs.SAMPLER_STATE;
+         border_color_state = r->state.vs.SAMPLER_BORDER_COLOR_STATE;
+
+         if (DIRTY(SAMPLER_VS))
+            emit_border_color = true;
+
+         session->sampler_vs_changed = true;
+      } else {
+         skip = true;
+      }
+      break;
+   case PIPE_SHADER_FRAGMENT:
+      if (DIRTY(SAMPLER_FS) || DIRTY(VIEW_FS)) {
+         sampler_state = &r->state.wm.SAMPLER_STATE;
+         border_color_state = r->state.wm.SAMPLER_BORDER_COLOR_STATE;
+
+         if (DIRTY(SAMPLER_FS))
+            emit_border_color = true;
+
+         session->sampler_fs_changed = true;
+      } else {
+         skip = true;
+      }
+      break;
+   default:
+      skip = true;
+      break;
+   }
+
+   if (skip)
+      return;
+
+   if (emit_border_color) {
+      int i;
+
+      for (i = 0; i < num_samplers; i++) {
+         border_color_state[i] = (samplers[i]) ?
+            gen6_SAMPLER_BORDER_COLOR_STATE(r->builder, samplers[i]) : 0;
+      }
+   }
+
+   /* should we take the minimum of num_samplers and num_views? */
+   *sampler_state = gen6_SAMPLER_STATE(r->builder,
+         samplers, views,
+         border_color_state,
+         MIN2(num_samplers, num_views));
+}
+
+static void
+gen6_emit_draw_dynamic_pcb(struct ilo_render *r,
+                           const struct ilo_state_vector *vec,
+                           struct gen6_draw_session *session)
+{
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   /* push constant buffer for VS */
+   if (DIRTY(VS) || DIRTY(CBUF) || DIRTY(CLIP)) {
+      const int cbuf0_size = (vec->vs) ?
+            ilo_shader_get_kernel_param(vec->vs,
+                  ILO_KERNEL_PCB_CBUF0_SIZE) : 0;
+      const int clip_state_size = (vec->vs) ?
+            ilo_shader_get_kernel_param(vec->vs,
+                  ILO_KERNEL_VS_PCB_UCP_SIZE) : 0;
+      const int total_size = cbuf0_size + clip_state_size;
+
+      if (total_size) {
+         void *pcb;
+
+         r->state.vs.PUSH_CONSTANT_BUFFER =
+            gen6_push_constant_buffer(r->builder, total_size, &pcb);
+         r->state.vs.PUSH_CONSTANT_BUFFER_size = total_size;
+
+         if (cbuf0_size) {
+            const struct ilo_cbuf_state *cbuf =
+               &vec->cbuf[PIPE_SHADER_VERTEX];
+
+            if (cbuf0_size <= cbuf->cso[0].user_buffer_size) {
+               memcpy(pcb, cbuf->cso[0].user_buffer, cbuf0_size);
+            } else {
+               memcpy(pcb, cbuf->cso[0].user_buffer,
+                     cbuf->cso[0].user_buffer_size);
+               memset(pcb + cbuf->cso[0].user_buffer_size, 0,
+                     cbuf0_size - cbuf->cso[0].user_buffer_size);
+            }
+
+            pcb += cbuf0_size;
+         }
+
+         if (clip_state_size)
+            memcpy(pcb, &vec->clip, clip_state_size);
+
+         session->pcb_vs_changed = true;
+      } else if (r->state.vs.PUSH_CONSTANT_BUFFER_size) {
+         r->state.vs.PUSH_CONSTANT_BUFFER = 0;
+         r->state.vs.PUSH_CONSTANT_BUFFER_size = 0;
+
+         session->pcb_vs_changed = true;
+      }
+   }
+
+   /* push constant buffer for FS */
+   if (DIRTY(FS) || DIRTY(CBUF)) {
+      const int cbuf0_size = (vec->fs) ?
+         ilo_shader_get_kernel_param(vec->fs, ILO_KERNEL_PCB_CBUF0_SIZE) : 0;
+
+      if (cbuf0_size) {
+         const struct ilo_cbuf_state *cbuf = &vec->cbuf[PIPE_SHADER_FRAGMENT];
+         void *pcb;
+
+         r->state.wm.PUSH_CONSTANT_BUFFER =
+            gen6_push_constant_buffer(r->builder, cbuf0_size, &pcb);
+         r->state.wm.PUSH_CONSTANT_BUFFER_size = cbuf0_size;
+
+         if (cbuf0_size <= cbuf->cso[0].user_buffer_size) {
+            memcpy(pcb, cbuf->cso[0].user_buffer, cbuf0_size);
+         } else {
+            memcpy(pcb, cbuf->cso[0].user_buffer,
+                  cbuf->cso[0].user_buffer_size);
+            memset(pcb + cbuf->cso[0].user_buffer_size, 0,
+                  cbuf0_size - cbuf->cso[0].user_buffer_size);
+         }
+
+         session->pcb_fs_changed = true;
+      } else if (r->state.wm.PUSH_CONSTANT_BUFFER_size) {
+         r->state.wm.PUSH_CONSTANT_BUFFER = 0;
+         r->state.wm.PUSH_CONSTANT_BUFFER_size = 0;
+
+         session->pcb_fs_changed = true;
+      }
+   }
+}
+
+#undef DIRTY
+
+int
+ilo_render_get_draw_dynamic_states_len(const struct ilo_render *render,
+                                       const struct ilo_state_vector *vec)
+{
+   static int static_len;
+   int sh_type, len;
+
+   ILO_DEV_ASSERT(render->dev, 6, 7.5);
+
+   if (!static_len) {
+      /* 64 bytes, or 16 dwords */
+      const int alignment = 64 / 4;
+
+      /* pad first */
+      len = alignment - 1;
+
+      /* CC states */
+      len += align(GEN6_BLEND_STATE__SIZE * ILO_MAX_DRAW_BUFFERS, alignment);
+      len += align(GEN6_DEPTH_STENCIL_STATE__SIZE, alignment);
+      len += align(GEN6_COLOR_CALC_STATE__SIZE, alignment);
+
+      /* viewport arrays */
+      if (ilo_dev_gen(render->dev) >= ILO_GEN(7)) {
+         len += 15 + /* pad first */
+            align(GEN7_SF_CLIP_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 16) +
+            align(GEN6_CC_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
+            align(GEN6_SCISSOR_RECT__SIZE * ILO_MAX_VIEWPORTS, 8);
+      } else {
+         len += 7 + /* pad first */
+            align(GEN6_SF_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
+            align(GEN6_CLIP_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
+            align(GEN6_CC_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
+            align(GEN6_SCISSOR_RECT__SIZE * ILO_MAX_VIEWPORTS, 8);
+      }
+
+      static_len = len;
+   }
+
+   len = static_len;
+
+   for (sh_type = 0; sh_type < PIPE_SHADER_TYPES; sh_type++) {
+      const int alignment = 32 / 4;
+      int num_samplers, pcb_len;
+
+      num_samplers = vec->sampler[sh_type].count;
+      pcb_len = 0;
+
+      switch (sh_type) {
+      case PIPE_SHADER_VERTEX:
+         if (vec->vs) {
+            pcb_len = ilo_shader_get_kernel_param(vec->vs,
+                  ILO_KERNEL_PCB_CBUF0_SIZE);
+            pcb_len += ilo_shader_get_kernel_param(vec->vs,
+                  ILO_KERNEL_VS_PCB_UCP_SIZE);
+         }
+         break;
+      case PIPE_SHADER_GEOMETRY:
+         break;
+      case PIPE_SHADER_FRAGMENT:
+         if (vec->fs) {
+            pcb_len = ilo_shader_get_kernel_param(vec->fs,
+                  ILO_KERNEL_PCB_CBUF0_SIZE);
+         }
+         break;
+      default:
+         break;
+      }
+
+      /* SAMPLER_STATE array and SAMPLER_BORDER_COLORs */
+      if (num_samplers) {
+         len += align(GEN6_SAMPLER_STATE__SIZE * num_samplers, alignment) +
+            align(GEN6_SAMPLER_BORDER_COLOR__SIZE, alignment) * num_samplers;
+      }
+
+      /* PCB */
+      if (pcb_len)
+         len += align(pcb_len, alignment);
+   }
+
+   return len;
+}
+
+void
+ilo_render_emit_draw_dynamic_states(struct ilo_render *render,
+                                    const struct ilo_state_vector *vec,
+                                    struct gen6_draw_session *session)
+{
+   const unsigned dynamic_used = ilo_builder_dynamic_used(render->builder);
+
+   ILO_DEV_ASSERT(render->dev, 6, 7.5);
+
+   if (ilo_dev_gen(render->dev) >= ILO_GEN(7))
+      gen7_emit_draw_dynamic_viewports(render, vec, session);
+   else
+      gen6_emit_draw_dynamic_viewports(render, vec, session);
+
+   gen6_emit_draw_dynamic_cc(render, vec, session);
+   gen6_emit_draw_dynamic_scissors(render, vec, session);
+   gen6_emit_draw_dynamic_pcb(render, vec, session);
+
+   gen6_emit_draw_dynamic_samplers(render, vec,
+         PIPE_SHADER_VERTEX, session);
+   gen6_emit_draw_dynamic_samplers(render, vec,
+         PIPE_SHADER_FRAGMENT, session);
+
+   assert(ilo_builder_dynamic_used(render->builder) <= dynamic_used +
+         ilo_render_get_draw_dynamic_states_len(render, vec));
+}
+
+int
+ilo_render_get_rectlist_dynamic_states_len(const struct ilo_render *render,
+                                           const struct ilo_blitter *blitter)
+{
+   ILO_DEV_ASSERT(render->dev, 6, 7.5);
+
+   return 64;
+}
+
+void
+ilo_render_emit_rectlist_dynamic_states(struct ilo_render *render,
+                                        const struct ilo_blitter *blitter)
+{
+   const unsigned dynamic_used = ilo_builder_dynamic_used(render->builder);
+
+   ILO_DEV_ASSERT(render->dev, 6, 7.5);
+
+   if (blitter->uses & ILO_BLITTER_USE_DSA) {
+      render->state.DEPTH_STENCIL_STATE =
+         gen6_DEPTH_STENCIL_STATE(render->builder, &blitter->dsa);
+   }
+
+   if (blitter->uses & ILO_BLITTER_USE_CC) {
+      render->state.COLOR_CALC_STATE =
+         gen6_COLOR_CALC_STATE(render->builder, &blitter->cc.stencil_ref,
+               blitter->cc.alpha_ref, &blitter->cc.blend_color);
+   }
+
+   if (blitter->uses & ILO_BLITTER_USE_VIEWPORT) {
+      render->state.CC_VIEWPORT =
+         gen6_CC_VIEWPORT(render->builder, &blitter->viewport, 1);
+   }
+
+   assert(ilo_builder_dynamic_used(render->builder) <= dynamic_used +
+         ilo_render_get_rectlist_dynamic_states_len(render, blitter));
+}
index e7aac49..00ae4f7 100644 (file)
 
 #include "ilo_common.h"
 
-struct ilo_query;
+struct ilo_blitter;
 struct ilo_render;
 struct ilo_state_vector;
 
 struct gen6_draw_session {
    uint32_t pipe_dirty;
 
+   /* commands */
    int reduced_prim;
 
    bool prim_changed;
    bool primitive_restart_changed;
 
-   void (*emit_draw_states)(struct ilo_render *render,
-                            const struct ilo_state_vector *ilo,
-                            struct gen6_draw_session *session);
-
    void (*emit_draw_commands)(struct ilo_render *render,
                               const struct ilo_state_vector *ilo,
                               struct gen6_draw_session *session);
 
-   /* indirect states */
-   bool viewport_state_changed;
-   bool cc_state_blend_changed;
-   bool cc_state_dsa_changed;
-   bool cc_state_cc_changed;
-   bool scissor_state_changed;
+   /* dynamic states */
+   bool viewport_changed;
+   bool scissor_changed;
+
+   bool cc_changed;
+   bool dsa_changed;
+   bool blend_changed;
+
+   bool sampler_vs_changed;
+   bool sampler_gs_changed;
+   bool sampler_fs_changed;
+
+   bool pcb_vs_changed;
+   bool pcb_gs_changed;
+   bool pcb_fs_changed;
+
+   /* surface states */
    bool binding_table_vs_changed;
    bool binding_table_gs_changed;
    bool binding_table_fs_changed;
-   bool sampler_state_vs_changed;
-   bool sampler_state_gs_changed;
-   bool sampler_state_fs_changed;
-   bool pcb_state_vs_changed;
-   bool pcb_state_gs_changed;
-   bool pcb_state_fs_changed;
 
    int num_surfaces[PIPE_SHADER_TYPES];
 };
 
-struct gen6_rectlist_session {
-   uint32_t DEPTH_STENCIL_STATE;
-   uint32_t COLOR_CALC_STATE;
-   uint32_t CC_VIEWPORT;
-};
+int
+ilo_render_get_draw_dynamic_states_len(const struct ilo_render *render,
+                                       const struct ilo_state_vector *vec);
+
+void
+ilo_render_emit_draw_dynamic_states(struct ilo_render *render,
+                                    const struct ilo_state_vector *vec,
+                                    struct gen6_draw_session *session);
+
+int
+ilo_render_get_rectlist_dynamic_states_len(const struct ilo_render *render,
+                                           const struct ilo_blitter *blitter);
+
+void
+ilo_render_emit_rectlist_dynamic_states(struct ilo_render *render,
+                                        const struct ilo_blitter *blitter);
+
+int
+ilo_render_get_draw_surface_states_len(const struct ilo_render *render,
+                                       const struct ilo_state_vector *vec);
+
+void
+ilo_render_emit_draw_surface_states(struct ilo_render *render,
+                                    const struct ilo_state_vector *vec,
+                                    struct gen6_draw_session *session);
 
 void
 gen6_wa_pre_pipe_control(struct ilo_render *r, uint32_t dw1);
@@ -139,15 +161,6 @@ gen6_draw_wm_raster(struct ilo_render *r,
                     struct gen6_draw_session *session);
 
 void
-gen6_draw_states(struct ilo_render *r,
-                 const struct ilo_state_vector *ilo,
-                 struct gen6_draw_session *session);
-
-int
-gen6_render_estimate_state_size(const struct ilo_render *render,
-                                const struct ilo_state_vector *ilo);
-
-void
 ilo_render_init_gen6(struct ilo_render *render);
 
 #endif /* ILO_RENDER_GEN_H */
index 66f97fd..d080975 100644 (file)
@@ -276,25 +276,25 @@ gen6_draw_common_base_address(struct ilo_render *r,
        * 3DSTATE_SCISSOR_STATE_POINTERS is not on the list, but it is
        * reasonable to also reissue the command.  Same to PCB.
        */
-      session->viewport_state_changed = true;
+      session->viewport_changed = true;
 
-      session->cc_state_blend_changed = true;
-      session->cc_state_dsa_changed = true;
-      session->cc_state_cc_changed = true;
+      session->scissor_changed = true;
 
-      session->scissor_state_changed = true;
+      session->blend_changed = true;
+      session->dsa_changed = true;
+      session->cc_changed = true;
+
+      session->sampler_vs_changed = true;
+      session->sampler_gs_changed = true;
+      session->sampler_fs_changed = true;
+
+      session->pcb_vs_changed = true;
+      session->pcb_gs_changed = true;
+      session->pcb_fs_changed = true;
 
       session->binding_table_vs_changed = true;
       session->binding_table_gs_changed = true;
       session->binding_table_fs_changed = true;
-
-      session->sampler_state_vs_changed = true;
-      session->sampler_state_gs_changed = true;
-      session->sampler_state_fs_changed = true;
-
-      session->pcb_state_vs_changed = true;
-      session->pcb_state_gs_changed = true;
-      session->pcb_state_fs_changed = true;
    }
 }
 
@@ -378,7 +378,7 @@ gen6_draw_common_pointers_1(struct ilo_render *r,
                             struct gen6_draw_session *session)
 {
    /* 3DSTATE_VIEWPORT_STATE_POINTERS */
-   if (session->viewport_state_changed) {
+   if (session->viewport_changed) {
       gen6_3DSTATE_VIEWPORT_STATE_POINTERS(r->builder,
             r->state.CLIP_VIEWPORT,
             r->state.SF_VIEWPORT,
@@ -392,9 +392,9 @@ gen6_draw_common_pointers_2(struct ilo_render *r,
                             struct gen6_draw_session *session)
 {
    /* 3DSTATE_CC_STATE_POINTERS */
-   if (session->cc_state_blend_changed ||
-       session->cc_state_dsa_changed ||
-       session->cc_state_cc_changed) {
+   if (session->blend_changed ||
+       session->dsa_changed ||
+       session->cc_changed) {
       gen6_3DSTATE_CC_STATE_POINTERS(r->builder,
             r->state.BLEND_STATE,
             r->state.DEPTH_STENCIL_STATE,
@@ -402,9 +402,9 @@ gen6_draw_common_pointers_2(struct ilo_render *r,
    }
 
    /* 3DSTATE_SAMPLER_STATE_POINTERS */
-   if (session->sampler_state_vs_changed ||
-       session->sampler_state_gs_changed ||
-       session->sampler_state_fs_changed) {
+   if (session->sampler_vs_changed ||
+       session->sampler_gs_changed ||
+       session->sampler_fs_changed) {
       gen6_3DSTATE_SAMPLER_STATE_POINTERS(r->builder,
             r->state.vs.SAMPLER_STATE,
             0,
@@ -418,7 +418,7 @@ gen6_draw_common_pointers_3(struct ilo_render *r,
                             struct gen6_draw_session *session)
 {
    /* 3DSTATE_SCISSOR_STATE_POINTERS */
-   if (session->scissor_state_changed) {
+   if (session->scissor_changed) {
       gen6_3DSTATE_SCISSOR_STATE_POINTERS(r->builder,
             r->state.SCISSOR_RECT);
    }
@@ -522,7 +522,7 @@ gen6_draw_vs(struct ilo_render *r,
 {
    const bool emit_3dstate_vs = (DIRTY(VS) || DIRTY(SAMPLER_VS) ||
                                  r->instruction_bo_changed);
-   const bool emit_3dstate_constant_vs = session->pcb_state_vs_changed;
+   const bool emit_3dstate_constant_vs = session->pcb_vs_changed;
 
    /*
     * the classic i965 does this in upload_vs_state(), citing a spec that I
@@ -556,7 +556,7 @@ gen6_draw_gs(struct ilo_render *r,
              struct gen6_draw_session *session)
 {
    /* 3DSTATE_CONSTANT_GS */
-   if (session->pcb_state_gs_changed)
+   if (session->pcb_gs_changed)
       gen6_3DSTATE_CONSTANT_GS(r->builder, NULL, NULL, 0);
 
    /* 3DSTATE_GS */
@@ -708,7 +708,7 @@ gen6_draw_wm(struct ilo_render *r,
              struct gen6_draw_session *session)
 {
    /* 3DSTATE_CONSTANT_PS */
-   if (session->pcb_state_fs_changed) {
+   if (session->pcb_fs_changed) {
       gen6_3DSTATE_CONSTANT_PS(r->builder,
             &r->state.wm.PUSH_CONSTANT_BUFFER,
             &r->state.wm.PUSH_CONSTANT_BUFFER_size,
@@ -832,504 +832,6 @@ gen6_draw_wm_raster(struct ilo_render *r,
    }
 }
 
-static void
-gen6_draw_state_viewports(struct ilo_render *r,
-                          const struct ilo_state_vector *vec,
-                          struct gen6_draw_session *session)
-{
-   /* SF_CLIP_VIEWPORT and CC_VIEWPORT */
-   if (ilo_dev_gen(r->dev) >= ILO_GEN(7) && DIRTY(VIEWPORT)) {
-      r->state.SF_CLIP_VIEWPORT = gen7_SF_CLIP_VIEWPORT(r->builder,
-            vec->viewport.cso, vec->viewport.count);
-
-      r->state.CC_VIEWPORT = gen6_CC_VIEWPORT(r->builder,
-            vec->viewport.cso, vec->viewport.count);
-
-      session->viewport_state_changed = true;
-   }
-   /* SF_VIEWPORT, CLIP_VIEWPORT, and CC_VIEWPORT */
-   else if (DIRTY(VIEWPORT)) {
-      r->state.CLIP_VIEWPORT = gen6_CLIP_VIEWPORT(r->builder,
-            vec->viewport.cso, vec->viewport.count);
-
-      r->state.SF_VIEWPORT = gen6_SF_VIEWPORT(r->builder,
-            vec->viewport.cso, vec->viewport.count);
-
-      r->state.CC_VIEWPORT = gen6_CC_VIEWPORT(r->builder,
-            vec->viewport.cso, vec->viewport.count);
-
-      session->viewport_state_changed = true;
-   }
-}
-
-static void
-gen6_draw_state_cc(struct ilo_render *r,
-                   const struct ilo_state_vector *vec,
-                   struct gen6_draw_session *session)
-{
-   /* BLEND_STATE */
-   if (DIRTY(BLEND) || DIRTY(FB) || DIRTY(DSA)) {
-      r->state.BLEND_STATE = gen6_BLEND_STATE(r->builder,
-            vec->blend, &vec->fb, vec->dsa);
-
-      session->cc_state_blend_changed = true;
-   }
-
-   /* COLOR_CALC_STATE */
-   if (DIRTY(DSA) || DIRTY(STENCIL_REF) || DIRTY(BLEND_COLOR)) {
-      r->state.COLOR_CALC_STATE =
-         gen6_COLOR_CALC_STATE(r->builder, &vec->stencil_ref,
-               vec->dsa->alpha_ref, &vec->blend_color);
-
-      session->cc_state_cc_changed = true;
-   }
-
-   /* DEPTH_STENCIL_STATE */
-   if (DIRTY(DSA)) {
-      r->state.DEPTH_STENCIL_STATE =
-         gen6_DEPTH_STENCIL_STATE(r->builder, vec->dsa);
-
-      session->cc_state_dsa_changed = true;
-   }
-}
-
-static void
-gen6_draw_state_scissors(struct ilo_render *r,
-                         const struct ilo_state_vector *vec,
-                         struct gen6_draw_session *session)
-{
-   /* SCISSOR_RECT */
-   if (DIRTY(SCISSOR) || DIRTY(VIEWPORT)) {
-      /* there should be as many scissors as there are viewports */
-      r->state.SCISSOR_RECT = gen6_SCISSOR_RECT(r->builder,
-            &vec->scissor, vec->viewport.count);
-
-      session->scissor_state_changed = true;
-   }
-}
-
-static void
-gen6_draw_state_surfaces_rt(struct ilo_render *r,
-                            const struct ilo_state_vector *vec,
-                            struct gen6_draw_session *session)
-{
-   /* SURFACE_STATEs for render targets */
-   if (DIRTY(FB)) {
-      const struct ilo_fb_state *fb = &vec->fb;
-      const int offset = ILO_WM_DRAW_SURFACE(0);
-      uint32_t *surface_state = &r->state.wm.SURFACE_STATE[offset];
-      int i;
-
-      for (i = 0; i < fb->state.nr_cbufs; i++) {
-         const struct ilo_surface_cso *surface =
-            (const struct ilo_surface_cso *) fb->state.cbufs[i];
-
-         if (!surface) {
-            surface_state[i] =
-               gen6_SURFACE_STATE(r->builder, &fb->null_rt, true);
-         }
-         else {
-            assert(surface && surface->is_rt);
-            surface_state[i] =
-               gen6_SURFACE_STATE(r->builder, &surface->u.rt, true);
-         }
-      }
-
-      /*
-       * Upload at least one render target, as
-       * brw_update_renderbuffer_surfaces() does.  I don't know why.
-       */
-      if (i == 0) {
-         surface_state[i] =
-            gen6_SURFACE_STATE(r->builder, &fb->null_rt, true);
-
-         i++;
-      }
-
-      memset(&surface_state[i], 0, (ILO_MAX_DRAW_BUFFERS - i) * 4);
-
-      if (i && session->num_surfaces[PIPE_SHADER_FRAGMENT] < offset + i)
-         session->num_surfaces[PIPE_SHADER_FRAGMENT] = offset + i;
-
-      session->binding_table_fs_changed = true;
-   }
-}
-
-static void
-gen6_draw_state_surfaces_so(struct ilo_render *r,
-                            const struct ilo_state_vector *vec,
-                            struct gen6_draw_session *session)
-{
-   const struct ilo_so_state *so = &vec->so;
-
-   if (ilo_dev_gen(r->dev) != ILO_GEN(6))
-      return;
-
-   /* SURFACE_STATEs for stream output targets */
-   if (DIRTY(VS) || DIRTY(GS) || DIRTY(SO)) {
-      const struct pipe_stream_output_info *so_info =
-         (vec->gs) ? ilo_shader_get_kernel_so_info(vec->gs) :
-         (vec->vs) ? ilo_shader_get_kernel_so_info(vec->vs) : NULL;
-      const int offset = ILO_GS_SO_SURFACE(0);
-      uint32_t *surface_state = &r->state.gs.SURFACE_STATE[offset];
-      int i;
-
-      for (i = 0; so_info && i < so_info->num_outputs; i++) {
-         const int target = so_info->output[i].output_buffer;
-         const struct pipe_stream_output_target *so_target =
-            (target < so->count) ? so->states[target] : NULL;
-
-         if (so_target) {
-            surface_state[i] = gen6_so_SURFACE_STATE(r->builder,
-                  so_target, so_info, i);
-         }
-         else {
-            surface_state[i] = 0;
-         }
-      }
-
-      memset(&surface_state[i], 0, (ILO_MAX_SO_BINDINGS - i) * 4);
-
-      if (i && session->num_surfaces[PIPE_SHADER_GEOMETRY] < offset + i)
-         session->num_surfaces[PIPE_SHADER_GEOMETRY] = offset + i;
-
-      session->binding_table_gs_changed = true;
-   }
-}
-
-static void
-gen6_draw_state_surfaces_view(struct ilo_render *r,
-                              const struct ilo_state_vector *vec,
-                              int shader_type,
-                              struct gen6_draw_session *session)
-{
-   const struct ilo_view_state *view = &vec->view[shader_type];
-   uint32_t *surface_state;
-   int offset, i;
-   bool skip = false;
-
-   /* SURFACE_STATEs for sampler views */
-   switch (shader_type) {
-   case PIPE_SHADER_VERTEX:
-      if (DIRTY(VIEW_VS)) {
-         offset = ILO_VS_TEXTURE_SURFACE(0);
-         surface_state = &r->state.vs.SURFACE_STATE[offset];
-
-         session->binding_table_vs_changed = true;
-      }
-      else {
-         skip = true;
-      }
-      break;
-   case PIPE_SHADER_FRAGMENT:
-      if (DIRTY(VIEW_FS)) {
-         offset = ILO_WM_TEXTURE_SURFACE(0);
-         surface_state = &r->state.wm.SURFACE_STATE[offset];
-
-         session->binding_table_fs_changed = true;
-      }
-      else {
-         skip = true;
-      }
-      break;
-   default:
-      skip = true;
-      break;
-   }
-
-   if (skip)
-      return;
-
-   for (i = 0; i < view->count; i++) {
-      if (view->states[i]) {
-         const struct ilo_view_cso *cso =
-            (const struct ilo_view_cso *) view->states[i];
-
-         surface_state[i] =
-            gen6_SURFACE_STATE(r->builder, &cso->surface, false);
-      }
-      else {
-         surface_state[i] = 0;
-      }
-   }
-
-   memset(&surface_state[i], 0, (ILO_MAX_SAMPLER_VIEWS - i) * 4);
-
-   if (i && session->num_surfaces[shader_type] < offset + i)
-      session->num_surfaces[shader_type] = offset + i;
-}
-
-static void
-gen6_draw_state_surfaces_const(struct ilo_render *r,
-                               const struct ilo_state_vector *vec,
-                               int shader_type,
-                               struct gen6_draw_session *session)
-{
-   const struct ilo_cbuf_state *cbuf = &vec->cbuf[shader_type];
-   uint32_t *surface_state;
-   bool *binding_table_changed;
-   int offset, count, i;
-
-   if (!DIRTY(CBUF))
-      return;
-
-   /* SURFACE_STATEs for constant buffers */
-   switch (shader_type) {
-   case PIPE_SHADER_VERTEX:
-      offset = ILO_VS_CONST_SURFACE(0);
-      surface_state = &r->state.vs.SURFACE_STATE[offset];
-      binding_table_changed = &session->binding_table_vs_changed;
-      break;
-   case PIPE_SHADER_FRAGMENT:
-      offset = ILO_WM_CONST_SURFACE(0);
-      surface_state = &r->state.wm.SURFACE_STATE[offset];
-      binding_table_changed = &session->binding_table_fs_changed;
-      break;
-   default:
-      return;
-      break;
-   }
-
-   /* constants are pushed via PCB */
-   if (cbuf->enabled_mask == 0x1 && !cbuf->cso[0].resource) {
-      memset(surface_state, 0, ILO_MAX_CONST_BUFFERS * 4);
-      return;
-   }
-
-   count = util_last_bit(cbuf->enabled_mask);
-   for (i = 0; i < count; i++) {
-      if (cbuf->cso[i].resource) {
-         surface_state[i] = gen6_SURFACE_STATE(r->builder,
-               &cbuf->cso[i].surface, false);
-      }
-      else {
-         surface_state[i] = 0;
-      }
-   }
-
-   memset(&surface_state[count], 0, (ILO_MAX_CONST_BUFFERS - count) * 4);
-
-   if (count && session->num_surfaces[shader_type] < offset + count)
-      session->num_surfaces[shader_type] = offset + count;
-
-   *binding_table_changed = true;
-}
-
-static void
-gen6_draw_state_binding_tables(struct ilo_render *r,
-                               const struct ilo_state_vector *vec,
-                               int shader_type,
-                               struct gen6_draw_session *session)
-{
-   uint32_t *binding_table_state, *surface_state;
-   int *binding_table_state_size, size;
-   bool skip = false;
-
-   /* BINDING_TABLE_STATE */
-   switch (shader_type) {
-   case PIPE_SHADER_VERTEX:
-      surface_state = r->state.vs.SURFACE_STATE;
-      binding_table_state = &r->state.vs.BINDING_TABLE_STATE;
-      binding_table_state_size = &r->state.vs.BINDING_TABLE_STATE_size;
-
-      skip = !session->binding_table_vs_changed;
-      break;
-   case PIPE_SHADER_GEOMETRY:
-      surface_state = r->state.gs.SURFACE_STATE;
-      binding_table_state = &r->state.gs.BINDING_TABLE_STATE;
-      binding_table_state_size = &r->state.gs.BINDING_TABLE_STATE_size;
-
-      skip = !session->binding_table_gs_changed;
-      break;
-   case PIPE_SHADER_FRAGMENT:
-      surface_state = r->state.wm.SURFACE_STATE;
-      binding_table_state = &r->state.wm.BINDING_TABLE_STATE;
-      binding_table_state_size = &r->state.wm.BINDING_TABLE_STATE_size;
-
-      skip = !session->binding_table_fs_changed;
-      break;
-   default:
-      skip = true;
-      break;
-   }
-
-   if (skip)
-      return;
-
-   /*
-    * If we have seemingly less SURFACE_STATEs than before, it could be that
-    * we did not touch those reside at the tail in this upload.  Loop over
-    * them to figure out the real number of SURFACE_STATEs.
-    */
-   for (size = *binding_table_state_size;
-         size > session->num_surfaces[shader_type]; size--) {
-      if (surface_state[size - 1])
-         break;
-   }
-   if (size < session->num_surfaces[shader_type])
-      size = session->num_surfaces[shader_type];
-
-   *binding_table_state = gen6_BINDING_TABLE_STATE(r->builder,
-         surface_state, size);
-   *binding_table_state_size = size;
-}
-
-static void
-gen6_draw_state_samplers(struct ilo_render *r,
-                         const struct ilo_state_vector *vec,
-                         int shader_type,
-                         struct gen6_draw_session *session)
-{
-   const struct ilo_sampler_cso * const *samplers =
-      vec->sampler[shader_type].cso;
-   const struct pipe_sampler_view * const *views =
-      (const struct pipe_sampler_view **) vec->view[shader_type].states;
-   const int num_samplers = vec->sampler[shader_type].count;
-   const int num_views = vec->view[shader_type].count;
-   uint32_t *sampler_state, *border_color_state;
-   bool emit_border_color = false;
-   bool skip = false;
-
-   /* SAMPLER_BORDER_COLOR_STATE and SAMPLER_STATE */
-   switch (shader_type) {
-   case PIPE_SHADER_VERTEX:
-      if (DIRTY(SAMPLER_VS) || DIRTY(VIEW_VS)) {
-         sampler_state = &r->state.vs.SAMPLER_STATE;
-         border_color_state = r->state.vs.SAMPLER_BORDER_COLOR_STATE;
-
-         if (DIRTY(SAMPLER_VS))
-            emit_border_color = true;
-
-         session->sampler_state_vs_changed = true;
-      }
-      else {
-         skip = true;
-      }
-      break;
-   case PIPE_SHADER_FRAGMENT:
-      if (DIRTY(SAMPLER_FS) || DIRTY(VIEW_FS)) {
-         sampler_state = &r->state.wm.SAMPLER_STATE;
-         border_color_state = r->state.wm.SAMPLER_BORDER_COLOR_STATE;
-
-         if (DIRTY(SAMPLER_FS))
-            emit_border_color = true;
-
-         session->sampler_state_fs_changed = true;
-      }
-      else {
-         skip = true;
-      }
-      break;
-   default:
-      skip = true;
-      break;
-   }
-
-   if (skip)
-      return;
-
-   if (emit_border_color) {
-      int i;
-
-      for (i = 0; i < num_samplers; i++) {
-         border_color_state[i] = (samplers[i]) ?
-            gen6_SAMPLER_BORDER_COLOR_STATE(r->builder, samplers[i]) : 0;
-      }
-   }
-
-   /* should we take the minimum of num_samplers and num_views? */
-   *sampler_state = gen6_SAMPLER_STATE(r->builder,
-         samplers, views,
-         border_color_state,
-         MIN2(num_samplers, num_views));
-}
-
-static void
-gen6_draw_state_pcb(struct ilo_render *r,
-                    const struct ilo_state_vector *vec,
-                    struct gen6_draw_session *session)
-{
-   /* push constant buffer for VS */
-   if (DIRTY(VS) || DIRTY(CBUF) || DIRTY(CLIP)) {
-      const int cbuf0_size = (vec->vs) ?
-            ilo_shader_get_kernel_param(vec->vs,
-                  ILO_KERNEL_PCB_CBUF0_SIZE) : 0;
-      const int clip_state_size = (vec->vs) ?
-            ilo_shader_get_kernel_param(vec->vs,
-                  ILO_KERNEL_VS_PCB_UCP_SIZE) : 0;
-      const int total_size = cbuf0_size + clip_state_size;
-
-      if (total_size) {
-         void *pcb;
-
-         r->state.vs.PUSH_CONSTANT_BUFFER =
-            gen6_push_constant_buffer(r->builder, total_size, &pcb);
-         r->state.vs.PUSH_CONSTANT_BUFFER_size = total_size;
-
-         if (cbuf0_size) {
-            const struct ilo_cbuf_state *cbuf =
-               &vec->cbuf[PIPE_SHADER_VERTEX];
-
-            if (cbuf0_size <= cbuf->cso[0].user_buffer_size) {
-               memcpy(pcb, cbuf->cso[0].user_buffer, cbuf0_size);
-            }
-            else {
-               memcpy(pcb, cbuf->cso[0].user_buffer,
-                     cbuf->cso[0].user_buffer_size);
-               memset(pcb + cbuf->cso[0].user_buffer_size, 0,
-                     cbuf0_size - cbuf->cso[0].user_buffer_size);
-            }
-
-            pcb += cbuf0_size;
-         }
-
-         if (clip_state_size)
-            memcpy(pcb, &vec->clip, clip_state_size);
-
-         session->pcb_state_vs_changed = true;
-      }
-      else if (r->state.vs.PUSH_CONSTANT_BUFFER_size) {
-         r->state.vs.PUSH_CONSTANT_BUFFER = 0;
-         r->state.vs.PUSH_CONSTANT_BUFFER_size = 0;
-
-         session->pcb_state_vs_changed = true;
-      }
-   }
-
-   /* push constant buffer for FS */
-   if (DIRTY(FS) || DIRTY(CBUF)) {
-      const int cbuf0_size = (vec->fs) ?
-         ilo_shader_get_kernel_param(vec->fs, ILO_KERNEL_PCB_CBUF0_SIZE) : 0;
-
-      if (cbuf0_size) {
-         const struct ilo_cbuf_state *cbuf = &vec->cbuf[PIPE_SHADER_FRAGMENT];
-         void *pcb;
-
-         r->state.wm.PUSH_CONSTANT_BUFFER =
-            gen6_push_constant_buffer(r->builder, cbuf0_size, &pcb);
-         r->state.wm.PUSH_CONSTANT_BUFFER_size = cbuf0_size;
-
-         if (cbuf0_size <= cbuf->cso[0].user_buffer_size) {
-            memcpy(pcb, cbuf->cso[0].user_buffer, cbuf0_size);
-         }
-         else {
-            memcpy(pcb, cbuf->cso[0].user_buffer,
-                  cbuf->cso[0].user_buffer_size);
-            memset(pcb + cbuf->cso[0].user_buffer_size, 0,
-                  cbuf0_size - cbuf->cso[0].user_buffer_size);
-         }
-
-         session->pcb_state_fs_changed = true;
-      }
-      else if (r->state.wm.PUSH_CONSTANT_BUFFER_size) {
-         r->state.wm.PUSH_CONSTANT_BUFFER = 0;
-         r->state.wm.PUSH_CONSTANT_BUFFER_size = 0;
-
-         session->pcb_state_fs_changed = true;
-      }
-   }
-}
-
 #undef DIRTY
 
 static void
@@ -1365,36 +867,6 @@ gen6_draw_commands(struct ilo_render *render,
 }
 
 void
-gen6_draw_states(struct ilo_render *render,
-                 const struct ilo_state_vector *vec,
-                 struct gen6_draw_session *session)
-{
-   int shader_type;
-
-   gen6_draw_state_viewports(render, vec, session);
-   gen6_draw_state_cc(render, vec, session);
-   gen6_draw_state_scissors(render, vec, session);
-   gen6_draw_state_pcb(render, vec, session);
-
-   /*
-    * upload all SURAFCE_STATEs together so that we know there are minimal
-    * paddings
-    */
-   gen6_draw_state_surfaces_rt(render, vec, session);
-   gen6_draw_state_surfaces_so(render, vec, session);
-   for (shader_type = 0; shader_type < PIPE_SHADER_TYPES; shader_type++) {
-      gen6_draw_state_surfaces_view(render, vec, shader_type, session);
-      gen6_draw_state_surfaces_const(render, vec, shader_type, session);
-   }
-
-   for (shader_type = 0; shader_type < PIPE_SHADER_TYPES; shader_type++) {
-      gen6_draw_state_samplers(render, vec, shader_type, session);
-      /* this must be called after all SURFACE_STATEs are uploaded */
-      gen6_draw_state_binding_tables(render, vec, shader_type, session);
-   }
-}
-
-void
 gen6_draw_prepare(struct ilo_render *render,
                   const struct ilo_state_vector *vec,
                   struct gen6_draw_session *session)
@@ -1430,7 +902,8 @@ gen6_draw_emit(struct ilo_render *render,
    else
       session->pipe_dirty = vec->dirty;
 
-   session->emit_draw_states(render, vec, session);
+   ilo_render_emit_draw_dynamic_states(render, vec, session);
+   ilo_render_emit_draw_surface_states(render, vec, session);
 
    /* force all commands to be uploaded if the HW context changed */
    if (render->hw_ctx_changed)
@@ -1464,7 +937,6 @@ ilo_render_emit_draw_gen6(struct ilo_render *render,
 
    gen6_draw_prepare(render, vec, &session);
 
-   session.emit_draw_states = gen6_draw_states;
    session.emit_draw_commands = gen6_draw_commands;
 
    gen6_draw_emit(render, vec, &session);
@@ -1473,8 +945,7 @@ ilo_render_emit_draw_gen6(struct ilo_render *render,
 
 static void
 gen6_rectlist_vs_to_sf(struct ilo_render *r,
-                       const struct ilo_blitter *blitter,
-                       struct gen6_rectlist_session *session)
+                       const struct ilo_blitter *blitter)
 {
    gen6_3DSTATE_CONSTANT_VS(r->builder, NULL, NULL, 0);
    gen6_3DSTATE_VS(r->builder, NULL, 0);
@@ -1490,8 +961,7 @@ gen6_rectlist_vs_to_sf(struct ilo_render *r,
 
 static void
 gen6_rectlist_wm(struct ilo_render *r,
-                 const struct ilo_blitter *blitter,
-                 struct gen6_rectlist_session *session)
+                 const struct ilo_blitter *blitter)
 {
    uint32_t hiz_op;
 
@@ -1518,8 +988,7 @@ gen6_rectlist_wm(struct ilo_render *r,
 
 static void
 gen6_rectlist_wm_depth(struct ilo_render *r,
-                       const struct ilo_blitter *blitter,
-                       struct gen6_rectlist_session *session)
+                       const struct ilo_blitter *blitter)
 {
    gen6_wa_pre_depth(r);
 
@@ -1545,8 +1014,7 @@ gen6_rectlist_wm_depth(struct ilo_render *r,
 
 static void
 gen6_rectlist_wm_multisample(struct ilo_render *r,
-                             const struct ilo_blitter *blitter,
-                             struct gen6_rectlist_session *session)
+                             const struct ilo_blitter *blitter)
 {
    const uint32_t *packed_sample_pos = (blitter->fb.num_samples > 1) ?
       &r->packed_sample_position_4x : &r->packed_sample_position_1x;
@@ -1562,12 +1030,11 @@ gen6_rectlist_wm_multisample(struct ilo_render *r,
 
 static void
 gen6_rectlist_commands(struct ilo_render *r,
-                       const struct ilo_blitter *blitter,
-                       struct gen6_rectlist_session *session)
+                       const struct ilo_blitter *blitter)
 {
    gen6_wa_pre_non_pipelined(r);
 
-   gen6_rectlist_wm_multisample(r, blitter, session);
+   gen6_rectlist_wm_multisample(r, blitter);
 
    gen6_state_base_address(r->builder, true);
 
@@ -1588,18 +1055,18 @@ gen6_rectlist_commands(struct ilo_render *r,
    if (blitter->uses &
        (ILO_BLITTER_USE_DSA | ILO_BLITTER_USE_CC)) {
       gen6_3DSTATE_CC_STATE_POINTERS(r->builder, 0,
-            session->DEPTH_STENCIL_STATE, session->COLOR_CALC_STATE);
+            r->state.DEPTH_STENCIL_STATE, r->state.COLOR_CALC_STATE);
    }
 
-   gen6_rectlist_vs_to_sf(r, blitter, session);
-   gen6_rectlist_wm(r, blitter, session);
+   gen6_rectlist_vs_to_sf(r, blitter);
+   gen6_rectlist_wm(r, blitter);
 
    if (blitter->uses & ILO_BLITTER_USE_VIEWPORT) {
       gen6_3DSTATE_VIEWPORT_STATE_POINTERS(r->builder,
-            0, 0, session->CC_VIEWPORT);
+            0, 0, r->state.CC_VIEWPORT);
    }
 
-   gen6_rectlist_wm_depth(r, blitter, session);
+   gen6_rectlist_wm_depth(r, blitter);
 
    gen6_3DSTATE_DRAWING_RECTANGLE(r->builder, 0, 0,
          blitter->fb.width, blitter->fb.height);
@@ -1608,36 +1075,11 @@ gen6_rectlist_commands(struct ilo_render *r,
 }
 
 static void
-gen6_rectlist_states(struct ilo_render *r,
-                     const struct ilo_blitter *blitter,
-                     struct gen6_rectlist_session *session)
-{
-   if (blitter->uses & ILO_BLITTER_USE_DSA) {
-      session->DEPTH_STENCIL_STATE =
-         gen6_DEPTH_STENCIL_STATE(r->builder, &blitter->dsa);
-   }
-
-   if (blitter->uses & ILO_BLITTER_USE_CC) {
-      session->COLOR_CALC_STATE =
-         gen6_COLOR_CALC_STATE(r->builder, &blitter->cc.stencil_ref,
-               blitter->cc.alpha_ref, &blitter->cc.blend_color);
-   }
-
-   if (blitter->uses & ILO_BLITTER_USE_VIEWPORT) {
-      session->CC_VIEWPORT =
-         gen6_CC_VIEWPORT(r->builder, &blitter->viewport, 1);
-   }
-}
-
-static void
 ilo_render_emit_rectlist_gen6(struct ilo_render *render,
                               const struct ilo_blitter *blitter)
 {
-   struct gen6_rectlist_session session;
-
-   memset(&session, 0, sizeof(session));
-   gen6_rectlist_states(render, blitter, &session);
-   gen6_rectlist_commands(render, blitter, &session);
+   ilo_render_emit_rectlist_dynamic_states(render, blitter);
+   gen6_rectlist_commands(render, blitter);
 }
 
 static int
@@ -1686,117 +1128,6 @@ gen6_render_max_command_size(const struct ilo_render *render)
    return size;
 }
 
-int
-gen6_render_estimate_state_size(const struct ilo_render *render,
-                                const struct ilo_state_vector *vec)
-{
-   static int static_size;
-   int sh_type, size;
-
-   if (!static_size) {
-      /* 64 bytes, or 16 dwords */
-      const int alignment = 64 / 4;
-
-      /* pad first */
-      size = alignment - 1;
-
-      /* CC states */
-      size += align(GEN6_BLEND_STATE__SIZE * ILO_MAX_DRAW_BUFFERS, alignment);
-      size += align(GEN6_DEPTH_STENCIL_STATE__SIZE, alignment);
-      size += align(GEN6_COLOR_CALC_STATE__SIZE, alignment);
-
-      /* viewport arrays */
-      if (ilo_dev_gen(render->dev) >= ILO_GEN(7)) {
-         size +=
-            align(GEN7_SF_CLIP_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 16) +
-            align(GEN6_CC_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
-            align(GEN6_SCISSOR_RECT__SIZE * ILO_MAX_VIEWPORTS, 8);
-      }
-      else {
-         size +=
-            align(GEN6_SF_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
-            align(GEN6_CLIP_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
-            align(GEN6_CC_VIEWPORT__SIZE * ILO_MAX_VIEWPORTS, 8) +
-            align(GEN6_SCISSOR_RECT__SIZE * ILO_MAX_VIEWPORTS, 8);
-      }
-
-      static_size = size;
-   }
-
-   size = static_size;
-
-   for (sh_type = 0; sh_type < PIPE_SHADER_TYPES; sh_type++) {
-      const int alignment = 32 / 4;
-      int num_samplers, num_surfaces, pcb_size;
-
-      /* samplers */
-      num_samplers = vec->sampler[sh_type].count;
-
-      /* sampler views and constant buffers */
-      num_surfaces = vec->view[sh_type].count +
-         util_bitcount(vec->cbuf[sh_type].enabled_mask);
-
-      pcb_size = 0;
-
-      switch (sh_type) {
-      case PIPE_SHADER_VERTEX:
-         if (vec->vs) {
-            if (ilo_dev_gen(render->dev) == ILO_GEN(6)) {
-               const struct pipe_stream_output_info *so_info =
-                  ilo_shader_get_kernel_so_info(vec->vs);
-
-               /* stream outputs */
-               num_surfaces += so_info->num_outputs;
-            }
-
-            pcb_size = ilo_shader_get_kernel_param(vec->vs,
-                  ILO_KERNEL_PCB_CBUF0_SIZE);
-            pcb_size += ilo_shader_get_kernel_param(vec->vs,
-                  ILO_KERNEL_VS_PCB_UCP_SIZE);
-         }
-         break;
-      case PIPE_SHADER_GEOMETRY:
-         if (vec->gs && ilo_dev_gen(render->dev) == ILO_GEN(6)) {
-            const struct pipe_stream_output_info *so_info =
-               ilo_shader_get_kernel_so_info(vec->gs);
-
-            /* stream outputs */
-            num_surfaces += so_info->num_outputs;
-         }
-         break;
-      case PIPE_SHADER_FRAGMENT:
-         /* render targets */
-         num_surfaces += vec->fb.state.nr_cbufs;
-
-         if (vec->fs) {
-            pcb_size = ilo_shader_get_kernel_param(vec->fs,
-                  ILO_KERNEL_PCB_CBUF0_SIZE);
-         }
-         break;
-      default:
-         break;
-      }
-
-      /* SAMPLER_STATE array and SAMPLER_BORDER_COLORs */
-      if (num_samplers) {
-         size += align(GEN6_SAMPLER_STATE__SIZE * num_samplers, alignment) +
-            align(GEN6_SAMPLER_BORDER_COLOR__SIZE, alignment) * num_samplers;
-      }
-
-      /* BINDING_TABLE_STATE and SURFACE_STATEs */
-      if (num_surfaces) {
-         size += align(num_surfaces, alignment) +
-            align(GEN6_SURFACE_STATE__SIZE, alignment) * num_surfaces;
-      }
-
-      /* PCB */
-      if (pcb_size)
-         size += align(pcb_size, alignment);
-   }
-
-   return size;
-}
-
 static int
 ilo_render_estimate_size_gen6(struct ilo_render *render,
                               enum ilo_render_action action,
@@ -1807,14 +1138,20 @@ ilo_render_estimate_size_gen6(struct ilo_render *render,
    switch (action) {
    case ILO_RENDER_DRAW:
       {
-         const struct ilo_state_vector *ilo = arg;
+         const struct ilo_state_vector *vec = arg;
 
          size = gen6_render_max_command_size(render) +
-            gen6_render_estimate_state_size(render, ilo);
+            ilo_render_get_draw_dynamic_states_len(render, vec) +
+            ilo_render_get_draw_surface_states_len(render, vec);
       }
       break;
    case ILO_RENDER_RECTLIST:
-      size = 64 + 256; /* states + commands */
+      {
+         const struct ilo_blitter *blitter = arg;
+
+         size = ilo_render_get_rectlist_dynamic_states_len(render, blitter);
+         size += 256; /* commands */
+      }
       break;
    default:
       assert(!"unknown render action");
index 344f872..7d708c1 100644 (file)
@@ -296,7 +296,7 @@ gen7_draw_common_pointers_1(struct ilo_render *r,
                             struct gen6_draw_session *session)
 {
    /* 3DSTATE_VIEWPORT_STATE_POINTERS_{CC,SF_CLIP} */
-   if (session->viewport_state_changed) {
+   if (session->viewport_changed) {
       gen7_3DSTATE_VIEWPORT_STATE_POINTERS_CC(r->builder,
             r->state.CC_VIEWPORT);
 
@@ -311,19 +311,19 @@ gen7_draw_common_pointers_2(struct ilo_render *r,
                             struct gen6_draw_session *session)
 {
    /* 3DSTATE_BLEND_STATE_POINTERS */
-   if (session->cc_state_blend_changed) {
+   if (session->blend_changed) {
       gen7_3DSTATE_BLEND_STATE_POINTERS(r->builder,
             r->state.BLEND_STATE);
    }
 
    /* 3DSTATE_CC_STATE_POINTERS */
-   if (session->cc_state_cc_changed) {
+   if (session->cc_changed) {
       gen7_3DSTATE_CC_STATE_POINTERS(r->builder,
             r->state.COLOR_CALC_STATE);
    }
 
    /* 3DSTATE_DEPTH_STENCIL_STATE_POINTERS */
-   if (session->cc_state_dsa_changed) {
+   if (session->dsa_changed) {
       gen7_3DSTATE_DEPTH_STENCIL_STATE_POINTERS(r->builder,
             r->state.DEPTH_STENCIL_STATE);
    }
@@ -334,10 +334,12 @@ gen7_draw_vs(struct ilo_render *r,
              const struct ilo_state_vector *vec,
              struct gen6_draw_session *session)
 {
-   const bool emit_3dstate_binding_table = session->binding_table_vs_changed;
-   const bool emit_3dstate_sampler_state = session->sampler_state_vs_changed;
+   const bool emit_3dstate_binding_table =
+      session->binding_table_vs_changed;
+   const bool emit_3dstate_sampler_state =
+      session->sampler_vs_changed;
    /* see gen6_draw_vs() */
-   const bool emit_3dstate_constant_vs = session->pcb_state_vs_changed;
+   const bool emit_3dstate_constant_vs = session->pcb_vs_changed;
    const bool emit_3dstate_vs = (DIRTY(VS) || DIRTY(SAMPLER_VS) ||
            r->instruction_bo_changed);
 
@@ -527,13 +529,13 @@ gen7_draw_wm(struct ilo_render *r,
    }
 
    /* 3DSTATE_SAMPLER_STATE_POINTERS_PS */
-   if (session->sampler_state_fs_changed) {
+   if (session->sampler_fs_changed) {
       gen7_3DSTATE_SAMPLER_STATE_POINTERS_PS(r->builder,
             r->state.wm.SAMPLER_STATE);
    }
 
    /* 3DSTATE_CONSTANT_PS */
-   if (session->pcb_state_fs_changed) {
+   if (session->pcb_fs_changed) {
       gen7_3DSTATE_CONSTANT_PS(r->builder,
             &r->state.wm.PUSH_CONSTANT_BUFFER,
             &r->state.wm.PUSH_CONSTANT_BUFFER_size,
@@ -555,7 +557,7 @@ gen7_draw_wm(struct ilo_render *r,
    }
 
    /* 3DSTATE_SCISSOR_STATE_POINTERS */
-   if (session->scissor_state_changed) {
+   if (session->scissor_changed) {
       gen6_3DSTATE_SCISSOR_STATE_POINTERS(r->builder,
             r->state.SCISSOR_RECT);
    }
@@ -568,13 +570,13 @@ gen7_draw_wm(struct ilo_render *r,
          (DIRTY(FB) || DIRTY(DSA) || r->state_bo_changed);
 
       if (emit_3dstate_ps ||
-          session->pcb_state_fs_changed ||
-          session->viewport_state_changed ||
+          session->pcb_fs_changed ||
+          session->viewport_changed ||
           session->binding_table_fs_changed ||
-          session->sampler_state_fs_changed ||
-          session->cc_state_cc_changed ||
-          session->cc_state_blend_changed ||
-          session->cc_state_dsa_changed)
+          session->sampler_fs_changed ||
+          session->cc_changed ||
+          session->blend_changed ||
+          session->dsa_changed)
          gen7_wa_post_ps_and_later(r);
 
       if (emit_3dstate_depth_buffer)
@@ -692,7 +694,6 @@ ilo_render_emit_draw_gen7(struct ilo_render *render,
 
    gen6_draw_prepare(render, vec, &session);
 
-   session.emit_draw_states = gen6_draw_states;
    session.emit_draw_commands = gen7_draw_commands;
 
    gen6_draw_emit(render, vec, &session);
@@ -701,8 +702,7 @@ ilo_render_emit_draw_gen7(struct ilo_render *render,
 
 static void
 gen7_rectlist_pcb_alloc(struct ilo_render *r,
-                        const struct ilo_blitter *blitter,
-                        struct gen6_rectlist_session *session)
+                        const struct ilo_blitter *blitter)
 {
    /*
     * Push constant buffers are only allowed to take up at most the first
@@ -723,8 +723,7 @@ gen7_rectlist_pcb_alloc(struct ilo_render *r,
 
 static void
 gen7_rectlist_urb(struct ilo_render *r,
-                  const struct ilo_blitter *blitter,
-                  struct gen6_rectlist_session *session)
+                  const struct ilo_blitter *blitter)
 {
    /* the first 16KB are reserved for VS and PS PCBs */
    const int offset =
@@ -740,8 +739,7 @@ gen7_rectlist_urb(struct ilo_render *r,
 
 static void
 gen7_rectlist_vs_to_sf(struct ilo_render *r,
-                       const struct ilo_blitter *blitter,
-                       struct gen6_rectlist_session *session)
+                       const struct ilo_blitter *blitter)
 {
    gen7_3DSTATE_CONSTANT_VS(r->builder, NULL, NULL, 0);
    gen6_3DSTATE_VS(r->builder, NULL, 0);
@@ -769,8 +767,7 @@ gen7_rectlist_vs_to_sf(struct ilo_render *r,
 
 static void
 gen7_rectlist_wm(struct ilo_render *r,
-                 const struct ilo_blitter *blitter,
-                 struct gen6_rectlist_session *session)
+                 const struct ilo_blitter *blitter)
 {
    uint32_t hiz_op;
 
@@ -799,8 +796,7 @@ gen7_rectlist_wm(struct ilo_render *r,
 
 static void
 gen7_rectlist_wm_depth(struct ilo_render *r,
-                       const struct ilo_blitter *blitter,
-                       struct gen6_rectlist_session *session)
+                       const struct ilo_blitter *blitter)
 {
    gen7_wa_pre_depth(r);
 
@@ -826,8 +822,7 @@ gen7_rectlist_wm_depth(struct ilo_render *r,
 
 static void
 gen7_rectlist_wm_multisample(struct ilo_render *r,
-                             const struct ilo_blitter *blitter,
-                             struct gen6_rectlist_session *session)
+                             const struct ilo_blitter *blitter)
 {
    const uint32_t *packed_sample_pos =
       (blitter->fb.num_samples > 4) ? r->packed_sample_position_8x :
@@ -845,10 +840,9 @@ gen7_rectlist_wm_multisample(struct ilo_render *r,
 
 static void
 gen7_rectlist_commands(struct ilo_render *r,
-                       const struct ilo_blitter *blitter,
-                       struct gen6_rectlist_session *session)
+                       const struct ilo_blitter *blitter)
 {
-   gen7_rectlist_wm_multisample(r, blitter, session);
+   gen7_rectlist_wm_multisample(r, blitter);
 
    gen6_state_base_address(r->builder, true);
 
@@ -858,32 +852,32 @@ gen7_rectlist_commands(struct ilo_render *r,
    gen6_3DSTATE_VERTEX_ELEMENTS(r->builder,
          &blitter->ve, false, false);
 
-   gen7_rectlist_pcb_alloc(r, blitter, session);
+   gen7_rectlist_pcb_alloc(r, blitter);
 
    /* needed for any VS-related commands */
    gen7_wa_pre_vs(r);
 
-   gen7_rectlist_urb(r, blitter, session);
+   gen7_rectlist_urb(r, blitter);
 
    if (blitter->uses & ILO_BLITTER_USE_DSA) {
       gen7_3DSTATE_DEPTH_STENCIL_STATE_POINTERS(r->builder,
-            session->DEPTH_STENCIL_STATE);
+            r->state.DEPTH_STENCIL_STATE);
    }
 
    if (blitter->uses & ILO_BLITTER_USE_CC) {
       gen7_3DSTATE_CC_STATE_POINTERS(r->builder,
-            session->COLOR_CALC_STATE);
+            r->state.COLOR_CALC_STATE);
    }
 
-   gen7_rectlist_vs_to_sf(r, blitter, session);
-   gen7_rectlist_wm(r, blitter, session);
+   gen7_rectlist_vs_to_sf(r, blitter);
+   gen7_rectlist_wm(r, blitter);
 
    if (blitter->uses & ILO_BLITTER_USE_VIEWPORT) {
       gen7_3DSTATE_VIEWPORT_STATE_POINTERS_CC(r->builder,
-            session->CC_VIEWPORT);
+            r->state.CC_VIEWPORT);
    }
 
-   gen7_rectlist_wm_depth(r, blitter, session);
+   gen7_rectlist_wm_depth(r, blitter);
 
    gen6_3DSTATE_DRAWING_RECTANGLE(r->builder, 0, 0,
          blitter->fb.width, blitter->fb.height);
@@ -892,36 +886,11 @@ gen7_rectlist_commands(struct ilo_render *r,
 }
 
 static void
-gen7_rectlist_states(struct ilo_render *r,
-                     const struct ilo_blitter *blitter,
-                     struct gen6_rectlist_session *session)
-{
-   if (blitter->uses & ILO_BLITTER_USE_DSA) {
-      session->DEPTH_STENCIL_STATE =
-         gen6_DEPTH_STENCIL_STATE(r->builder, &blitter->dsa);
-   }
-
-   if (blitter->uses & ILO_BLITTER_USE_CC) {
-      session->COLOR_CALC_STATE =
-         gen6_COLOR_CALC_STATE(r->builder, &blitter->cc.stencil_ref,
-               blitter->cc.alpha_ref, &blitter->cc.blend_color);
-   }
-
-   if (blitter->uses & ILO_BLITTER_USE_VIEWPORT) {
-      session->CC_VIEWPORT =
-         gen6_CC_VIEWPORT(r->builder, &blitter->viewport, 1);
-   }
-}
-
-static void
 ilo_render_emit_rectlist_gen7(struct ilo_render *render,
                               const struct ilo_blitter *blitter)
 {
-   struct gen6_rectlist_session session;
-
-   memset(&session, 0, sizeof(session));
-   gen7_rectlist_states(render, blitter, &session);
-   gen7_rectlist_commands(render, blitter, &session);
+   ilo_render_emit_rectlist_dynamic_states(render, blitter);
+   gen7_rectlist_commands(render, blitter);
 }
 
 static int
@@ -985,14 +954,20 @@ ilo_render_estimate_size_gen7(struct ilo_render *render,
    switch (action) {
    case ILO_RENDER_DRAW:
       {
-         const struct ilo_state_vector *ilo = arg;
+         const struct ilo_state_vector *vec = arg;
 
          size = gen7_render_max_command_size(render) +
-            gen6_render_estimate_state_size(render, ilo);
+            ilo_render_get_draw_dynamic_states_len(render, vec) +
+            ilo_render_get_draw_surface_states_len(render, vec);
       }
       break;
    case ILO_RENDER_RECTLIST:
-      size = 64 + 256; /* states + commands */
+      {
+         const struct ilo_blitter *blitter = arg;
+
+         size = ilo_render_get_rectlist_dynamic_states_len(render, blitter);
+         size += 256; /* commands */
+      }
       break;
    default:
       assert(!"unknown render action");
diff --git a/src/gallium/drivers/ilo/ilo_render_surface.c b/src/gallium/drivers/ilo/ilo_render_surface.c
new file mode 100644 (file)
index 0000000..50d3728
--- /dev/null
@@ -0,0 +1,405 @@
+/*
+ * Mesa 3-D graphics library
+ *
+ * Copyright (C) 2012-2014 LunarG, Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included
+ * in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ *
+ * Authors:
+ *    Chia-I Wu <olv@lunarg.com>
+ */
+
+#include "ilo_common.h"
+#include "ilo_blitter.h"
+#include "ilo_builder_3d.h"
+#include "ilo_state.h"
+#include "ilo_render_gen.h"
+#include "ilo_render.h"
+
+#define DIRTY(state) (session->pipe_dirty & ILO_DIRTY_ ## state)
+
+static void
+gen6_emit_draw_surface_rt(struct ilo_render *r,
+                          const struct ilo_state_vector *vec,
+                          struct gen6_draw_session *session)
+{
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   /* SURFACE_STATEs for render targets */
+   if (DIRTY(FB)) {
+      const struct ilo_fb_state *fb = &vec->fb;
+      const int offset = ILO_WM_DRAW_SURFACE(0);
+      uint32_t *surface_state = &r->state.wm.SURFACE_STATE[offset];
+      int i;
+
+      for (i = 0; i < fb->state.nr_cbufs; i++) {
+         const struct ilo_surface_cso *surface =
+            (const struct ilo_surface_cso *) fb->state.cbufs[i];
+
+         if (!surface) {
+            surface_state[i] =
+               gen6_SURFACE_STATE(r->builder, &fb->null_rt, true);
+         } else {
+            assert(surface && surface->is_rt);
+            surface_state[i] =
+               gen6_SURFACE_STATE(r->builder, &surface->u.rt, true);
+         }
+      }
+
+      /*
+       * Upload at least one render target, as
+       * brw_update_renderbuffer_surfaces() does.  I don't know why.
+       */
+      if (i == 0) {
+         surface_state[i] =
+            gen6_SURFACE_STATE(r->builder, &fb->null_rt, true);
+
+         i++;
+      }
+
+      memset(&surface_state[i], 0, (ILO_MAX_DRAW_BUFFERS - i) * 4);
+
+      if (i && session->num_surfaces[PIPE_SHADER_FRAGMENT] < offset + i)
+         session->num_surfaces[PIPE_SHADER_FRAGMENT] = offset + i;
+
+      session->binding_table_fs_changed = true;
+   }
+}
+
+static void
+gen6_emit_draw_surface_so(struct ilo_render *r,
+                          const struct ilo_state_vector *vec,
+                          struct gen6_draw_session *session)
+{
+   const struct ilo_so_state *so = &vec->so;
+
+   ILO_DEV_ASSERT(r->dev, 6, 6);
+
+   /* SURFACE_STATEs for stream output targets */
+   if (DIRTY(VS) || DIRTY(GS) || DIRTY(SO)) {
+      const struct pipe_stream_output_info *so_info =
+         (vec->gs) ? ilo_shader_get_kernel_so_info(vec->gs) :
+         (vec->vs) ? ilo_shader_get_kernel_so_info(vec->vs) : NULL;
+      const int offset = ILO_GS_SO_SURFACE(0);
+      uint32_t *surface_state = &r->state.gs.SURFACE_STATE[offset];
+      int i;
+
+      for (i = 0; so_info && i < so_info->num_outputs; i++) {
+         const int target = so_info->output[i].output_buffer;
+         const struct pipe_stream_output_target *so_target =
+            (target < so->count) ? so->states[target] : NULL;
+
+         if (so_target) {
+            surface_state[i] = gen6_so_SURFACE_STATE(r->builder,
+                  so_target, so_info, i);
+         } else {
+            surface_state[i] = 0;
+         }
+      }
+
+      memset(&surface_state[i], 0, (ILO_MAX_SO_BINDINGS - i) * 4);
+
+      if (i && session->num_surfaces[PIPE_SHADER_GEOMETRY] < offset + i)
+         session->num_surfaces[PIPE_SHADER_GEOMETRY] = offset + i;
+
+      session->binding_table_gs_changed = true;
+   }
+}
+
+static void
+gen6_emit_draw_surface_view(struct ilo_render *r,
+                            const struct ilo_state_vector *vec,
+                            int shader_type,
+                            struct gen6_draw_session *session)
+{
+   const struct ilo_view_state *view = &vec->view[shader_type];
+   uint32_t *surface_state;
+   int offset, i;
+   bool skip = false;
+
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   /* SURFACE_STATEs for sampler views */
+   switch (shader_type) {
+   case PIPE_SHADER_VERTEX:
+      if (DIRTY(VIEW_VS)) {
+         offset = ILO_VS_TEXTURE_SURFACE(0);
+         surface_state = &r->state.vs.SURFACE_STATE[offset];
+
+         session->binding_table_vs_changed = true;
+      } else {
+         skip = true;
+      }
+      break;
+   case PIPE_SHADER_FRAGMENT:
+      if (DIRTY(VIEW_FS)) {
+         offset = ILO_WM_TEXTURE_SURFACE(0);
+         surface_state = &r->state.wm.SURFACE_STATE[offset];
+
+         session->binding_table_fs_changed = true;
+      } else {
+         skip = true;
+      }
+      break;
+   default:
+      skip = true;
+      break;
+   }
+
+   if (skip)
+      return;
+
+   for (i = 0; i < view->count; i++) {
+      if (view->states[i]) {
+         const struct ilo_view_cso *cso =
+            (const struct ilo_view_cso *) view->states[i];
+
+         surface_state[i] =
+            gen6_SURFACE_STATE(r->builder, &cso->surface, false);
+      } else {
+         surface_state[i] = 0;
+      }
+   }
+
+   memset(&surface_state[i], 0, (ILO_MAX_SAMPLER_VIEWS - i) * 4);
+
+   if (i && session->num_surfaces[shader_type] < offset + i)
+      session->num_surfaces[shader_type] = offset + i;
+}
+
+static void
+gen6_emit_draw_surface_const(struct ilo_render *r,
+                             const struct ilo_state_vector *vec,
+                             int shader_type,
+                             struct gen6_draw_session *session)
+{
+   const struct ilo_cbuf_state *cbuf = &vec->cbuf[shader_type];
+   uint32_t *surface_state;
+   bool *binding_table_changed;
+   int offset, count, i;
+
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   if (!DIRTY(CBUF))
+      return;
+
+   /* SURFACE_STATEs for constant buffers */
+   switch (shader_type) {
+   case PIPE_SHADER_VERTEX:
+      offset = ILO_VS_CONST_SURFACE(0);
+      surface_state = &r->state.vs.SURFACE_STATE[offset];
+      binding_table_changed = &session->binding_table_vs_changed;
+      break;
+   case PIPE_SHADER_FRAGMENT:
+      offset = ILO_WM_CONST_SURFACE(0);
+      surface_state = &r->state.wm.SURFACE_STATE[offset];
+      binding_table_changed = &session->binding_table_fs_changed;
+      break;
+   default:
+      return;
+      break;
+   }
+
+   /* constants are pushed via PCB */
+   if (cbuf->enabled_mask == 0x1 && !cbuf->cso[0].resource) {
+      memset(surface_state, 0, ILO_MAX_CONST_BUFFERS * 4);
+      return;
+   }
+
+   count = util_last_bit(cbuf->enabled_mask);
+   for (i = 0; i < count; i++) {
+      if (cbuf->cso[i].resource) {
+         surface_state[i] = gen6_SURFACE_STATE(r->builder,
+               &cbuf->cso[i].surface, false);
+      } else {
+         surface_state[i] = 0;
+      }
+   }
+
+   memset(&surface_state[count], 0, (ILO_MAX_CONST_BUFFERS - count) * 4);
+
+   if (count && session->num_surfaces[shader_type] < offset + count)
+      session->num_surfaces[shader_type] = offset + count;
+
+   *binding_table_changed = true;
+}
+
+static void
+gen6_emit_draw_surface_binding_tables(struct ilo_render *r,
+                                      const struct ilo_state_vector *vec,
+                                      int shader_type,
+                                      struct gen6_draw_session *session)
+{
+   uint32_t *binding_table_state, *surface_state;
+   int *binding_table_state_size, size;
+   bool skip = false;
+
+   ILO_DEV_ASSERT(r->dev, 6, 7.5);
+
+   /* BINDING_TABLE_STATE */
+   switch (shader_type) {
+   case PIPE_SHADER_VERTEX:
+      surface_state = r->state.vs.SURFACE_STATE;
+      binding_table_state = &r->state.vs.BINDING_TABLE_STATE;
+      binding_table_state_size = &r->state.vs.BINDING_TABLE_STATE_size;
+
+      skip = !session->binding_table_vs_changed;
+      break;
+   case PIPE_SHADER_GEOMETRY:
+      surface_state = r->state.gs.SURFACE_STATE;
+      binding_table_state = &r->state.gs.BINDING_TABLE_STATE;
+      binding_table_state_size = &r->state.gs.BINDING_TABLE_STATE_size;
+
+      skip = !session->binding_table_gs_changed;
+      break;
+   case PIPE_SHADER_FRAGMENT:
+      surface_state = r->state.wm.SURFACE_STATE;
+      binding_table_state = &r->state.wm.BINDING_TABLE_STATE;
+      binding_table_state_size = &r->state.wm.BINDING_TABLE_STATE_size;
+
+      skip = !session->binding_table_fs_changed;
+      break;
+   default:
+      skip = true;
+      break;
+   }
+
+   if (skip)
+      return;
+
+   /*
+    * If we have seemingly less SURFACE_STATEs than before, it could be that
+    * we did not touch those reside at the tail in this upload.  Loop over
+    * them to figure out the real number of SURFACE_STATEs.
+    */
+   for (size = *binding_table_state_size;
+         size > session->num_surfaces[shader_type]; size--) {
+      if (surface_state[size - 1])
+         break;
+   }
+   if (size < session->num_surfaces[shader_type])
+      size = session->num_surfaces[shader_type];
+
+   *binding_table_state = gen6_BINDING_TABLE_STATE(r->builder,
+         surface_state, size);
+   *binding_table_state_size = size;
+}
+
+#undef DIRTY
+
+int
+ilo_render_get_draw_surface_states_len(const struct ilo_render *render,
+                                       const struct ilo_state_vector *vec)
+{
+   int sh_type, len;
+
+   ILO_DEV_ASSERT(render->dev, 6, 7.5);
+
+   len = 0;
+
+   for (sh_type = 0; sh_type < PIPE_SHADER_TYPES; sh_type++) {
+      const int alignment = 32 / 4;
+      int num_surfaces = 0;
+
+      switch (sh_type) {
+      case PIPE_SHADER_VERTEX:
+         if (vec->view[sh_type].count) {
+            num_surfaces = ILO_VS_TEXTURE_SURFACE(vec->view[sh_type].count);
+         } else {
+            num_surfaces = ILO_VS_CONST_SURFACE(
+                  util_last_bit(vec->cbuf[sh_type].enabled_mask));
+         }
+
+         if (vec->vs) {
+            if (ilo_dev_gen(render->dev) == ILO_GEN(6)) {
+               const struct pipe_stream_output_info *so_info =
+                  ilo_shader_get_kernel_so_info(vec->vs);
+
+               /* stream outputs */
+               num_surfaces += so_info->num_outputs;
+            }
+         }
+         break;
+      case PIPE_SHADER_GEOMETRY:
+         if (vec->gs && ilo_dev_gen(render->dev) == ILO_GEN(6)) {
+            const struct pipe_stream_output_info *so_info =
+               ilo_shader_get_kernel_so_info(vec->gs);
+
+            /* stream outputs */
+            num_surfaces += so_info->num_outputs;
+         }
+         break;
+      case PIPE_SHADER_FRAGMENT:
+         if (vec->view[sh_type].count) {
+            num_surfaces = ILO_WM_TEXTURE_SURFACE(vec->view[sh_type].count);
+         } else if (vec->cbuf[sh_type].enabled_mask) {
+            num_surfaces = ILO_WM_CONST_SURFACE(
+                  util_last_bit(vec->cbuf[sh_type].enabled_mask));
+         } else {
+            num_surfaces = vec->fb.state.nr_cbufs;
+         }
+         break;
+      default:
+         break;
+      }
+
+      /* BINDING_TABLE_STATE and SURFACE_STATEs */
+      if (num_surfaces) {
+         len += align(num_surfaces, alignment) +
+            align(GEN6_SURFACE_STATE__SIZE, alignment) * num_surfaces;
+      }
+   }
+
+   return len;
+}
+
+void
+ilo_render_emit_draw_surface_states(struct ilo_render *render,
+                                    const struct ilo_state_vector *vec,
+                                    struct gen6_draw_session *session)
+{
+   const unsigned surface_used = ilo_builder_surface_used(render->builder);
+   int shader_type;
+
+   ILO_DEV_ASSERT(render->dev, 6, 7.5);
+
+   /*
+    * upload all SURAFCE_STATEs together so that we know there are minimal
+    * paddings
+    */
+
+   gen6_emit_draw_surface_rt(render, vec, session);
+
+   if (ilo_dev_gen(render->dev) == ILO_GEN(6))
+      gen6_emit_draw_surface_so(render, vec, session);
+
+   for (shader_type = 0; shader_type < PIPE_SHADER_TYPES; shader_type++) {
+      gen6_emit_draw_surface_view(render, vec, shader_type, session);
+      gen6_emit_draw_surface_const(render, vec, shader_type, session);
+   }
+
+   /* this must be called after all SURFACE_STATEs have been uploaded */
+   for (shader_type = 0; shader_type < PIPE_SHADER_TYPES; shader_type++) {
+      gen6_emit_draw_surface_binding_tables(render, vec,
+            shader_type, session);
+   }
+
+   assert(ilo_builder_surface_used(render->builder) <= surface_used +
+         ilo_render_get_draw_surface_states_len(render, vec));
+}