radv: allow to use a compute shader for resetting the query pool
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Thu, 26 Oct 2017 16:03:24 +0000 (18:03 +0200)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Fri, 27 Oct 2017 11:47:06 +0000 (13:47 +0200)
Serious Sam Fusion 2017 uses a huge number of occlusion queries,
and the allocated query pool buffer is greater than 4096 bytes.

This slightly improves performance (tested in Ultra) from
117.2 FPS to 119.7 FPS (~+2%) on my RX480.

This also improves Talos, from 69 FPS to 72/73 FPS (~+5%).

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
src/amd/vulkan/radv_query.c

index 06bf14ab6806de2c2e1af53be722d762c13a6d40..16ffd8464873445e021cc946237ef0c317987a56 100644 (file)
@@ -1058,16 +1058,18 @@ void radv_CmdResetQueryPool(
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
        RADV_FROM_HANDLE(radv_query_pool, pool, queryPool);
-       uint64_t va = radv_buffer_get_va(pool->bo);
+       struct radv_cmd_state *state = &cmd_buffer->state;
 
-       cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs, pool->bo, 8);
+       state->flush_bits |= radv_fill_buffer(cmd_buffer, pool->bo,
+                                             firstQuery * pool->stride,
+                                             queryCount * pool->stride, 0);
 
-       si_cp_dma_clear_buffer(cmd_buffer, va + firstQuery * pool->stride,
-                              queryCount * pool->stride, 0);
        if (pool->type == VK_QUERY_TYPE_TIMESTAMP ||
-           pool->type == VK_QUERY_TYPE_PIPELINE_STATISTICS)
-               si_cp_dma_clear_buffer(cmd_buffer, va + pool->availability_offset + firstQuery * 4,
-                                      queryCount * 4, 0);
+           pool->type == VK_QUERY_TYPE_PIPELINE_STATISTICS) {
+               state->flush_bits |= radv_fill_buffer(cmd_buffer, pool->bo,
+                                                     pool->availability_offset + firstQuery * 4,
+                                                     queryCount * 4, 0);
+       }
 }
 
 void radv_CmdBeginQuery(