summaryrefslogtreecommitdiffstats
path: root/src/gallium/drivers/etnaviv
diff options
context:
space:
mode:
authorLucas Stach <[email protected]>2017-09-26 18:27:55 +0200
committerLucas Stach <[email protected]>2017-10-10 12:29:03 +0200
commit03b1f8ba205d3eff59784faf805272a4d7aa4b18 (patch)
tree0eaa06e65e402256264f831f5251e3dc7ceab749 /src/gallium/drivers/etnaviv
parent5ec21eb1a0c9fa08978784e58456fdd212aab4d7 (diff)
etnaviv: fix bogus flush requests in transfer handling
The logic to decide if we need to flush the GPU command stream was broken and hard to reason about. Fix and clarify this. Fixes the data sync subtests from piglit arb_vertex_buffer_object. Signed-off-by: Lucas Stach <[email protected]> Reviewed-by: Wladimir J. van der Laan <[email protected]>
Diffstat (limited to 'src/gallium/drivers/etnaviv')
-rw-r--r--src/gallium/drivers/etnaviv/etnaviv_transfer.c15
1 files changed, 10 insertions, 5 deletions
diff --git a/src/gallium/drivers/etnaviv/etnaviv_transfer.c b/src/gallium/drivers/etnaviv/etnaviv_transfer.c
index ee5cda5e8ec..08ec1987d8f 100644
--- a/src/gallium/drivers/etnaviv/etnaviv_transfer.c
+++ b/src/gallium/drivers/etnaviv/etnaviv_transfer.c
@@ -243,11 +243,16 @@ etna_transfer_map(struct pipe_context *pctx, struct pipe_resource *prsc,
struct etna_resource_level *res_level = &rsc->levels[level];
- /* Always sync if we have the temporary resource. The PIPE_TRANSFER_READ
- * case could be optimised if we knew whether the resource has outstanding
- * rendering. */
- if ((usage & PIPE_TRANSFER_READ || trans->rsc) &&
- rsc->status & ETNA_PENDING_WRITE)
+ /*
+ * Always flush if we have the temporary resource and have a copy to this
+ * outstanding. Otherwise infer flush requirement from resource access and
+ * current GPU usage (reads must wait for GPU writes, writes must have
+ * exclusive access to the buffer).
+ */
+ if ((trans->rsc && (etna_resource(trans->rsc)->status & ETNA_PENDING_WRITE)) ||
+ (!trans->rsc &&
+ (((usage & PIPE_TRANSFER_READ) && (rsc->status & ETNA_PENDING_WRITE)) ||
+ ((usage & PIPE_TRANSFER_WRITE) && rsc->status))))
pctx->flush(pctx, NULL, 0);
/* XXX we don't handle PIPE_TRANSFER_FLUSH_EXPLICIT; this flag can be ignored