freedreno: rebind_resource() *before* bo changes
This will matter in the next patch, where we need the original rsc->seqno. It means slight shuffling of where we call rebind_resource() in the `fd_try_shadow_resource()` path. Signed-off-by: Rob Clark <robdclark@chromium.org> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4744>
This commit is contained in:
parent
d9e56d8a69
commit
ca05e6b04d
|
@ -264,6 +264,7 @@ fd_try_shadow_resource(struct fd_context *ctx, struct fd_resource *rsc,
|
|||
* should empty/destroy rsc->batches hashset)
|
||||
*/
|
||||
fd_bc_invalidate_resource(rsc, false);
|
||||
rebind_resource(rsc);
|
||||
|
||||
mtx_lock(&ctx->screen->lock);
|
||||
|
||||
|
@ -386,8 +387,6 @@ fd_resource_uncompress(struct fd_context *ctx, struct fd_resource *rsc)
|
|||
|
||||
/* shadow should not fail in any cases where we need to uncompress: */
|
||||
debug_assert(success);
|
||||
|
||||
rebind_resource(rsc);
|
||||
}
|
||||
|
||||
static struct fd_resource *
|
||||
|
@ -635,8 +634,8 @@ fd_resource_transfer_map(struct pipe_context *pctx,
|
|||
|
||||
if (usage & PIPE_TRANSFER_DISCARD_WHOLE_RESOURCE) {
|
||||
if (needs_flush || fd_resource_busy(rsc, op)) {
|
||||
realloc_bo(rsc, fd_bo_size(rsc->bo));
|
||||
rebind_resource(rsc);
|
||||
realloc_bo(rsc, fd_bo_size(rsc->bo));
|
||||
}
|
||||
} else if ((usage & PIPE_TRANSFER_WRITE) &&
|
||||
prsc->target == PIPE_BUFFER &&
|
||||
|
@ -679,7 +678,6 @@ fd_resource_transfer_map(struct pipe_context *pctx,
|
|||
if (needs_flush && fd_try_shadow_resource(ctx, rsc, level,
|
||||
box, DRM_FORMAT_MOD_LINEAR)) {
|
||||
needs_flush = busy = false;
|
||||
rebind_resource(rsc);
|
||||
ctx->stats.shadow_uploads++;
|
||||
} else {
|
||||
struct fd_resource *staging_rsc;
|
||||
|
|
Loading…
Reference in New Issue