From 3d3194e93cbfe0fc42dd96218e8e999f5ccfe726 Mon Sep 17 00:00:00 2001 From: Tom Stellard Date: Tue, 26 Jun 2012 20:54:32 +0000 Subject: [PATCH] r600g: Add r600_context_pipe_state_emit() This function is used when dispatching compute shader in order to avoid mixing compute and 3D registers in the context's dirty list. This allows the compute code to resuse 3D functions like evergreen_cb, which return a struct r600_pipe_state and still have control over when and how the register writes are emitted. --- src/gallium/drivers/r600/r600.h | 1 + src/gallium/drivers/r600/r600_hw_context.c | 76 ++++++++++++++++++++-- 2 files changed, 71 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers/r600/r600.h b/src/gallium/drivers/r600/r600.h index d4e01d9694e..0ae79598554 100644 --- a/src/gallium/drivers/r600/r600.h +++ b/src/gallium/drivers/r600/r600.h @@ -197,6 +197,7 @@ struct r600_screen; void r600_get_backend_mask(struct r600_context *ctx); int r600_context_init(struct r600_context *ctx); void r600_context_fini(struct r600_context *ctx); +void r600_context_pipe_state_emit(struct r600_context *ctx, struct r600_pipe_state *state, unsigned pkt_flags); void r600_context_pipe_state_set(struct r600_context *ctx, struct r600_pipe_state *state); void r600_context_pipe_state_set_ps_resource(struct r600_context *ctx, struct r600_pipe_resource_state *state, unsigned rid); void r600_context_pipe_state_set_vs_resource(struct r600_context *ctx, struct r600_pipe_resource_state *state, unsigned rid); diff --git a/src/gallium/drivers/r600/r600_hw_context.c b/src/gallium/drivers/r600/r600_hw_context.c index 77c35538a29..cbb89143f2a 100644 --- a/src/gallium/drivers/r600/r600_hw_context.c +++ b/src/gallium/drivers/r600/r600_hw_context.c @@ -836,12 +836,80 @@ void r600_context_dirty_block(struct r600_context *ctx, } } +/** + * If reg needs a reloc, this function will add it to its block's reloc list. + * @return true if reg needs a reloc, false otherwise + */ +static bool r600_reg_set_block_reloc(struct r600_pipe_reg *reg) +{ + unsigned reloc_id; + + if (!reg->block->pm4_bo_index[reg->id]) { + return false; + } + /* find relocation */ + reloc_id = reg->block->pm4_bo_index[reg->id]; + pipe_resource_reference( + (struct pipe_resource**)®->block->reloc[reloc_id].bo, + ®->bo->b.b); + reg->block->reloc[reloc_id].bo_usage = reg->bo_usage; + return true; +} + +/** + * This function will emit all the registers in state directly to the command + * stream allowing you to bypass the r600_context dirty list. + * + * This is used for dispatching compute shaders to avoid mixing compute and + * 3D states in the context's dirty list. + * + * @param pkt_flags Should be either 0 or RADEON_CP_PACKET3_COMPUTE_MODE. This + * value will be passed on to r600_context_block_emit_dirty an or'd against + * the PKT3 headers. + */ +void r600_context_pipe_state_emit(struct r600_context *ctx, + struct r600_pipe_state *state, + unsigned pkt_flags) +{ + unsigned i; + + /* Mark all blocks as dirty: + * Since two registers can be in the same block, we need to make sure + * we mark all the blocks dirty before we emit any of them. If we were + * to mark blocks dirty and emit them in the same loop, like this: + * + * foreach (reg in state->regs) { + * mark_dirty(reg->block) + * emit_block(reg->block) + * } + * + * Then if we have two registers in this state that are in the same + * block, we would end up emitting that block twice. + */ + for (i = 0; i < state->nregs; i++) { + struct r600_pipe_reg *reg = &state->regs[i]; + /* Mark all the registers in the block as dirty */ + reg->block->nreg_dirty = reg->block->nreg; + reg->block->status |= R600_BLOCK_STATUS_DIRTY; + /* Update the reloc for this register if necessary. */ + r600_reg_set_block_reloc(reg); + } + + /* Emit the registers writes */ + for (i = 0; i < state->nregs; i++) { + struct r600_pipe_reg *reg = &state->regs[i]; + if (reg->block->status & R600_BLOCK_STATUS_DIRTY) { + r600_context_block_emit_dirty(ctx, reg->block, pkt_flags); + } + } +} + void r600_context_pipe_state_set(struct r600_context *ctx, struct r600_pipe_state *state) { struct r600_block *block; int dirty; for (int i = 0; i < state->nregs; i++) { - unsigned id, reloc_id; + unsigned id; struct r600_pipe_reg *reg = &state->regs[i]; block = reg->block; @@ -855,11 +923,7 @@ void r600_context_pipe_state_set(struct r600_context *ctx, struct r600_pipe_stat } if (block->flags & REG_FLAG_DIRTY_ALWAYS) dirty |= R600_BLOCK_STATUS_DIRTY; - if (block->pm4_bo_index[id]) { - /* find relocation */ - reloc_id = block->pm4_bo_index[id]; - pipe_resource_reference((struct pipe_resource**)&block->reloc[reloc_id].bo, ®->bo->b.b); - block->reloc[reloc_id].bo_usage = reg->bo_usage; + if (r600_reg_set_block_reloc(reg)) { /* always force dirty for relocs for now */ dirty |= R600_BLOCK_STATUS_DIRTY; } -- 2.30.2