freedreno: fix indexbuffer upload

My fault for not having time to test Marek's patches while they were on
list.

Fixes: 330d0607 ("gallium: remove pipe_index_buffer and set_index_buffer")
Signed-off-by: Rob Clark <robdclark@gmail.com>
This commit is contained in:
Rob Clark 2017-05-13 14:43:45 -04:00
parent d4e4c36c7c
commit 06a51fb4e5
6 changed files with 26 additions and 15 deletions

View File

@ -109,7 +109,7 @@ fd2_draw_vbo(struct fd_context *ctx, const struct pipe_draw_info *info,
OUT_RING(ring, info->min_index); /* VGT_MIN_VTX_INDX */ OUT_RING(ring, info->min_index); /* VGT_MIN_VTX_INDX */
fd_draw_emit(ctx->batch, ring, ctx->primtypes[info->mode], fd_draw_emit(ctx->batch, ring, ctx->primtypes[info->mode],
IGNORE_VISIBILITY, info); IGNORE_VISIBILITY, info, index_offset);
OUT_PKT3(ring, CP_SET_CONSTANT, 2); OUT_PKT3(ring, CP_SET_CONSTANT, 2);
OUT_RING(ring, CP_REG(REG_A2XX_UNKNOWN_2010)); OUT_RING(ring, CP_REG(REG_A2XX_UNKNOWN_2010));

View File

@ -55,7 +55,7 @@ add_sat(uint32_t a, int32_t b)
static void static void
draw_impl(struct fd_context *ctx, struct fd_ringbuffer *ring, draw_impl(struct fd_context *ctx, struct fd_ringbuffer *ring,
struct fd3_emit *emit) struct fd3_emit *emit, unsigned index_offset)
{ {
const struct pipe_draw_info *info = emit->info; const struct pipe_draw_info *info = emit->info;
enum pc_di_primtype primtype = ctx->primtypes[info->mode]; enum pc_di_primtype primtype = ctx->primtypes[info->mode];
@ -86,7 +86,7 @@ draw_impl(struct fd_context *ctx, struct fd_ringbuffer *ring,
fd_draw_emit(ctx->batch, ring, primtype, fd_draw_emit(ctx->batch, ring, primtype,
emit->key.binning_pass ? IGNORE_VISIBILITY : USE_VISIBILITY, emit->key.binning_pass ? IGNORE_VISIBILITY : USE_VISIBILITY,
info); info, index_offset);
} }
/* fixup dirty shader state in case some "unrelated" (from the state- /* fixup dirty shader state in case some "unrelated" (from the state-
@ -157,14 +157,14 @@ fd3_draw_vbo(struct fd_context *ctx, const struct pipe_draw_info *info,
emit.key.binning_pass = false; emit.key.binning_pass = false;
emit.dirty = dirty; emit.dirty = dirty;
draw_impl(ctx, ctx->batch->draw, &emit); draw_impl(ctx, ctx->batch->draw, &emit, index_offset);
/* and now binning pass: */ /* and now binning pass: */
emit.key.binning_pass = true; emit.key.binning_pass = true;
emit.dirty = dirty & ~(FD_DIRTY_BLEND); emit.dirty = dirty & ~(FD_DIRTY_BLEND);
emit.vp = NULL; /* we changed key so need to refetch vp */ emit.vp = NULL; /* we changed key so need to refetch vp */
emit.fp = NULL; emit.fp = NULL;
draw_impl(ctx, ctx->batch->binning, &emit); draw_impl(ctx, ctx->batch->binning, &emit, index_offset);
fd_context_all_clean(ctx); fd_context_all_clean(ctx);

View File

@ -105,7 +105,7 @@ fd4_draw_emit(struct fd_batch *batch, struct fd_ringbuffer *ring,
enum pc_di_primtype primtype, enum pc_di_primtype primtype,
enum pc_di_vis_cull_mode vismode, enum pc_di_vis_cull_mode vismode,
const struct pipe_draw_info *info, const struct pipe_draw_info *info,
unsigned index_offset) unsigned index_offset)
{ {
struct pipe_resource *idx_buffer = NULL; struct pipe_resource *idx_buffer = NULL;
enum a4xx_index_size idx_type; enum a4xx_index_size idx_type;

View File

@ -81,7 +81,7 @@ fd5_draw_emit(struct fd_batch *batch, struct fd_ringbuffer *ring,
enum pc_di_primtype primtype, enum pc_di_primtype primtype,
enum pc_di_vis_cull_mode vismode, enum pc_di_vis_cull_mode vismode,
const struct pipe_draw_info *info, const struct pipe_draw_info *info,
unsigned index_offset) unsigned index_offset)
{ {
struct pipe_resource *idx_buffer = NULL; struct pipe_resource *idx_buffer = NULL;
enum a4xx_index_size idx_type; enum a4xx_index_size idx_type;

View File

@ -91,11 +91,20 @@ fd_draw_vbo(struct pipe_context *pctx, const struct pipe_draw_info *info)
} }
/* Upload a user index buffer. */ /* Upload a user index buffer. */
struct pipe_resource *indexbuf = info->has_user_indices ? NULL : info->index.resource; struct pipe_resource *indexbuf = NULL;
unsigned index_offset = 0; unsigned index_offset = 0;
if (info->index_size && info->has_user_indices && struct pipe_draw_info new_info;
!util_upload_index_buffer(pctx, info, &indexbuf, &index_offset)) { if (info->index_size) {
return; if (info->has_user_indices) {
if (!util_upload_index_buffer(pctx, info, &indexbuf, &index_offset))
return;
new_info = *info;
new_info.index.resource = indexbuf;
new_info.has_user_indices = false;
info = &new_info;
} else {
indexbuf = info->index.resource;
}
} }
if (ctx->in_blit) { if (ctx->in_blit) {
@ -224,7 +233,8 @@ fd_draw_vbo(struct pipe_context *pctx, const struct pipe_draw_info *info)
fd_context_all_dirty(ctx); fd_context_all_dirty(ctx);
fd_batch_check_size(batch); fd_batch_check_size(batch);
if (info->index_size && indexbuf != info->index.resource)
if (info == &new_info)
pipe_resource_reference(&indexbuf, NULL); pipe_resource_reference(&indexbuf, NULL);
} }

View File

@ -115,7 +115,8 @@ static inline void
fd_draw_emit(struct fd_batch *batch, struct fd_ringbuffer *ring, fd_draw_emit(struct fd_batch *batch, struct fd_ringbuffer *ring,
enum pc_di_primtype primtype, enum pc_di_primtype primtype,
enum pc_di_vis_cull_mode vismode, enum pc_di_vis_cull_mode vismode,
const struct pipe_draw_info *info) const struct pipe_draw_info *info,
unsigned index_offset)
{ {
struct pipe_resource *idx_buffer = NULL; struct pipe_resource *idx_buffer = NULL;
enum pc_di_index_size idx_type = INDEX_SIZE_IGN; enum pc_di_index_size idx_type = INDEX_SIZE_IGN;
@ -128,7 +129,7 @@ fd_draw_emit(struct fd_batch *batch, struct fd_ringbuffer *ring,
idx_buffer = info->index.resource; idx_buffer = info->index.resource;
idx_type = size2indextype(info->index_size); idx_type = size2indextype(info->index_size);
idx_size = info->index_size * info->count; idx_size = info->index_size * info->count;
idx_offset = info->start * info->index_size; idx_offset = index_offset + info->start * info->index_size;
src_sel = DI_SRC_SEL_DMA; src_sel = DI_SRC_SEL_DMA;
} else { } else {
idx_buffer = NULL; idx_buffer = NULL;