From 4f338c9bbff090d606afdc22373cc7869b0d0c89 Mon Sep 17 00:00:00 2001 From: Rob Clark Date: Mon, 8 Sep 2014 11:18:01 -0400 Subject: [PATCH] freedreno: don't overflow cmdstream buffer so much We currently aren't too clever about dealing with running out of cmdstream buffer space. Since we use a single buffer for both drawing and tiling commands, we need to ensure there is enough space at the tail of the cmdstream buffer to fit the tiling commands. Until we get more clever, the easy solution is a threshold to trigger flushing rendering even if the application does not trigger flush (swap, changing render target, etc). This way we at least don't crash for apps that do several thousand draw calls (like some piglit tests do). Signed-off-by: Rob Clark --- src/gallium/drivers/freedreno/freedreno_draw.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/gallium/drivers/freedreno/freedreno_draw.c b/src/gallium/drivers/freedreno/freedreno_draw.c index e3c8cc8e5a0..e4634958ba9 100644 --- a/src/gallium/drivers/freedreno/freedreno_draw.c +++ b/src/gallium/drivers/freedreno/freedreno_draw.c @@ -157,8 +157,23 @@ fd_draw_vbo(struct pipe_context *pctx, const struct pipe_draw_info *info) /* and any buffers used, need to be resolved: */ ctx->resolve |= buffers; + DBG("%x num_draws=%u (%s/%s)", buffers, ctx->num_draws, + util_format_short_name(pipe_surface_format(pfb->cbufs[0])), + util_format_short_name(pipe_surface_format(pfb->zsbuf))); + fd_hw_query_set_stage(ctx, ctx->ring, FD_STAGE_DRAW); ctx->draw(ctx, info); + + /* if an app (or, well, piglit test) does many thousands of draws + * without flush (or anything which implicitly flushes, like + * changing render targets), we can exceed the ringbuffer size. + * Since we don't currently have a sane way to wrapparound, and + * we use the same buffer for both draw and tiling commands, for + * now we need to do this hack and trigger flush if we are running + * low on remaining space for cmds: + */ + if ((ctx->ring->cur - ctx->ring->start) > ctx->ring->size/8) + fd_context_render(pctx); } /* TODO figure out how to make better use of existing state mechanism -- 2.30.2