diff options
author | Rob Clark <[email protected]> | 2017-11-20 15:33:54 -0500 |
---|---|---|
committer | Rob Clark <[email protected]> | 2017-12-03 14:17:41 -0500 |
commit | 4b1d0d28449d03539db452e92cc29a9f5dd968d7 (patch) | |
tree | c683af29dd4b48b1a93d47df3a296eed224341e7 /src | |
parent | 91730fb0ffc77682cfd5cebbefdad2f8f92f2431 (diff) |
freedreno: small cleanups
Signed-off-by: Rob Clark <[email protected]>
Diffstat (limited to 'src')
-rw-r--r-- | src/gallium/drivers/freedreno/freedreno_resource.c | 25 |
1 files changed, 8 insertions, 17 deletions
diff --git a/src/gallium/drivers/freedreno/freedreno_resource.c b/src/gallium/drivers/freedreno/freedreno_resource.c index 8ba43a10d71..0de7efafd04 100644 --- a/src/gallium/drivers/freedreno/freedreno_resource.c +++ b/src/gallium/drivers/freedreno/freedreno_resource.c @@ -118,7 +118,7 @@ static void do_blit(struct fd_context *ctx, const struct pipe_blit_info *blit, bool fallback) { /* TODO size threshold too?? */ - if ((blit->src.resource->target != PIPE_BUFFER) && !fallback) { + if (!fallback) { /* do blit on gpu: */ fd_blitter_pipe_begin(ctx, false, true, FD_STAGE_BLIT); util_blitter_blit(ctx->blitter, blit); @@ -134,7 +134,7 @@ do_blit(struct fd_context *ctx, const struct pipe_blit_info *blit, bool fallback static bool fd_try_shadow_resource(struct fd_context *ctx, struct fd_resource *rsc, - unsigned level, unsigned usage, const struct pipe_box *box) + unsigned level, const struct pipe_box *box) { struct pipe_context *pctx = &ctx->base; struct pipe_resource *prsc = &rsc->base.b; @@ -151,19 +151,9 @@ fd_try_shadow_resource(struct fd_context *ctx, struct fd_resource *rsc, PIPE_BIND_RENDER_TARGET)) fallback = true; - /* these cases should be handled elsewhere.. just for future - * reference in case this gets split into a more generic(ish) - * helper. - */ - debug_assert(!(usage & PIPE_TRANSFER_READ)); - debug_assert(!(usage & PIPE_TRANSFER_DISCARD_WHOLE_RESOURCE)); - - /* if we do a gpu blit to clone the whole resource, we'll just - * end up stalling on that.. so only allow if we can discard - * current range (and blit, possibly cpu or gpu, the rest) - */ - if (!(usage & PIPE_TRANSFER_DISCARD_RANGE)) - return false; + /* do shadowing back-blits on the cpu for buffers: */ + if (prsc->target == PIPE_BUFFER) + fallback = true; bool whole_level = util_texrange_covers_whole_level(prsc, level, box->x, box->y, box->z, box->width, box->height, box->depth); @@ -522,8 +512,9 @@ fd_resource_transfer_map(struct pipe_context *pctx, * ie. we only *don't* want to go down this path if the blit * will trigger a flush! */ - if (ctx->screen->reorder && busy && !(usage & PIPE_TRANSFER_READ)) { - if (fd_try_shadow_resource(ctx, rsc, level, usage, box)) { + if (ctx->screen->reorder && busy && !(usage & PIPE_TRANSFER_READ) && + (usage & PIPE_TRANSFER_DISCARD_RANGE)) { + if (fd_try_shadow_resource(ctx, rsc, level, box)) { needs_flush = busy = false; rebind_resource(ctx, prsc); } |