From d60375cbc2510ab7ad90b2654c0f6324468415cf Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Mon, 16 Mar 2020 22:58:53 -0500 Subject: [PATCH] anv: Do an end-of-pipe sync before updating AUX table entries We've found in GL that an actual end-of-pipe sync is required before invalidating the aux tables and that a simple CS stall is insufficient. If we're about to modify the actual AUX table entries from the GPU, we should definitely make sure it's stopped dead before we do so. Cc: mesa-stable@lists.freedesktop.org Reviewed-by: Rafael Antognolli Tested-by: Marge Bot Part-of: --- src/intel/vulkan/genX_cmd_buffer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index a37bc96b216..0d78b1c0947 100644 --- a/src/intel/vulkan/genX_cmd_buffer.c +++ b/src/intel/vulkan/genX_cmd_buffer.c @@ -1021,7 +1021,7 @@ anv_image_init_aux_tt(struct anv_cmd_buffer *cmd_buffer, * with not having this stall in some cases if we were really careful but * it's better to play it safe. Full stall the GPU. */ - cmd_buffer->state.pending_pipe_bits |= ANV_PIPE_CS_STALL_BIT; + cmd_buffer->state.pending_pipe_bits |= ANV_PIPE_END_OF_PIPE_SYNC_BIT; genX(cmd_buffer_apply_pipe_flushes)(cmd_buffer); struct gen_mi_builder b; -- 2.30.2