diff options
author | Ilia Mirkin <[email protected]> | 2015-04-02 22:28:21 -0400 |
---|---|---|
committer | Rob Clark <[email protected]> | 2015-04-05 16:36:34 -0400 |
commit | ea0952a9db1b5887f915d8f750f5fa9c45719976 (patch) | |
tree | 1f58dadc4645a6772cff3e9115e8237658082b8c | |
parent | bfb0a8eb6967065be92e40ba620fc6fededde51a (diff) |
freedreno: fix resource flushing confusion
A resource flush is an upload of a hypothetically-staging texture to the
GPU. For a UMA system, this will largely be a no-op or
cache-maintenance. Move the render flush logic into transfer_map where
it belongs, and clear out the transfer_flush function.
Signed-off-by: Ilia Mirkin <[email protected]>
-rw-r--r-- | src/gallium/drivers/freedreno/freedreno_resource.c | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/src/gallium/drivers/freedreno/freedreno_resource.c b/src/gallium/drivers/freedreno/freedreno_resource.c index efafb8988b4..407a3528aaa 100644 --- a/src/gallium/drivers/freedreno/freedreno_resource.c +++ b/src/gallium/drivers/freedreno/freedreno_resource.c @@ -64,16 +64,6 @@ static void fd_resource_transfer_flush_region(struct pipe_context *pctx, struct pipe_transfer *ptrans, const struct pipe_box *box) { - struct fd_context *ctx = fd_context(pctx); - struct fd_resource *rsc = fd_resource(ptrans->resource); - - if (rsc->dirty) - fd_context_render(pctx); - - if (rsc->timestamp) { - fd_pipe_wait(ctx->screen->pipe, rsc->timestamp); - rsc->timestamp = 0; - } } static void @@ -127,13 +117,19 @@ fd_resource_transfer_map(struct pipe_context *pctx, if (usage & PIPE_TRANSFER_WRITE) op |= DRM_FREEDRENO_PREP_WRITE; - /* some state trackers (at least XA) don't do this.. */ - if (!(usage & (PIPE_TRANSFER_FLUSH_EXPLICIT | PIPE_TRANSFER_DISCARD_WHOLE_RESOURCE))) - fd_resource_transfer_flush_region(pctx, ptrans, box); - if (usage & PIPE_TRANSFER_DISCARD_WHOLE_RESOURCE) { realloc_bo(rsc, fd_bo_size(rsc->bo)); } else if (!(usage & PIPE_TRANSFER_UNSYNCHRONIZED)) { + /* If the GPU is writing to the resource, or if it is reading from the + * resource and we're trying to write to it, flush the renders. + */ + if (rsc->dirty) + fd_context_render(pctx); + + /* The GPU keeps track of how the various bo's are being used, and + * will wait if necessary for the proper operation to have + * completed. + */ ret = fd_bo_cpu_prep(rsc->bo, ctx->screen->pipe, op); if (ret) goto fail; |