[Mesa-dev] [PATCH 30/30] r600g: add support for geom shaders to r600/r700 chipsets

Alex Deucher alexdeucher at gmail.com
Tue Feb 4 16:01:31 CET 2014


On Mon, Feb 3, 2014 at 6:53 PM, Dave Airlie <airlied at gmail.com> wrote:
> From: Dave Airlie <airlied at redhat.com>
>
> This is my first attempt at enabling r600/r700 geometry shaders,
> the basic tests pass on both my rv770 and my rv635,
>
> It requires this kernel patch:
> http://www.spinics.net/lists/dri-devel/msg52745.html
>
> Signed-off-by: Dave Airlie <airlied at redhat.com>

A couple of comments below.

> ---
>  src/gallium/drivers/r600/r600_asm.c          |   2 +-
>  src/gallium/drivers/r600/r600_pipe.c         |  16 +-
>  src/gallium/drivers/r600/r600_pipe.h         |   3 +
>  src/gallium/drivers/r600/r600_shader.c       |  26 ++-
>  src/gallium/drivers/r600/r600_state.c        | 264 +++++++++++++++++++++++----
>  src/gallium/drivers/r600/r600_state_common.c |   2 +-
>  src/gallium/drivers/r600/r600d.h             |  50 ++++-
>  7 files changed, 314 insertions(+), 49 deletions(-)
>
> diff --git a/src/gallium/drivers/r600/r600_asm.c b/src/gallium/drivers/r600/r600_asm.c
> index 899a8ef..3afe7b2 100644
> --- a/src/gallium/drivers/r600/r600_asm.c
> +++ b/src/gallium/drivers/r600/r600_asm.c
> @@ -1535,7 +1535,7 @@ static int r600_bytecode_cf_build(struct r600_bytecode *bc, struct r600_bytecode
>                         S_SQ_CF_ALLOC_EXPORT_WORD1_BARRIER(cf->barrier) |
>                         S_SQ_CF_ALLOC_EXPORT_WORD1_CF_INST(opcode) |
>                         S_SQ_CF_ALLOC_EXPORT_WORD1_END_OF_PROGRAM(cf->end_of_program);
> -       } else if (cfop->flags & CF_STRM) {
> +       } else if (cfop->flags & CF_MEM) {
>                 bc->bytecode[id++] = S_SQ_CF_ALLOC_EXPORT_WORD0_RW_GPR(cf->output.gpr) |
>                         S_SQ_CF_ALLOC_EXPORT_WORD0_ELEM_SIZE(cf->output.elem_size) |
>                         S_SQ_CF_ALLOC_EXPORT_WORD0_ARRAY_BASE(cf->output.array_base) |
> diff --git a/src/gallium/drivers/r600/r600_pipe.c b/src/gallium/drivers/r600/r600_pipe.c
> index 7ed70e1..edf37ad 100644
> --- a/src/gallium/drivers/r600/r600_pipe.c
> +++ b/src/gallium/drivers/r600/r600_pipe.c
> @@ -372,7 +372,12 @@ static int r600_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
>                 return 1;
>
>         case PIPE_CAP_GLSL_FEATURE_LEVEL:
> -               return family >= CHIP_CEDAR ? 330 : 140;
> +               if (family >= CHIP_CEDAR)
> +                  return 330;
> +               /* pre-evergreen geom shaders need newer kernel */
> +               if (rscreen->b.info.drm_minor >= 37)
> +                  return 330;
> +               return 140;
>
>         /* Supported except the original R600. */
>         case PIPE_CAP_INDEP_BLEND_ENABLE:
> @@ -458,9 +463,12 @@ static int r600_get_shader_param(struct pipe_screen* pscreen, unsigned shader, e
>         case PIPE_SHADER_COMPUTE:
>                 break;
>         case PIPE_SHADER_GEOMETRY:
> -               if (rscreen->b.chip_class < EVERGREEN)
> -                       return 0;
> -               break;
> +               if (rscreen->b.family >= CHIP_CEDAR)
> +                       break;
> +               /* pre-evergreen geom shaders need newer kernel */
> +               if (rscreen->b.info.drm_minor >= 37)
> +                       break;
> +               return 0;
>         default:
>                 /* XXX: support tessellation on Evergreen */
>                 return 0;
> diff --git a/src/gallium/drivers/r600/r600_pipe.h b/src/gallium/drivers/r600/r600_pipe.h
> index 2d2c79b..9f27a17 100644
> --- a/src/gallium/drivers/r600/r600_pipe.h
> +++ b/src/gallium/drivers/r600/r600_pipe.h
> @@ -160,6 +160,7 @@ struct r600_sample_mask {
>  struct r600_config_state {
>         struct r600_atom atom;
>         unsigned sq_gpr_resource_mgmt_1;
> +       unsigned sq_gpr_resource_mgmt_2;
>  };
>
>  struct r600_stencil_ref
> @@ -565,6 +566,8 @@ r600_create_sampler_view_custom(struct pipe_context *ctx,
>  void r600_init_state_functions(struct r600_context *rctx);
>  void r600_init_atom_start_cs(struct r600_context *rctx);
>  void r600_update_ps_state(struct pipe_context *ctx, struct r600_pipe_shader *shader);
> +void r600_update_es_state(struct pipe_context *ctx, struct r600_pipe_shader *shader);
> +void r600_update_gs_state(struct pipe_context *ctx, struct r600_pipe_shader *shader);
>  void r600_update_vs_state(struct pipe_context *ctx, struct r600_pipe_shader *shader);
>  void *r600_create_db_flush_dsa(struct r600_context *rctx);
>  void *r600_create_resolve_blend(struct r600_context *rctx);
> diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c
> index f0e980b..ffe4749 100644
> --- a/src/gallium/drivers/r600/r600_shader.c
> +++ b/src/gallium/drivers/r600/r600_shader.c
> @@ -210,7 +210,8 @@ int r600_pipe_shader_create(struct pipe_context *ctx,
>                         evergreen_update_gs_state(ctx, shader);
>                         evergreen_update_vs_state(ctx, shader->gs_copy_shader);
>                 } else {
> -                       assert(!"not suported yet");
> +                       r600_update_gs_state(ctx, shader);
> +                       r600_update_vs_state(ctx, shader->gs_copy_shader);
>                 }
>                 break;
>         case TGSI_PROCESSOR_VERTEX:
> @@ -220,7 +221,10 @@ int r600_pipe_shader_create(struct pipe_context *ctx,
>                         else
>                                 evergreen_update_vs_state(ctx, shader);
>                 } else {
> -                       r600_update_vs_state(ctx, shader);
> +                       if (export_shader)
> +                               r600_update_es_state(ctx, shader);
> +                       else
> +                               r600_update_vs_state(ctx, shader);
>                 }
>                 break;
>         case TGSI_PROCESSOR_FRAGMENT:
> @@ -906,7 +910,11 @@ static int fetch_gs_input(struct r600_shader_ctx *ctx, struct tgsi_full_src_regi
>         vtx.dst_sel_y = 1;              /* SEL_Y */
>         vtx.dst_sel_z = 2;              /* SEL_Z */
>         vtx.dst_sel_w = 3;              /* SEL_W */
> -       vtx.use_const_fields = 1;
> +       if (ctx->bc->chip_class >= EVERGREEN) {
> +               vtx.use_const_fields = 1;
> +       } else {
> +               vtx.data_format = FMT_32_32_32_32_FLOAT;
> +       }
>
>         if ((r = r600_bytecode_add_vtx(ctx->bc, &vtx)))
>                 return r;
> @@ -1229,7 +1237,11 @@ static int generate_gs_copy_shader(struct r600_context *rctx,
>                 vtx.dst_sel_y = 1;
>                 vtx.dst_sel_z = 2;
>                 vtx.dst_sel_w = 3;
> -               vtx.use_const_fields = 1;
> +               if (rctx->b.chip_class >= EVERGREEN) {
> +                       vtx.use_const_fields = 1;
> +               } else {
> +                       vtx.data_format = FMT_32_32_32_32_FLOAT;
> +               }
>
>                 r600_bytecode_add_vtx(ctx.bc, &vtx);
>         }
> @@ -1550,7 +1562,7 @@ static int r600_shader_from_tgsi(struct r600_context *rctx,
>         if (ctx.type == TGSI_PROCESSOR_FRAGMENT && ctx.bc->chip_class >= EVERGREEN) {
>                 ctx.file_offset[TGSI_FILE_INPUT] = evergreen_gpr_count(&ctx);
>         }
> -       if (ctx.type == TGSI_PROCESSOR_GEOMETRY && ctx.bc->chip_class >= EVERGREEN) {
> +       if (ctx.type == TGSI_PROCESSOR_GEOMETRY/* && ctx.bc->chip_class >= EVERGREEN*/) {

Can just drop the evergreen part rather than just commenting it out.


>                 /* FIXME 1 would be enough in some cases (3 or less input vertices) */
>                 ctx.file_offset[TGSI_FILE_INPUT] = 2;
>         }
> @@ -6296,8 +6308,8 @@ static struct r600_shader_tgsi_instruction r600_shader_tgsi_instruction[] = {
>         {TGSI_OPCODE_TXF,       0, FETCH_OP_LD, tgsi_tex},
>         {TGSI_OPCODE_TXQ,       0, FETCH_OP_GET_TEXTURE_RESINFO, tgsi_tex},
>         {TGSI_OPCODE_CONT,      0, CF_OP_LOOP_CONTINUE, tgsi_loop_brk_cont},
> -       {TGSI_OPCODE_EMIT,      0, ALU_OP0_NOP, tgsi_unsupported},
> -       {TGSI_OPCODE_ENDPRIM,   0, ALU_OP0_NOP, tgsi_unsupported},
> +       {TGSI_OPCODE_EMIT,      0, CF_OP_EMIT_VERTEX, tgsi_gs_emit},
> +       {TGSI_OPCODE_ENDPRIM,   0, CF_OP_CUT_VERTEX, tgsi_gs_emit},
>         {TGSI_OPCODE_BGNLOOP,   0, ALU_OP0_NOP, tgsi_bgnloop},
>         {TGSI_OPCODE_BGNSUB,    0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_ENDLOOP,   0, ALU_OP0_NOP, tgsi_endloop},
> diff --git a/src/gallium/drivers/r600/r600_state.c b/src/gallium/drivers/r600/r600_state.c
> index e0c801e..c063e2b 100644
> --- a/src/gallium/drivers/r600/r600_state.c
> +++ b/src/gallium/drivers/r600/r600_state.c
> @@ -2064,6 +2064,7 @@ static void r600_emit_config_state(struct r600_context *rctx, struct r600_atom *
>         struct r600_config_state *a = (struct r600_config_state*)atom;
>
>         r600_write_config_reg(cs, R_008C04_SQ_GPR_RESOURCE_MGMT_1, a->sq_gpr_resource_mgmt_1);
> +       r600_write_config_reg(cs, R_008C08_SQ_GPR_RESOURCE_MGMT_2, a->sq_gpr_resource_mgmt_2);
>  }
>
>  static void r600_emit_vertex_buffers(struct r600_context *rctx, struct r600_atom *atom)
> @@ -2115,16 +2116,18 @@ static void r600_emit_constant_buffers(struct r600_context *rctx,
>                 struct r600_resource *rbuffer;
>                 unsigned offset;
>                 unsigned buffer_index = ffs(dirty_mask) - 1;
> -
> +               unsigned gs_ring_buffer = (buffer_index == R600_GS_RING_CONST_BUFFER);
>                 cb = &state->cb[buffer_index];
>                 rbuffer = (struct r600_resource*)cb->buffer;
>                 assert(rbuffer);
>
>                 offset = cb->buffer_offset;
>
> -               r600_write_context_reg(cs, reg_alu_constbuf_size + buffer_index * 4,
> -                                      ALIGN_DIVUP(cb->buffer_size >> 4, 16));
> -               r600_write_context_reg(cs, reg_alu_const_cache + buffer_index * 4, offset >> 8);
> +               if (!gs_ring_buffer) {
> +                       r600_write_context_reg(cs, reg_alu_constbuf_size + buffer_index * 4,
> +                                              ALIGN_DIVUP(cb->buffer_size >> 4, 16));
> +                       r600_write_context_reg(cs, reg_alu_const_cache + buffer_index * 4, offset >> 8);
> +               }
>
>                 radeon_emit(cs, PKT3(PKT3_NOP, 0, 0));
>                 radeon_emit(cs, r600_context_bo_reloc(&rctx->b, &rctx->b.rings.gfx, rbuffer, RADEON_USAGE_READ));
> @@ -2134,8 +2137,8 @@ static void r600_emit_constant_buffers(struct r600_context *rctx,
>                 radeon_emit(cs, offset); /* RESOURCEi_WORD0 */
>                 radeon_emit(cs, rbuffer->buf->size - offset - 1); /* RESOURCEi_WORD1 */
>                 radeon_emit(cs, /* RESOURCEi_WORD2 */
> -                                S_038008_ENDIAN_SWAP(r600_endian_swap(32)) |
> -                                S_038008_STRIDE(16));
> +                           S_038008_ENDIAN_SWAP(gs_ring_buffer ? ENDIAN_NONE : r600_endian_swap(32)) |
> +                           S_038008_STRIDE(gs_ring_buffer ? 4 : 16));
>                 radeon_emit(cs, 0); /* RESOURCEi_WORD3 */
>                 radeon_emit(cs, 0); /* RESOURCEi_WORD4 */
>                 radeon_emit(cs, 0); /* RESOURCEi_WORD5 */
> @@ -2320,34 +2323,124 @@ static void r600_emit_vertex_fetch_shader(struct r600_context *rctx, struct r600
>         radeon_emit(cs, r600_context_bo_reloc(&rctx->b, &rctx->b.rings.gfx, shader->buffer, RADEON_USAGE_READ));
>  }
>
> +static void r600_emit_shader_stages(struct r600_context *rctx, struct r600_atom *a)
> +{
> +       struct radeon_winsys_cs *cs = rctx->b.rings.gfx.cs;
> +       struct r600_shader_stages_state *state = (struct r600_shader_stages_state*)a;
> +
> +       uint32_t v2 = 0, primid = 0;
> +
> +       if (state->geom_enable) {
> +               uint32_t cut_val;
> +
> +               if (rctx->gs_shader->current->shader.gs_max_out_vertices <= 128)
> +                       cut_val = V_028A40_GS_CUT_128;
> +               else if (rctx->gs_shader->current->shader.gs_max_out_vertices <= 256)
> +                       cut_val = V_028A40_GS_CUT_256;
> +               else if (rctx->gs_shader->current->shader.gs_max_out_vertices <= 512)
> +                       cut_val = V_028A40_GS_CUT_512;
> +               else
> +                       cut_val = V_028A40_GS_CUT_1024;
> +
> +               v2 = S_028A40_MODE(V_028A40_GS_SCENARIO_G) |
> +                       S_028A40_CUT_MODE(cut_val);
> +
> +               if (rctx->gs_shader->current->shader.gs_prim_id_input)
> +                       primid = 1;
> +       }
> +
> +       r600_write_context_reg(cs, R_028A40_VGT_GS_MODE, v2);
> +       r600_write_context_reg(cs, R_028A84_VGT_PRIMITIVEID_EN, primid);
> +}
> +
> +static void r600_emit_gs_rings(struct r600_context *rctx, struct r600_atom *a)
> +{
> +       struct pipe_screen *screen = rctx->b.b.screen;
> +       struct radeon_winsys_cs *cs = rctx->b.rings.gfx.cs;
> +       struct r600_gs_rings_state *state = (struct r600_gs_rings_state*)a;
> +       struct r600_resource *rbuffer;
> +
> +       r600_write_config_reg(cs, R_008040_WAIT_UNTIL, S_008040_WAIT_3D_IDLE(1));
> +       radeon_emit(cs, PKT3(PKT3_EVENT_WRITE, 0, 0));
> +       radeon_emit(cs, EVENT_TYPE(EVENT_TYPE_VGT_FLUSH));
> +
> +       if (state->enable) {
> +               rbuffer =(struct r600_resource*)state->esgs_ring.buffer;
> +               r600_write_config_reg(cs, R_008C40_SQ_ESGS_RING_BASE,
> +                               (r600_resource_va(screen, &rbuffer->b.b)) >> 8);
> +               radeon_emit(cs, PKT3(PKT3_NOP, 0, 0));
> +               radeon_emit(cs, r600_context_bo_reloc(&rctx->b, &rctx->b.rings.gfx, rbuffer, RADEON_USAGE_READWRITE));
> +               r600_write_config_reg(cs, R_008C44_SQ_ESGS_RING_SIZE,
> +                               state->esgs_ring.buffer_size >> 8);
> +
> +               rbuffer =(struct r600_resource*)state->gsvs_ring.buffer;
> +               r600_write_config_reg(cs, R_008C48_SQ_GSVS_RING_BASE,
> +                               (r600_resource_va(screen, &rbuffer->b.b)) >> 8);
> +               radeon_emit(cs, PKT3(PKT3_NOP, 0, 0));
> +               radeon_emit(cs, r600_context_bo_reloc(&rctx->b, &rctx->b.rings.gfx, rbuffer, RADEON_USAGE_READWRITE));
> +               r600_write_config_reg(cs, R_008C4C_SQ_GSVS_RING_SIZE,
> +                               state->gsvs_ring.buffer_size >> 8);
> +       } else {
> +               r600_write_config_reg(cs, R_008C44_SQ_ESGS_RING_SIZE, 0);
> +               r600_write_config_reg(cs, R_008C4C_SQ_GSVS_RING_SIZE, 0);
> +       }
> +
> +       r600_write_config_reg(cs, R_008040_WAIT_UNTIL, S_008040_WAIT_3D_IDLE(1));
> +       radeon_emit(cs, PKT3(PKT3_EVENT_WRITE, 0, 0));
> +       radeon_emit(cs, EVENT_TYPE(EVENT_TYPE_VGT_FLUSH));
> +}
> +
>  /* Adjust GPR allocation on R6xx/R7xx */
>  bool r600_adjust_gprs(struct r600_context *rctx)
>  {
>         unsigned num_ps_gprs = rctx->ps_shader->current->shader.bc.ngpr;
> -       unsigned num_vs_gprs = rctx->vs_shader->current->shader.bc.ngpr;
> +       unsigned num_vs_gprs, num_es_gprs, num_gs_gprs;
>         unsigned new_num_ps_gprs = num_ps_gprs;
> -       unsigned new_num_vs_gprs = num_vs_gprs;
> +       unsigned new_num_vs_gprs, new_num_es_gprs, new_num_gs_gprs;
>         unsigned cur_num_ps_gprs = G_008C04_NUM_PS_GPRS(rctx->config_state.sq_gpr_resource_mgmt_1);
>         unsigned cur_num_vs_gprs = G_008C04_NUM_VS_GPRS(rctx->config_state.sq_gpr_resource_mgmt_1);
> +       unsigned cur_num_gs_gprs = G_008C08_NUM_GS_GPRS(rctx->config_state.sq_gpr_resource_mgmt_2);
> +       unsigned cur_num_es_gprs = G_008C08_NUM_ES_GPRS(rctx->config_state.sq_gpr_resource_mgmt_2);
>         unsigned def_num_ps_gprs = rctx->default_ps_gprs;
>         unsigned def_num_vs_gprs = rctx->default_vs_gprs;
> +       unsigned def_num_gs_gprs = 0;
> +       unsigned def_num_es_gprs = 0;
>         unsigned def_num_clause_temp_gprs = rctx->r6xx_num_clause_temp_gprs;
>         /* hardware will reserve twice num_clause_temp_gprs */
> -       unsigned max_gprs = def_num_ps_gprs + def_num_vs_gprs + def_num_clause_temp_gprs * 2;
> -       unsigned tmp;
> +       unsigned max_gprs = def_num_gs_gprs + def_num_es_gprs + def_num_ps_gprs + def_num_vs_gprs + def_num_clause_temp_gprs * 2;
> +       unsigned tmp, tmp2;
> +
> +       if (rctx->gs_shader) {
> +               num_es_gprs = rctx->vs_shader->current->shader.bc.ngpr;
> +               num_gs_gprs = rctx->gs_shader->current->shader.bc.ngpr;
> +               num_vs_gprs = rctx->gs_shader->current->gs_copy_shader->shader.bc.ngpr;
> +       } else {
> +               num_es_gprs = 0;
> +               num_gs_gprs = 0;
> +               num_vs_gprs = rctx->vs_shader->current->shader.bc.ngpr;
> +       }
> +       new_num_vs_gprs = num_vs_gprs;
> +       new_num_es_gprs = num_es_gprs;
> +       new_num_gs_gprs = num_gs_gprs;
>
>         /* the sum of all SQ_GPR_RESOURCE_MGMT*.NUM_*_GPRS must <= to max_gprs */
> -       if (new_num_ps_gprs > cur_num_ps_gprs || new_num_vs_gprs > cur_num_vs_gprs) {
> +       if (new_num_ps_gprs > cur_num_ps_gprs || new_num_vs_gprs > cur_num_vs_gprs ||
> +           new_num_es_gprs > cur_num_es_gprs || new_num_gs_gprs > cur_num_gs_gprs) {
>                 /* try to use switch back to default */
> -               if (new_num_ps_gprs > def_num_ps_gprs || new_num_vs_gprs > def_num_vs_gprs) {
> +               if (new_num_ps_gprs > def_num_ps_gprs || new_num_vs_gprs > def_num_vs_gprs ||
> +                   new_num_gs_gprs > def_num_gs_gprs || new_num_es_gprs > def_num_es_gprs) {
>                         /* always privilege vs stage so that at worst we have the
>                          * pixel stage producing wrong output (not the vertex
>                          * stage) */
> -                       new_num_ps_gprs = max_gprs - (new_num_vs_gprs + def_num_clause_temp_gprs * 2);
> +                       new_num_ps_gprs = max_gprs - ((new_num_vs_gprs - new_num_es_gprs - new_num_gs_gprs) + def_num_clause_temp_gprs * 2);
>                         new_num_vs_gprs = num_vs_gprs;
> +                       new_num_gs_gprs = num_gs_gprs;
> +                       new_num_es_gprs = num_es_gprs;
>                 } else {
>                         new_num_ps_gprs = def_num_ps_gprs;
>                         new_num_vs_gprs = def_num_vs_gprs;
> +                       new_num_es_gprs = def_num_es_gprs;
> +                       new_num_gs_gprs = def_num_gs_gprs;
>                 }
>         } else {
>                 return true;
> @@ -2359,10 +2452,11 @@ bool r600_adjust_gprs(struct r600_context *rctx)
>          * it will lockup. So in this case just discard the draw command
>          * and don't change the current gprs repartitions.
>          */
> -       if (num_ps_gprs > new_num_ps_gprs || num_vs_gprs > new_num_vs_gprs) {
> -               R600_ERR("ps & vs shader require too many register (%d + %d) "
> +       if (num_ps_gprs > new_num_ps_gprs || num_vs_gprs > new_num_vs_gprs ||
> +           num_gs_gprs > new_num_gs_gprs || num_es_gprs > new_num_es_gprs) {
> +               R600_ERR("shaders require too many register (%d + %d + %d + %d) "
>                          "for a combined maximum of %d\n",
> -                        num_ps_gprs, num_vs_gprs, max_gprs);
> +                        num_ps_gprs, num_vs_gprs, num_es_gprs, num_gs_gprs, max_gprs);
>                 return false;
>         }
>
> @@ -2370,8 +2464,12 @@ bool r600_adjust_gprs(struct r600_context *rctx)
>         tmp = S_008C04_NUM_PS_GPRS(new_num_ps_gprs) |
>                 S_008C04_NUM_VS_GPRS(new_num_vs_gprs) |
>                 S_008C04_NUM_CLAUSE_TEMP_GPRS(def_num_clause_temp_gprs);
> -       if (rctx->config_state.sq_gpr_resource_mgmt_1 != tmp) {
> +
> +       tmp2 = S_008C08_NUM_ES_GPRS(new_num_es_gprs) |
> +               S_008C08_NUM_GS_GPRS(new_num_gs_gprs);
> +       if (rctx->config_state.sq_gpr_resource_mgmt_1 != tmp || rctx->config_state.sq_gpr_resource_mgmt_2 != tmp2) {
>                 rctx->config_state.sq_gpr_resource_mgmt_1 = tmp;
> +               rctx->config_state.sq_gpr_resource_mgmt_2 = tmp2;
>                 rctx->config_state.atom.dirty = true;
>                 rctx->b.flags |= R600_CONTEXT_WAIT_3D_IDLE;
>         }
> @@ -2489,19 +2587,19 @@ void r600_init_atom_start_cs(struct r600_context *rctx)
>                 num_es_stack_entries = 16;
>                 break;
>         case CHIP_RV770:
> -               num_ps_gprs = 192;
> +               num_ps_gprs = 130;
>                 num_vs_gprs = 56;
>                 num_temp_gprs = 4;
> -               num_gs_gprs = 0;
> -               num_es_gprs = 0;
> -               num_ps_threads = 188;
> +               num_gs_gprs = 31;
> +               num_es_gprs = 31;
> +               num_ps_threads = 180;
>                 num_vs_threads = 60;
> -               num_gs_threads = 0;
> -               num_es_threads = 0;
> -               num_ps_stack_entries = 256;
> -               num_vs_stack_entries = 256;
> -               num_gs_stack_entries = 0;
> -               num_es_stack_entries = 0;
> +               num_gs_threads = 4;
> +               num_es_threads = 4;
> +               num_ps_stack_entries = 128;
> +               num_vs_stack_entries = 128;
> +               num_gs_stack_entries = 128;
> +               num_es_stack_entries = 128;
>                 break;
>         case CHIP_RV730:
>         case CHIP_RV740:
> @@ -2510,10 +2608,10 @@ void r600_init_atom_start_cs(struct r600_context *rctx)
>                 num_temp_gprs = 4;
>                 num_gs_gprs = 0;
>                 num_es_gprs = 0;
> -               num_ps_threads = 188;
> +               num_ps_threads = 180;
>                 num_vs_threads = 60;
> -               num_gs_threads = 0;
> -               num_es_threads = 0;
> +               num_gs_threads = 4;
> +               num_es_threads = 4;
>                 num_ps_stack_entries = 128;
>                 num_vs_stack_entries = 128;
>                 num_gs_stack_entries = 0;
> @@ -2525,10 +2623,10 @@ void r600_init_atom_start_cs(struct r600_context *rctx)
>                 num_temp_gprs = 4;
>                 num_gs_gprs = 0;
>                 num_es_gprs = 0;
> -               num_ps_threads = 144;
> +               num_ps_threads = 136;
>                 num_vs_threads = 48;
> -               num_gs_threads = 0;
> -               num_es_threads = 0;
> +               num_gs_threads = 4;
> +               num_es_threads = 4;
>                 num_ps_stack_entries = 128;
>                 num_vs_stack_entries = 128;
>                 num_gs_stack_entries = 0;
> @@ -2704,9 +2802,12 @@ void r600_init_atom_start_cs(struct r600_context *rctx)
>         r600_store_value(cb, 0); /* R_028240_PA_SC_GENERIC_SCISSOR_TL */
>         r600_store_value(cb, S_028244_BR_X(8192) | S_028244_BR_Y(8192)); /* R_028244_PA_SC_GENERIC_SCISSOR_BR */
>
> -       r600_store_context_reg_seq(cb, R_0288CC_SQ_PGM_CF_OFFSET_PS, 2);
> +       r600_store_context_reg_seq(cb, R_0288CC_SQ_PGM_CF_OFFSET_PS, 5);
>         r600_store_value(cb, 0); /* R_0288CC_SQ_PGM_CF_OFFSET_PS */
>         r600_store_value(cb, 0); /* R_0288D0_SQ_PGM_CF_OFFSET_VS */
> +       r600_store_value(cb, 0); /* R_0288D4_SQ_PGM_CF_OFFSET_GS */
> +       r600_store_value(cb, 0); /* R_0288D8_SQ_PGM_CF_OFFSET_ES */
> +       r600_store_value(cb, 0); /* R_0288DC_SQ_PGM_CF_OFFSET_FS */
>
>          r600_store_context_reg(cb, R_0288E0_SQ_VTX_SEMANTIC_CLEAR, ~0);
>
> @@ -2715,7 +2816,6 @@ void r600_init_atom_start_cs(struct r600_context *rctx)
>         r600_store_value(cb, 0); /* R_028404_VGT_MIN_VTX_INDX */
>
>         r600_store_context_reg(cb, R_0288A4_SQ_PGM_RESOURCES_FS, 0);
> -       r600_store_context_reg(cb, R_0288DC_SQ_PGM_CF_OFFSET_FS, 0);
>
>         if (rctx->b.chip_class == R700 && rctx->screen->b.has_streamout)
>                 r600_store_context_reg(cb, R_028354_SX_SURFACE_SYNC, S_028354_SURFACE_SYNC_MASK(0xf));
> @@ -2726,6 +2826,7 @@ void r600_init_atom_start_cs(struct r600_context *rctx)
>
>         r600_store_loop_const(cb, R_03E200_SQ_LOOP_CONST_0, 0x1000FFF);
>         r600_store_loop_const(cb, R_03E200_SQ_LOOP_CONST_0 + (32 * 4), 0x1000FFF);
> +       r600_store_loop_const(cb, R_03E200_SQ_LOOP_CONST_0 + (64 * 4), 0x1000FFF);
>  }
>
>  void r600_update_ps_state(struct pipe_context *ctx, struct r600_pipe_shader *shader)
> @@ -2898,6 +2999,95 @@ void r600_update_vs_state(struct pipe_context *ctx, struct r600_pipe_shader *sha
>                 S_02881C_USE_VTX_POINT_SIZE(rshader->vs_out_point_size);
>  }
>
> +static unsigned r600_conv_prim_to_gs_out(unsigned mode)
> +{
> +       static const int prim_conv[] = {
> +               V_028A6C_OUTPRIM_TYPE_POINTLIST,
> +               V_028A6C_OUTPRIM_TYPE_LINESTRIP,
> +               V_028A6C_OUTPRIM_TYPE_LINESTRIP,
> +               V_028A6C_OUTPRIM_TYPE_LINESTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_LINESTRIP,
> +               V_028A6C_OUTPRIM_TYPE_LINESTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP,
> +               V_028A6C_OUTPRIM_TYPE_TRISTRIP
> +       };
> +       assert(mode < Elements(prim_conv));
> +
> +       return prim_conv[mode];
> +}
> +
> +void r600_update_gs_state(struct pipe_context *ctx, struct r600_pipe_shader *shader)
> +{
> +       struct r600_context *rctx = (struct r600_context *)ctx;
> +       struct r600_command_buffer *cb = &shader->command_buffer;
> +       struct r600_shader *rshader = &shader->shader;
> +       struct r600_shader *cp_shader = &shader->gs_copy_shader->shader;
> +       unsigned gsvs_itemsize =
> +                       (cp_shader->ring_item_size * rshader->gs_max_out_vertices) >> 2;
> +
> +       r600_init_command_buffer(cb, 64);
> +
> +       /* VGT_GS_MODE is written by evergreen_emit_shader_stages */


Update the comment about VGT_GS_MODE.

> +
> +       r600_store_context_reg(cb, R_028AB8_VGT_VTX_CNT_EN, 1);
> +
> +       if (rctx->b.chip_class >= R700) {
> +               r600_store_context_reg(cb, R_028B38_VGT_GS_MAX_VERT_OUT,
> +                                      S_028B38_MAX_VERT_OUT(rshader->gs_max_out_vertices));
> +       }
> +       r600_store_context_reg(cb, R_028A6C_VGT_GS_OUT_PRIM_TYPE,
> +                              r600_conv_prim_to_gs_out(rshader->gs_output_prim));
> +
> +       r600_store_context_reg_seq(cb, R_0288C8_SQ_GS_VERT_ITEMSIZE, 4);
> +       r600_store_value(cb, cp_shader->ring_item_size >> 2);
> +       r600_store_value(cb, 0);
> +       r600_store_value(cb, 0);
> +       r600_store_value(cb, 0);
> +
> +       r600_store_context_reg(cb, R_0288A8_SQ_ESGS_RING_ITEMSIZE,
> +                              (rshader->ring_item_size) >> 2);
> +
> +       r600_store_context_reg(cb, R_0288AC_SQ_GSVS_RING_ITEMSIZE,
> +                              gsvs_itemsize);
> +
> +       /* FIXME calculate these values somehow ??? */
> +       r600_store_config_reg_seq(cb, R_0088C8_VGT_GS_PER_ES, 2);
> +       r600_store_value(cb, 0x80); /* GS_PER_ES */
> +       r600_store_value(cb, 0x100); /* ES_PER_GS */
> +       r600_store_config_reg_seq(cb, R_0088E8_VGT_GS_PER_VS, 1);
> +       r600_store_value(cb, 0x2); /* GS_PER_VS */
> +
> +       r600_store_context_reg(cb, R_02887C_SQ_PGM_RESOURCES_GS,
> +                              S_02887C_NUM_GPRS(rshader->bc.ngpr) |
> +                              S_02887C_STACK_SIZE(rshader->bc.nstack));
> +       r600_store_context_reg(cb, R_02886C_SQ_PGM_START_GS,
> +                              r600_resource_va(ctx->screen, (void *)shader->bo) >> 8);
> +       /* After that, the NOP relocation packet must be emitted (shader->bo, RADEON_USAGE_READ). */
> +}
> +
> +void r600_update_es_state(struct pipe_context *ctx, struct r600_pipe_shader *shader)
> +{
> +       struct r600_command_buffer *cb = &shader->command_buffer;
> +       struct r600_shader *rshader = &shader->shader;
> +
> +       r600_init_command_buffer(cb, 32);
> +
> +       r600_store_context_reg(cb, R_028890_SQ_PGM_RESOURCES_ES,
> +                              S_028890_NUM_GPRS(rshader->bc.ngpr) |
> +                              S_028890_STACK_SIZE(rshader->bc.nstack));
> +       r600_store_context_reg(cb, R_028880_SQ_PGM_START_ES,
> +                              r600_resource_va(ctx->screen, (void *)shader->bo) >> 8);
> +       /* After that, the NOP relocation packet must be emitted (shader->bo, RADEON_USAGE_READ). */
> +}
> +
> +
>  void *r600_create_resolve_blend(struct r600_context *rctx)
>  {
>         struct pipe_blend_state blend;
> @@ -3259,6 +3449,10 @@ void r600_init_state_functions(struct r600_context *rctx)
>         rctx->atoms[id++] = &rctx->b.streamout.begin_atom;
>         r600_init_atom(rctx, &rctx->vertex_shader.atom, id++, r600_emit_shader, 23);
>         r600_init_atom(rctx, &rctx->pixel_shader.atom, id++, r600_emit_shader, 0);
> +       r600_init_atom(rctx, &rctx->geometry_shader.atom, id++, r600_emit_shader, 0);
> +       r600_init_atom(rctx, &rctx->export_shader.atom, id++, r600_emit_shader, 0);
> +       r600_init_atom(rctx, &rctx->shader_stages.atom, id++, r600_emit_shader_stages, 0);
> +       r600_init_atom(rctx, &rctx->gs_rings.atom, id++, r600_emit_gs_rings, 0);
>
>         rctx->b.b.create_blend_state = r600_create_blend_state;
>         rctx->b.b.create_depth_stencil_alpha_state = r600_create_dsa_state;
> diff --git a/src/gallium/drivers/r600/r600_state_common.c b/src/gallium/drivers/r600/r600_state_common.c
> index ffa1eb0..d141026 100644
> --- a/src/gallium/drivers/r600/r600_state_common.c
> +++ b/src/gallium/drivers/r600/r600_state_common.c
> @@ -1162,7 +1162,7 @@ static bool r600_update_derived_state(struct r600_context *rctx)
>                 if (unlikely(!rctx->gs_shader->current))
>                         return false;
>
> -               if (rctx->b.chip_class >= EVERGREEN && !rctx->shader_stages.geom_enable) {
> +               if (!rctx->shader_stages.geom_enable) {
>                         rctx->shader_stages.geom_enable = true;
>                         rctx->shader_stages.atom.dirty = true;
>                 }
> diff --git a/src/gallium/drivers/r600/r600d.h b/src/gallium/drivers/r600/r600d.h
> index 9ec45c6..05d1f0a 100644
> --- a/src/gallium/drivers/r600/r600d.h
> +++ b/src/gallium/drivers/r600/r600d.h
> @@ -201,6 +201,19 @@
>  /* Registers */
>  #define R_008490_CP_STRMOUT_CNTL                    0x008490
>  #define   S_008490_OFFSET_UPDATE_DONE(x)               (((x) & 0x1) << 0)
> +#define R_008C40_SQ_ESGS_RING_BASE                   0x008C40
> +#define R_008C44_SQ_ESGS_RING_SIZE                   0x008C44
> +#define R_008C48_SQ_GSVS_RING_BASE                   0x008C48
> +#define R_008C4C_SQ_GSVS_RING_SIZE                   0x008C4C
> +#define R_008C50_SQ_ESTMP_RING_BASE                  0x008C50
> +#define R_008C54_SQ_ESTMP_RING_SIZE                  0x008C54
> +#define R_008C50_SQ_GSTMP_RING_BASE                  0x008C58
> +#define R_008C54_SQ_GSTMP_RING_SIZE                  0x008C5C
> +
> +#define R_0088C8_VGT_GS_PER_ES                       0x0088C8
> +#define R_0088CC_VGT_ES_PER_GS                       0x0088CC
> +#define R_0088E8_VGT_GS_PER_VS                       0x0088E8
> +
>  #define R_008960_VGT_STRMOUT_BUFFER_FILLED_SIZE_0    0x008960 /* read-only */
>  #define R_008964_VGT_STRMOUT_BUFFER_FILLED_SIZE_1    0x008964 /* read-only */
>  #define R_008968_VGT_STRMOUT_BUFFER_FILLED_SIZE_2    0x008968 /* read-only */
> @@ -1825,12 +1838,20 @@
>  #define   S_028A40_MODE(x)                             (((x) & 0x3) << 0)
>  #define   G_028A40_MODE(x)                             (((x) >> 0) & 0x3)
>  #define   C_028A40_MODE                                0xFFFFFFFC
> +#define     V_028A40_GS_OFF                            0
> +#define     V_028A40_GS_SCENARIO_A                     1
> +#define     V_028A40_GS_SCENARIO_B                     2
> +#define     V_028A40_GS_SCENARIO_G                     3
>  #define   S_028A40_ES_PASSTHRU(x)                      (((x) & 0x1) << 2)
>  #define   G_028A40_ES_PASSTHRU(x)                      (((x) >> 2) & 0x1)
>  #define   C_028A40_ES_PASSTHRU                         0xFFFFFFFB
>  #define   S_028A40_CUT_MODE(x)                         (((x) & 0x3) << 3)
>  #define   G_028A40_CUT_MODE(x)                         (((x) >> 3) & 0x3)
>  #define   C_028A40_CUT_MODE                            0xFFFFFFE7
> +#define     V_028A40_GS_CUT_1024                       0
> +#define     V_028A40_GS_CUT_512                        1
> +#define     V_028A40_GS_CUT_256                        2
> +#define     V_028A40_GS_CUT_128                        3
>  #define R_008DFC_SQ_CF_WORD0                         0x008DFC
>  #define   S_008DFC_ADDR(x)                             (((x) & 0xFFFFFFFF) << 0)
>  #define   G_008DFC_ADDR(x)                             (((x) >> 0) & 0xFFFFFFFF)
> @@ -2333,6 +2354,26 @@
>  #define   S_028D44_ALPHA_TO_MASK_OFFSET3(x)            (((x) & 0x3) << 14)
>  #define   S_028D44_OFFSET_ROUND(x)                     (((x) & 0x1) << 16)
>  #define R_028868_SQ_PGM_RESOURCES_VS                 0x028868
> +#define R_028890_SQ_PGM_RESOURCES_ES                 0x028890
> +#define   S_028890_NUM_GPRS(x)                         (((x) & 0xFF) << 0)
> +#define   G_028890_NUM_GPRS(x)                         (((x) >> 0) & 0xFF)
> +#define   C_028890_NUM_GPRS                            0xFFFFFF00
> +#define   S_028890_STACK_SIZE(x)                       (((x) & 0xFF) << 8)
> +#define   G_028890_STACK_SIZE(x)                       (((x) >> 8) & 0xFF)
> +#define   C_028890_STACK_SIZE                          0xFFFF00FF
> +#define   S_028890_DX10_CLAMP(x)                       (((x) & 0x1) << 21)
> +#define   G_028890_DX10_CLAMP(x)                       (((x) >> 21) & 0x1)
> +#define   C_028890_DX10_CLAMP                          0xFFDFFFFF
> +#define R_02887C_SQ_PGM_RESOURCES_GS                 0x02887C
> +#define   S_02887C_NUM_GPRS(x)                         (((x) & 0xFF) << 0)
> +#define   G_02887C_NUM_GPRS(x)                         (((x) >> 0) & 0xFF)
> +#define   C_02887C_NUM_GPRS                            0xFFFFFF00
> +#define   S_02887C_STACK_SIZE(x)                       (((x) & 0xFF) << 8)
> +#define   G_02887C_STACK_SIZE(x)                       (((x) >> 8) & 0xFF)
> +#define   C_02887C_STACK_SIZE                          0xFFFF00FF
> +#define   S_02887C_DX10_CLAMP(x)                       (((x) & 0x1) << 21)
> +#define   G_02887C_DX10_CLAMP(x)                       (((x) >> 21) & 0x1)
> +#define   C_02887C_DX10_CLAMP                          0xFFDFFFFF
>  #define R_0286CC_SPI_PS_IN_CONTROL_0                 0x0286CC
>  #define R_0286D0_SPI_PS_IN_CONTROL_1                 0x0286D0
>  #define R_028644_SPI_PS_INPUT_CNTL_0                 0x028644
> @@ -2422,11 +2463,15 @@
>  #define   G_028C04_MAX_SAMPLE_DIST(x)                  (((x) >> 13) & 0xF)
>  #define   C_028C04_MAX_SAMPLE_DIST                     0xFFFE1FFF
>  #define R_0288CC_SQ_PGM_CF_OFFSET_PS                 0x0288CC
> -#define R_0288DC_SQ_PGM_CF_OFFSET_FS                 0x0288DC
>  #define R_0288D0_SQ_PGM_CF_OFFSET_VS                 0x0288D0
> +#define R_0288D4_SQ_PGM_CF_OFFSET_GS                 0x0288D4
> +#define R_0288D8_SQ_PGM_CF_OFFSET_ES                 0x0288D8
> +#define R_0288DC_SQ_PGM_CF_OFFSET_FS                 0x0288DC
>  #define R_028840_SQ_PGM_START_PS                     0x028840
>  #define R_028894_SQ_PGM_START_FS                     0x028894
>  #define R_028858_SQ_PGM_START_VS                     0x028858
> +#define R_02886C_SQ_PGM_START_GS                     0x02886C
> +#define R_028880_SQ_PGM_START_ES                     0x028880
>  #define R_028080_CB_COLOR0_VIEW                      0x028080
>  #define   S_028080_SLICE_START(x)                      (((x) & 0x7FF) << 0)
>  #define   G_028080_SLICE_START(x)                      (((x) >> 0) & 0x7FF)
> @@ -2864,6 +2909,7 @@
>  #define R_0283F4_SQ_VTX_SEMANTIC_29                  0x0283F4
>  #define R_0283F8_SQ_VTX_SEMANTIC_30                  0x0283F8
>  #define R_0283FC_SQ_VTX_SEMANTIC_31                  0x0283FC
> +#define R_0288C8_SQ_GS_VERT_ITEMSIZE                 0x0288C8
>  #define R_0288E0_SQ_VTX_SEMANTIC_CLEAR               0x0288E0
>  #define R_028400_VGT_MAX_VTX_INDX                    0x028400
>  #define   S_028400_MAX_INDX(x)                         (((x) & 0xFFFFFFFF) << 0)
> @@ -3288,6 +3334,8 @@
>  #define R_028B28_VGT_STRMOUT_DRAW_OPAQUE_OFFSET             0x028B28
>  #define R_028B2C_VGT_STRMOUT_DRAW_OPAQUE_BUFFER_FILLED_SIZE 0x028B2C
>  #define R_028B30_VGT_STRMOUT_DRAW_OPAQUE_VERTEX_STRIDE 0x028B30
> +#define R_028B38_VGT_GS_MAX_VERT_OUT                 0x028B38 /* r7xx */
> +#define   S_028B38_MAX_VERT_OUT(x)                      (((x) & 0x7FF) << 0)
>  #define R_028B44_VGT_STRMOUT_BASE_OFFSET_HI_0       0x028B44
>  #define R_028B48_VGT_STRMOUT_BASE_OFFSET_HI_1       0x028B48
>  #define R_028B4C_VGT_STRMOUT_BASE_OFFSET_HI_2       0x028B4C
> --
> 1.8.3.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list