radv: do not set pending_reset_query in BeginCommandBuffer()
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Thu, 1 Mar 2018 09:53:49 +0000 (10:53 +0100)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Fri, 2 Mar 2018 08:44:12 +0000 (09:44 +0100)
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>
src/amd/vulkan/radv_cmd_buffer.c

index cfdc531acd5a8b0b7259e56b12689b099034c440..2b41baea3d34f406936755c4b5c5db6ae134d479 100644 (file)
@@ -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;
 }