From: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Date: Thu, 1 Mar 2018 09:53:49 +0000 (+0100)
Subject: radv: do not set pending_reset_query in BeginCommandBuffer()
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c133a3411bbf47c2ba7d9cdae7e35a64fe276068;p=mesa.git

radv: do not set pending_reset_query in BeginCommandBuffer()

This is just useless for two reasons:
1) flush_bits is not set accordingly, so nothing will be flushed
   in BeginQuery().
2) we always flush caches in EndCommandBuffer(), so if a reset
   is done in a previous command buffer we are safe.

Cc: "18.0" <mesa-stable@lists.freedesktop.org>
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Alex Smith <asmith@feralinteractive.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
---

diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index cfdc531acd5..2b41baea3d3 100644
--- a/src/amd/vulkan/radv_cmd_buffer.c
+++ b/src/amd/vulkan/radv_cmd_buffer.c
@@ -1930,13 +1930,6 @@ VkResult radv_BeginCommandBuffer(
 
 	cmd_buffer->status = RADV_CMD_BUFFER_STATUS_RECORDING;
 
-	/* Force cache flushes before starting a new query in case the
-	 * corresponding pool has been resetted from a different command
-	 * buffer. This is because we have to flush caches between reset and
-	 * begin if the compute shader path has been used.
-	 */
-	cmd_buffer->pending_reset_query = true;
-
 	return result;
 }