aco: Fix signed-vs-unsigned warning.
[mesa.git] / src / amd / vulkan / radv_query.c
index ff9ceceb3ed199caf406693b664a08d183427e4c..6f660c109e6ca97500e3904e97e1164a63bb6988 100644 (file)
 static const int pipelinestat_block_size = 11 * 8;
 static const unsigned pipeline_statistics_indices[] = {7, 6, 3, 4, 5, 2, 1, 0, 8, 9, 10};
 
-static unsigned get_max_db(struct radv_device *device)
+static unsigned
+radv_get_pipeline_statistics_index(const VkQueryPipelineStatisticFlagBits flag)
 {
-       unsigned num_db = device->physical_device->rad_info.num_render_backends;
-       MAYBE_UNUSED unsigned rb_mask = device->physical_device->rad_info.enabled_rb_mask;
-
-       /* Otherwise we need to change the query reset procedure */
-       assert(rb_mask == ((1ull << num_db) - 1));
-
-       return num_db;
+       int offset = ffs(flag) - 1;
+       assert(offset < ARRAY_SIZE(pipeline_statistics_indices));
+       return pipeline_statistics_indices[offset];
 }
 
-
 static nir_ssa_def *nir_test_flag(nir_builder *b, nir_ssa_def *flags, uint32_t flag)
 {
        return nir_i2b(b, nir_iand(b, flags, nir_imm_int(b, flag)));
@@ -108,12 +104,14 @@ build_occlusion_query_shader(struct radv_device *device) {
         *      uint64_t dst_offset = dst_stride * global_id.x;
         *      bool available = true;
         *      for (int i = 0; i < db_count; ++i) {
-        *              uint64_t start = src_buf[src_offset + 16 * i];
-        *              uint64_t end = src_buf[src_offset + 16 * i + 8];
-        *              if ((start & (1ull << 63)) && (end & (1ull << 63)))
-        *                      result += end - start;
-        *              else
-        *                      available = false;
+        *              if (enabled_rb_mask & (1 << i)) {
+        *                      uint64_t start = src_buf[src_offset + 16 * i];
+        *                      uint64_t end = src_buf[src_offset + 16 * i + 8];
+        *                      if ((start & (1ull << 63)) && (end & (1ull << 63)))
+        *                              result += end - start;
+        *                      else
+        *                              available = false;
+        *              }
         *      }
         *      uint32_t elem_size = flags & VK_QUERY_RESULT_64_BIT ? 8 : 4;
         *      if ((flags & VK_QUERY_RESULT_PARTIAL_BIT) || available) {
@@ -139,7 +137,8 @@ build_occlusion_query_shader(struct radv_device *device) {
        nir_variable *start = nir_local_variable_create(b.impl, glsl_uint64_t_type(), "start");
        nir_variable *end = nir_local_variable_create(b.impl, glsl_uint64_t_type(), "end");
        nir_variable *available = nir_local_variable_create(b.impl, glsl_bool_type(), "available");
-       unsigned db_count = get_max_db(device);
+       unsigned enabled_rb_mask = device->physical_device->rad_info.enabled_rb_mask;
+       unsigned db_count = device->physical_device->rad_info.num_render_backends;
 
        nir_ssa_def *flags = radv_load_push_int(&b, 0, "flags");
 
@@ -187,6 +186,16 @@ build_occlusion_query_shader(struct radv_device *device) {
        nir_ssa_def *current_outer_count = nir_load_var(&b, outer_counter);
        radv_break_on_count(&b, outer_counter, nir_imm_int(&b, db_count));
 
+       nir_ssa_def *enabled_cond =
+               nir_iand(&b, nir_imm_int(&b, enabled_rb_mask),
+                            nir_ishl(&b, nir_imm_int(&b, 1), current_outer_count));
+
+       nir_if *enabled_if = nir_if_create(b.shader);
+       enabled_if->condition = nir_src_for_ssa(nir_i2b(&b, enabled_cond));
+       nir_cf_node_insert(b.cursor, &enabled_if->cf_node);
+
+       b.cursor = nir_after_cf_list(&enabled_if->then_list);
+
        nir_ssa_def *load_offset = nir_imul(&b, current_outer_count, nir_imm_int(&b, 16));
        load_offset = nir_iadd(&b, input_base, load_offset);
 
@@ -195,6 +204,7 @@ build_occlusion_query_shader(struct radv_device *device) {
        load->src[1] = nir_src_for_ssa(load_offset);
        nir_ssa_dest_init(&load->instr, &load->dest, 2, 64, NULL);
        load->num_components = 2;
+       nir_intrinsic_set_align(load, 16, 0);
        nir_builder_instr_insert(&b, &load->instr);
 
        nir_store_var(&b, start, nir_channel(&b, &load->dest.ssa, 0), 0x1);
@@ -242,6 +252,7 @@ build_occlusion_query_shader(struct radv_device *device) {
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(output_base);
        nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 8, 0);
        store->num_components = 1;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -252,6 +263,7 @@ build_occlusion_query_shader(struct radv_device *device) {
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(output_base);
        nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 4, 0);
        store->num_components = 1;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -270,6 +282,7 @@ build_occlusion_query_shader(struct radv_device *device) {
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(nir_iadd(&b, result_size, output_base));
        nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 4, 0);
        store->num_components = 1;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -375,6 +388,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
        load->src[1] = nir_src_for_ssa(avail_offset);
        nir_ssa_dest_init(&load->instr, &load->dest, 1, 32, NULL);
        load->num_components = 1;
+       nir_intrinsic_set_align(load, 4, 0);
        nir_builder_instr_insert(&b, &load->instr);
        nir_ssa_def *available32 = &load->dest.ssa;
 
@@ -395,6 +409,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(nir_iadd(&b, output_base, nir_imul(&b, elem_count, elem_size)));
        nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 4, 0);
        store->num_components = 1;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -420,6 +435,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
                                                            nir_imm_int(&b, pipeline_statistics_indices[i] * 8)));
                nir_ssa_dest_init(&load->instr, &load->dest, 1, 64, NULL);
                load->num_components = 1;
+               nir_intrinsic_set_align(load, 8, 0);
                nir_builder_instr_insert(&b, &load->instr);
                nir_ssa_def *start = &load->dest.ssa;
 
@@ -429,6 +445,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
                                                            nir_imm_int(&b, pipeline_statistics_indices[i] * 8 + pipelinestat_block_size)));
                nir_ssa_dest_init(&load->instr, &load->dest, 1, 64, NULL);
                load->num_components = 1;
+               nir_intrinsic_set_align(load, 8, 0);
                nir_builder_instr_insert(&b, &load->instr);
                nir_ssa_def *end = &load->dest.ssa;
 
@@ -446,6 +463,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
                store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
                store->src[2] = nir_src_for_ssa(nir_load_var(&b, output_offset));
                nir_intrinsic_set_write_mask(store, 0x1);
+               nir_intrinsic_set_align(store, 8, 0);
                store->num_components = 1;
                nir_builder_instr_insert(&b, &store->instr);
 
@@ -456,6 +474,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
                store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
                store->src[2] = nir_src_for_ssa(nir_load_var(&b, output_offset));
                nir_intrinsic_set_write_mask(store, 0x1);
+               nir_intrinsic_set_align(store, 4, 0);
                store->num_components = 1;
                nir_builder_instr_insert(&b, &store->instr);
 
@@ -502,6 +521,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(output_elem);
        nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 8, 0);
        store->num_components = 1;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -512,6 +532,7 @@ build_pipeline_statistics_query_shader(struct radv_device *device) {
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(output_elem);
        nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 4, 0);
        store->num_components = 1;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -620,6 +641,7 @@ build_tfb_query_shader(struct radv_device *device)
        load1->src[1] = nir_src_for_ssa(input_base);
        nir_ssa_dest_init(&load1->instr, &load1->dest, 4, 32, NULL);
        load1->num_components = 4;
+       nir_intrinsic_set_align(load1, 32, 0);
        nir_builder_instr_insert(&b, &load1->instr);
 
        nir_intrinsic_instr *load2 = nir_intrinsic_instr_create(b.shader, nir_intrinsic_load_ssbo);
@@ -627,6 +649,7 @@ build_tfb_query_shader(struct radv_device *device)
        load2->src[1] = nir_src_for_ssa(nir_iadd(&b, input_base, nir_imm_int(&b, 16)));
        nir_ssa_dest_init(&load2->instr, &load2->dest, 4, 32, NULL);
        load2->num_components = 4;
+       nir_intrinsic_set_align(load2, 16, 0);
        nir_builder_instr_insert(&b, &load2->instr);
 
        /* Check if result is available. */
@@ -702,6 +725,7 @@ build_tfb_query_shader(struct radv_device *device)
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(output_base);
        nir_intrinsic_set_write_mask(store, 0x3);
+       nir_intrinsic_set_align(store, 8, 0);
        store->num_components = 2;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -712,6 +736,7 @@ build_tfb_query_shader(struct radv_device *device)
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(output_base);
        nir_intrinsic_set_write_mask(store, 0x3);
+       nir_intrinsic_set_align(store, 4, 0);
        store->num_components = 2;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -732,6 +757,194 @@ build_tfb_query_shader(struct radv_device *device)
        store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
        store->src[2] = nir_src_for_ssa(nir_iadd(&b, result_size, output_base));
        nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 4, 0);
+       store->num_components = 1;
+       nir_builder_instr_insert(&b, &store->instr);
+
+       b.cursor = nir_after_cf_node(&availability_if->cf_node);
+
+       return b.shader;
+}
+
+static nir_shader *
+build_timestamp_query_shader(struct radv_device *device)
+{
+       /* the shader this builds is roughly
+        *
+        * uint32_t src_stride = 8;
+        *
+        * location(binding = 0) buffer dst_buf;
+        * location(binding = 1) buffer src_buf;
+        *
+        * void main() {
+        *      uint64_t result = 0;
+        *      bool available = false;
+        *      uint64_t src_offset = src_stride * global_id.x;
+        *      uint64_t dst_offset = dst_stride * global_id.x;
+        *      uint64_t timestamp = src_buf[src_offset];
+        *      if (timestamp != TIMESTAMP_NOT_READY) {
+        *              result = timestamp;
+        *              available = true;
+        *      }
+        *      uint32_t result_size = flags & VK_QUERY_RESULT_64_BIT ? 8 : 4;
+        *      if ((flags & VK_QUERY_RESULT_PARTIAL_BIT) || available) {
+        *              if (flags & VK_QUERY_RESULT_64_BIT) {
+        *                      dst_buf[dst_offset] = result;
+        *              } else {
+        *                      dst_buf[dst_offset] = (uint32_t)result;
+        *              }
+        *      }
+        *      if (flags & VK_QUERY_RESULT_WITH_AVAILABILITY_BIT) {
+        *              dst_buf[dst_offset + result_size] = available;
+        *      }
+        * }
+        */
+       nir_builder b;
+       nir_builder_init_simple_shader(&b, NULL, MESA_SHADER_COMPUTE, NULL);
+       b.shader->info.name = ralloc_strdup(b.shader, "timestamp_query");
+       b.shader->info.cs.local_size[0] = 64;
+       b.shader->info.cs.local_size[1] = 1;
+       b.shader->info.cs.local_size[2] = 1;
+
+       /* Create and initialize local variables. */
+       nir_variable *result =
+               nir_local_variable_create(b.impl, glsl_uint64_t_type(), "result");
+       nir_variable *available =
+               nir_local_variable_create(b.impl, glsl_bool_type(), "available");
+
+       nir_store_var(&b, result, nir_imm_int64(&b, 0), 0x1);
+       nir_store_var(&b, available, nir_imm_false(&b), 0x1);
+
+       nir_ssa_def *flags = radv_load_push_int(&b, 0, "flags");
+
+       /* Load resources. */
+       nir_intrinsic_instr *dst_buf = nir_intrinsic_instr_create(b.shader,
+                                                                 nir_intrinsic_vulkan_resource_index);
+       dst_buf->src[0] = nir_src_for_ssa(nir_imm_int(&b, 0));
+       dst_buf->num_components = 1;
+       nir_intrinsic_set_desc_set(dst_buf, 0);
+       nir_intrinsic_set_binding(dst_buf, 0);
+       nir_ssa_dest_init(&dst_buf->instr, &dst_buf->dest, dst_buf->num_components, 32, NULL);
+       nir_builder_instr_insert(&b, &dst_buf->instr);
+
+       nir_intrinsic_instr *src_buf = nir_intrinsic_instr_create(b.shader,
+                                                                 nir_intrinsic_vulkan_resource_index);
+       src_buf->src[0] = nir_src_for_ssa(nir_imm_int(&b, 0));
+       src_buf->num_components = 1;
+       nir_intrinsic_set_desc_set(src_buf, 0);
+       nir_intrinsic_set_binding(src_buf, 1);
+       nir_ssa_dest_init(&src_buf->instr, &src_buf->dest, src_buf->num_components, 32, NULL);
+       nir_builder_instr_insert(&b, &src_buf->instr);
+
+       /* Compute global ID. */
+       nir_ssa_def *invoc_id = nir_load_local_invocation_id(&b);
+       nir_ssa_def *wg_id = nir_load_work_group_id(&b);
+       nir_ssa_def *block_size = nir_imm_ivec4(&b,
+                                               b.shader->info.cs.local_size[0],
+                                               b.shader->info.cs.local_size[1],
+                                               b.shader->info.cs.local_size[2], 0);
+       nir_ssa_def *global_id = nir_iadd(&b, nir_imul(&b, wg_id, block_size), invoc_id);
+       global_id = nir_channel(&b, global_id, 0); // We only care about x here.
+
+       /* Compute src/dst strides. */
+       nir_ssa_def *input_stride = nir_imm_int(&b, 8);
+       nir_ssa_def *input_base = nir_imul(&b, input_stride, global_id);
+       nir_ssa_def *output_stride = radv_load_push_int(&b, 4, "output_stride");
+       nir_ssa_def *output_base = nir_imul(&b, output_stride, global_id);
+
+       /* Load data from the query pool. */
+       nir_intrinsic_instr *load = nir_intrinsic_instr_create(b.shader, nir_intrinsic_load_ssbo);
+       load->src[0] = nir_src_for_ssa(&src_buf->dest.ssa);
+       load->src[1] = nir_src_for_ssa(input_base);
+       nir_ssa_dest_init(&load->instr, &load->dest, 2, 32, NULL);
+       load->num_components = 2;
+       nir_intrinsic_set_align(load, 8, 0);
+       nir_builder_instr_insert(&b, &load->instr);
+
+       /* Pack the timestamp. */
+       nir_ssa_def *timestamp;
+       timestamp = nir_pack_64_2x32(&b, nir_vec2(&b,
+                                                 nir_channel(&b, &load->dest.ssa, 0),
+                                                 nir_channel(&b, &load->dest.ssa, 1)));
+
+       /* Check if result is available. */
+       nir_ssa_def *result_is_available =
+               nir_i2b(&b, nir_ine(&b, timestamp,
+                                   nir_imm_int64(&b, TIMESTAMP_NOT_READY)));
+
+       /* Only store result if available. */
+       nir_if *available_if = nir_if_create(b.shader);
+       available_if->condition = nir_src_for_ssa(result_is_available);
+       nir_cf_node_insert(b.cursor, &available_if->cf_node);
+
+       b.cursor = nir_after_cf_list(&available_if->then_list);
+
+       nir_store_var(&b, result, timestamp, 0x1);
+       nir_store_var(&b, available, nir_imm_true(&b), 0x1);
+
+       b.cursor = nir_after_cf_node(&available_if->cf_node);
+
+       /* Determine if result is 64 or 32 bit. */
+       nir_ssa_def *result_is_64bit =
+               nir_test_flag(&b, flags, VK_QUERY_RESULT_64_BIT);
+       nir_ssa_def *result_size =
+               nir_bcsel(&b, result_is_64bit, nir_imm_int(&b, 8),
+                         nir_imm_int(&b, 4));
+
+       /* Store the result if complete or partial results have been requested. */
+       nir_if *store_if = nir_if_create(b.shader);
+       store_if->condition =
+               nir_src_for_ssa(nir_ior(&b, nir_test_flag(&b, flags, VK_QUERY_RESULT_PARTIAL_BIT),
+                                       nir_load_var(&b, available)));
+       nir_cf_node_insert(b.cursor, &store_if->cf_node);
+
+       b.cursor = nir_after_cf_list(&store_if->then_list);
+
+       /* Store result. */
+       nir_if *store_64bit_if = nir_if_create(b.shader);
+       store_64bit_if->condition = nir_src_for_ssa(result_is_64bit);
+       nir_cf_node_insert(b.cursor, &store_64bit_if->cf_node);
+
+       b.cursor = nir_after_cf_list(&store_64bit_if->then_list);
+
+       nir_intrinsic_instr *store = nir_intrinsic_instr_create(b.shader, nir_intrinsic_store_ssbo);
+       store->src[0] = nir_src_for_ssa(nir_load_var(&b, result));
+       store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
+       store->src[2] = nir_src_for_ssa(output_base);
+       nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 8, 0);
+       store->num_components = 1;
+       nir_builder_instr_insert(&b, &store->instr);
+
+       b.cursor = nir_after_cf_list(&store_64bit_if->else_list);
+
+       store = nir_intrinsic_instr_create(b.shader, nir_intrinsic_store_ssbo);
+       store->src[0] = nir_src_for_ssa(nir_u2u32(&b, nir_load_var(&b, result)));
+       store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
+       store->src[2] = nir_src_for_ssa(output_base);
+       nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 4, 0);
+       store->num_components = 1;
+       nir_builder_instr_insert(&b, &store->instr);
+
+       b.cursor = nir_after_cf_node(&store_64bit_if->cf_node);
+
+       b.cursor = nir_after_cf_node(&store_if->cf_node);
+
+       /* Store the availability bit if requested. */
+       nir_if *availability_if = nir_if_create(b.shader);
+       availability_if->condition =
+               nir_src_for_ssa(nir_test_flag(&b, flags, VK_QUERY_RESULT_WITH_AVAILABILITY_BIT));
+       nir_cf_node_insert(b.cursor, &availability_if->cf_node);
+
+       b.cursor = nir_after_cf_list(&availability_if->then_list);
+
+       store = nir_intrinsic_instr_create(b.shader, nir_intrinsic_store_ssbo);
+       store->src[0] = nir_src_for_ssa(nir_b2i32(&b, nir_load_var(&b, available)));
+       store->src[1] = nir_src_for_ssa(&dst_buf->dest.ssa);
+       store->src[2] = nir_src_for_ssa(nir_iadd(&b, result_size, output_base));
+       nir_intrinsic_set_write_mask(store, 0x1);
+       nir_intrinsic_set_align(store, 4, 0);
        store->num_components = 1;
        nir_builder_instr_insert(&b, &store->instr);
 
@@ -746,6 +959,7 @@ static VkResult radv_device_init_meta_query_state_internal(struct radv_device *d
        struct radv_shader_module occlusion_cs = { .nir = NULL };
        struct radv_shader_module pipeline_statistics_cs = { .nir = NULL };
        struct radv_shader_module tfb_cs = { .nir = NULL };
+       struct radv_shader_module timestamp_cs = { .nir = NULL };
 
        mtx_lock(&device->meta_state.mtx);
        if (device->meta_state.query.pipeline_statistics_query_pipeline) {
@@ -755,6 +969,7 @@ static VkResult radv_device_init_meta_query_state_internal(struct radv_device *d
        occlusion_cs.nir = build_occlusion_query_shader(device);
        pipeline_statistics_cs.nir = build_pipeline_statistics_query_shader(device);
        tfb_cs.nir = build_tfb_query_shader(device);
+       timestamp_cs.nir = build_timestamp_query_shader(device);
 
        VkDescriptorSetLayoutCreateInfo occlusion_ds_create_info = {
                .sType = VK_STRUCTURE_TYPE_DESCRIPTOR_SET_LAYOUT_CREATE_INFO,
@@ -863,12 +1078,36 @@ static VkResult radv_device_init_meta_query_state_internal(struct radv_device *d
                                             radv_pipeline_cache_to_handle(&device->meta_state.cache),
                                             1, &tfb_pipeline_info, NULL,
                                             &device->meta_state.query.tfb_query_pipeline);
+       if (result != VK_SUCCESS)
+               goto fail;
+
+       VkPipelineShaderStageCreateInfo timestamp_pipeline_shader_stage = {
+               .sType = VK_STRUCTURE_TYPE_PIPELINE_SHADER_STAGE_CREATE_INFO,
+               .stage = VK_SHADER_STAGE_COMPUTE_BIT,
+               .module = radv_shader_module_to_handle(&timestamp_cs),
+               .pName = "main",
+               .pSpecializationInfo = NULL,
+       };
+
+       VkComputePipelineCreateInfo timestamp_pipeline_info = {
+               .sType = VK_STRUCTURE_TYPE_COMPUTE_PIPELINE_CREATE_INFO,
+               .stage = timestamp_pipeline_shader_stage,
+               .flags = 0,
+               .layout = device->meta_state.query.p_layout,
+       };
+
+       result = radv_CreateComputePipelines(radv_device_to_handle(device),
+                                            radv_pipeline_cache_to_handle(&device->meta_state.cache),
+                                            1, &timestamp_pipeline_info, NULL,
+                                            &device->meta_state.query.timestamp_query_pipeline);
+
 fail:
        if (result != VK_SUCCESS)
                radv_device_finish_meta_query_state(device);
        ralloc_free(occlusion_cs.nir);
        ralloc_free(pipeline_statistics_cs.nir);
        ralloc_free(tfb_cs.nir);
+       ralloc_free(timestamp_cs.nir);
        mtx_unlock(&device->meta_state.mtx);
        return result;
 }
@@ -898,6 +1137,11 @@ void radv_device_finish_meta_query_state(struct radv_device *device)
                                     device->meta_state.query.occlusion_query_pipeline,
                                     &device->meta_state.alloc);
 
+       if (device->meta_state.query.timestamp_query_pipeline)
+               radv_DestroyPipeline(radv_device_to_handle(device),
+                                    device->meta_state.query.timestamp_query_pipeline,
+                                    &device->meta_state.alloc);
+
        if (device->meta_state.query.p_layout)
                radv_DestroyPipelineLayout(radv_device_to_handle(device),
                                           device->meta_state.query.p_layout,
@@ -1011,8 +1255,8 @@ static void radv_query_shader(struct radv_cmd_buffer *cmd_buffer,
                                      VK_SHADER_STAGE_COMPUTE_BIT, 0, sizeof(push_constants),
                                      &push_constants);
 
-       cmd_buffer->state.flush_bits |= RADV_CMD_FLAG_INV_GLOBAL_L2 |
-                                       RADV_CMD_FLAG_INV_VMEM_L1;
+       cmd_buffer->state.flush_bits |= RADV_CMD_FLAG_INV_L2 |
+                                       RADV_CMD_FLAG_INV_VCACHE;
 
        if (flags & VK_QUERY_RESULT_WAIT_BIT)
                cmd_buffer->state.flush_bits |= RADV_CMD_FLUSH_AND_INV_FRAMEBUFFER;
@@ -1025,6 +1269,22 @@ static void radv_query_shader(struct radv_cmd_buffer *cmd_buffer,
        radv_meta_restore(&saved_state, cmd_buffer);
 }
 
+static bool
+radv_query_pool_needs_gds(struct radv_device *device,
+                         struct radv_query_pool *pool)
+{
+       /* The number of primitives generated by geometry shader invocations is
+        * only counted by the hardware if GS uses the legacy path. When NGG GS
+        * is used, the hardware can't know the number of generated primitives
+        * and we have to it manually inside the shader. To achieve that, the
+        * driver does a plain GDS atomic to accumulate that value.
+        * TODO: fix use of NGG GS and non-NGG GS inside the same begin/end
+        * query.
+        */
+       return device->physical_device->use_ngg &&
+              (pool->pipeline_stats_mask & VK_QUERY_PIPELINE_STATISTIC_GEOMETRY_SHADER_PRIMITIVES_BIT);
+}
+
 VkResult radv_CreateQueryPool(
        VkDevice                                    _device,
        const VkQueryPoolCreateInfo*                pCreateInfo,
@@ -1042,7 +1302,7 @@ VkResult radv_CreateQueryPool(
 
        switch(pCreateInfo->queryType) {
        case VK_QUERY_TYPE_OCCLUSION:
-               pool->stride = 16 * get_max_db(device);
+               pool->stride = 16 * device->physical_device->rad_info.num_render_backends;
                break;
        case VK_QUERY_TYPE_PIPELINE_STATISTICS:
                pool->stride = pipelinestat_block_size * 2;
@@ -1123,12 +1383,13 @@ VkResult radv_GetQueryPoolResults(
 
                switch (pool->type) {
                case VK_QUERY_TYPE_TIMESTAMP: {
-                       available = *(uint64_t *)src != TIMESTAMP_NOT_READY;
+                       volatile uint64_t const *src64 = (volatile uint64_t const *)src;
+                       available = *src64 != TIMESTAMP_NOT_READY;
 
                        if (flags & VK_QUERY_RESULT_WAIT_BIT) {
-                               while (*(volatile uint64_t *)src == TIMESTAMP_NOT_READY)
+                               while (*src64 == TIMESTAMP_NOT_READY)
                                        ;
-                               available = *(uint64_t *)src != TIMESTAMP_NOT_READY;
+                               available = true;
                        }
 
                        if (!available && !(flags & VK_QUERY_RESULT_PARTIAL_BIT))
@@ -1136,23 +1397,28 @@ VkResult radv_GetQueryPoolResults(
 
                        if (flags & VK_QUERY_RESULT_64_BIT) {
                                if (available || (flags & VK_QUERY_RESULT_PARTIAL_BIT))
-                                       *(uint64_t*)dest = *(uint64_t*)src;
+                                       *(uint64_t*)dest = *src64;
                                dest += 8;
                        } else {
                                if (available || (flags & VK_QUERY_RESULT_PARTIAL_BIT))
-                                       *(uint32_t*)dest = *(uint32_t*)src;
+                                       *(uint32_t*)dest = *(volatile uint32_t*)src;
                                dest += 4;
                        }
                        break;
                }
                case VK_QUERY_TYPE_OCCLUSION: {
                        volatile uint64_t const *src64 = (volatile uint64_t const *)src;
+                       uint32_t db_count = device->physical_device->rad_info.num_render_backends;
+                       uint32_t enabled_rb_mask = device->physical_device->rad_info.enabled_rb_mask;
                        uint64_t sample_count = 0;
-                       int db_count = get_max_db(device);
                        available = 1;
 
                        for (int i = 0; i < db_count; ++i) {
                                uint64_t start, end;
+
+                               if (!(enabled_rb_mask & (1 << i)))
+                                       continue;
+
                                do {
                                        start = src64[2 * i];
                                        end = src64[2 * i + 1];
@@ -1183,13 +1449,13 @@ VkResult radv_GetQueryPoolResults(
                        if (flags & VK_QUERY_RESULT_WAIT_BIT)
                                while(!*(volatile uint32_t*)(pool->ptr + pool->availability_offset + 4 * query))
                                        ;
-                       available = *(uint32_t*)(pool->ptr + pool->availability_offset + 4 * query);
+                       available = *(volatile uint32_t*)(pool->ptr + pool->availability_offset + 4 * query);
 
                        if (!available && !(flags & VK_QUERY_RESULT_PARTIAL_BIT))
                                result = VK_NOT_READY;
 
-                       const uint64_t *start = (uint64_t*)src;
-                       const uint64_t *stop = (uint64_t*)(src + pipelinestat_block_size);
+                       const volatile uint64_t *start = (uint64_t*)src;
+                       const volatile uint64_t *stop = (uint64_t*)(src + pipelinestat_block_size);
                        if (flags & VK_QUERY_RESULT_64_BIT) {
                                uint64_t *dst = (uint64_t*)dest;
                                dest += util_bitcount(pool->pipeline_stats_mask) * 8;
@@ -1301,7 +1567,6 @@ void radv_CmdCopyQueryPoolResults(
        RADV_FROM_HANDLE(radv_query_pool, pool, queryPool);
        RADV_FROM_HANDLE(radv_buffer, dst_buffer, dstBuffer);
        struct radeon_cmdbuf *cs = cmd_buffer->cs;
-       unsigned elem_size = (flags & VK_QUERY_RESULT_64_BIT) ? 8 : 4;
        uint64_t va = radv_buffer_get_va(pool->bo);
        uint64_t dest_va = radv_buffer_get_va(dst_buffer->bo);
        dest_va += dst_buffer->offset + dstOffset;
@@ -1361,14 +1626,13 @@ void radv_CmdCopyQueryPoolResults(
                                  pool->availability_offset + 4 * firstQuery);
                break;
        case VK_QUERY_TYPE_TIMESTAMP:
-               for(unsigned i = 0; i < queryCount; ++i, dest_va += stride) {
-                       unsigned query = firstQuery + i;
-                       uint64_t local_src_va = va  + query * pool->stride;
-
-                       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 19);
+               if (flags & VK_QUERY_RESULT_WAIT_BIT) {
+                       for(unsigned i = 0; i < queryCount; ++i, dest_va += stride) {
+                               unsigned query = firstQuery + i;
+                               uint64_t local_src_va = va  + query * pool->stride;
 
+                               radeon_check_space(cmd_buffer->device->ws, cs, 7);
 
-                       if (flags & VK_QUERY_RESULT_WAIT_BIT) {
                                /* Wait on the high 32 bits of the timestamp in
                                 * case the low part is 0xffffffff.
                                 */
@@ -1377,30 +1641,14 @@ void radv_CmdCopyQueryPoolResults(
                                                 TIMESTAMP_NOT_READY >> 32,
                                                 0xffffffff);
                        }
-                       if (flags & VK_QUERY_RESULT_WITH_AVAILABILITY_BIT) {
-                               uint64_t avail_dest_va = dest_va + elem_size;
-
-                               radeon_emit(cs, PKT3(PKT3_COPY_DATA, 4, 0));
-                               radeon_emit(cs, COPY_DATA_SRC_SEL(COPY_DATA_SRC_MEM) |
-                                               COPY_DATA_DST_SEL(COPY_DATA_DST_MEM_GRBM));
-                               radeon_emit(cs, local_src_va);
-                               radeon_emit(cs, local_src_va >> 32);
-                               radeon_emit(cs, avail_dest_va);
-                               radeon_emit(cs, avail_dest_va >> 32);
-                       }
-
-                       radeon_emit(cs, PKT3(PKT3_COPY_DATA, 4, 0));
-                       radeon_emit(cs, COPY_DATA_SRC_SEL(COPY_DATA_SRC_MEM) |
-                                       COPY_DATA_DST_SEL(COPY_DATA_DST_MEM_GRBM) |
-                                       ((flags & VK_QUERY_RESULT_64_BIT) ? COPY_DATA_COUNT_SEL : 0));
-                       radeon_emit(cs, local_src_va);
-                       radeon_emit(cs, local_src_va >> 32);
-                       radeon_emit(cs, dest_va);
-                       radeon_emit(cs, dest_va >> 32);
-
-
-                       assert(cs->cdw <= cdw_max);
                }
+
+               radv_query_shader(cmd_buffer, &cmd_buffer->device->meta_state.query.timestamp_query_pipeline,
+                                 pool->bo, dst_buffer->bo,
+                                 firstQuery * pool->stride,
+                                 dst_buffer->offset + dstOffset,
+                                 pool->stride, stride,
+                                 queryCount, flags, 0, 0);
                break;
        case VK_QUERY_TYPE_TRANSFORM_FEEDBACK_STREAM_EXT:
                if (flags & VK_QUERY_RESULT_WAIT_BIT) {
@@ -1467,7 +1715,7 @@ void radv_CmdResetQueryPool(
        }
 }
 
-void radv_ResetQueryPoolEXT(
+void radv_ResetQueryPool(
        VkDevice                                   _device,
        VkQueryPool                                 queryPool,
        uint32_t                                    firstQuery,
@@ -1501,6 +1749,7 @@ static unsigned event_type_for_stream(unsigned stream)
 }
 
 static void emit_begin_query(struct radv_cmd_buffer *cmd_buffer,
+                            struct radv_query_pool *pool,
                             uint64_t va,
                             VkQueryType query_type,
                             VkQueryControlFlags flags,
@@ -1552,6 +1801,30 @@ static void emit_begin_query(struct radv_cmd_buffer *cmd_buffer,
                radeon_emit(cs, EVENT_TYPE(V_028A90_SAMPLE_PIPELINESTAT) | EVENT_INDEX(2));
                radeon_emit(cs, va);
                radeon_emit(cs, va >> 32);
+
+               if (radv_query_pool_needs_gds(cmd_buffer->device, pool)) {
+                       int idx = radv_get_pipeline_statistics_index(VK_QUERY_PIPELINE_STATISTIC_GEOMETRY_SHADER_PRIMITIVES_BIT);
+
+                       /* Make sure GDS is idle before copying the value. */
+                       cmd_buffer->state.flush_bits |= RADV_CMD_FLAG_PS_PARTIAL_FLUSH |
+                                                       RADV_CMD_FLAG_INV_L2;
+                       si_emit_cache_flush(cmd_buffer);
+
+                       va += 8 * idx;
+
+                       si_cs_emit_write_event_eop(cs,
+                                                  cmd_buffer->device->physical_device->rad_info.chip_class,
+                                                  radv_cmd_buffer_uses_mec(cmd_buffer),
+                                                  V_028A90_PS_DONE, 0,
+                                                  EOP_DST_SEL_TC_L2,
+                                                  EOP_DATA_SEL_GDS,
+                                                  va, EOP_DATA_GDS(0, 1), 0);
+
+                       /* Record that the command buffer needs GDS. */
+                       cmd_buffer->gds_needed = true;
+
+                       cmd_buffer->state.active_pipeline_gds_queries++;
+               }
                break;
        case VK_QUERY_TYPE_TRANSFORM_FEEDBACK_STREAM_EXT:
                radeon_check_space(cmd_buffer->device->ws, cs, 4);
@@ -1570,6 +1843,7 @@ static void emit_begin_query(struct radv_cmd_buffer *cmd_buffer,
 }
 
 static void emit_end_query(struct radv_cmd_buffer *cmd_buffer,
+                          struct radv_query_pool *pool,
                           uint64_t va, uint64_t avail_va,
                           VkQueryType query_type, uint32_t index)
 {
@@ -1613,9 +1887,31 @@ static void emit_end_query(struct radv_cmd_buffer *cmd_buffer,
                                           cmd_buffer->device->physical_device->rad_info.chip_class,
                                           radv_cmd_buffer_uses_mec(cmd_buffer),
                                           V_028A90_BOTTOM_OF_PIPE_TS, 0,
+                                          EOP_DST_SEL_MEM,
                                           EOP_DATA_SEL_VALUE_32BIT,
                                           avail_va, 1,
                                           cmd_buffer->gfx9_eop_bug_va);
+
+               if (radv_query_pool_needs_gds(cmd_buffer->device, pool)) {
+                       int idx = radv_get_pipeline_statistics_index(VK_QUERY_PIPELINE_STATISTIC_GEOMETRY_SHADER_PRIMITIVES_BIT);
+
+                       /* Make sure GDS is idle before copying the value. */
+                       cmd_buffer->state.flush_bits |= RADV_CMD_FLAG_PS_PARTIAL_FLUSH |
+                                                       RADV_CMD_FLAG_INV_L2;
+                       si_emit_cache_flush(cmd_buffer);
+
+                       va += 8 * idx;
+
+                       si_cs_emit_write_event_eop(cs,
+                                                  cmd_buffer->device->physical_device->rad_info.chip_class,
+                                                  radv_cmd_buffer_uses_mec(cmd_buffer),
+                                                  V_028A90_PS_DONE, 0,
+                                                  EOP_DST_SEL_TC_L2,
+                                                  EOP_DATA_SEL_GDS,
+                                                  va, EOP_DATA_GDS(0, 1), 0);
+
+                       cmd_buffer->state.active_pipeline_gds_queries--;
+               }
                break;
        case VK_QUERY_TYPE_TRANSFORM_FEEDBACK_STREAM_EXT:
                radeon_check_space(cmd_buffer->device->ws, cs, 4);
@@ -1633,8 +1929,12 @@ static void emit_end_query(struct radv_cmd_buffer *cmd_buffer,
 
        cmd_buffer->active_query_flush_bits |= RADV_CMD_FLAG_PS_PARTIAL_FLUSH |
                                               RADV_CMD_FLAG_CS_PARTIAL_FLUSH |
-                                              RADV_CMD_FLAG_INV_GLOBAL_L2 |
-                                              RADV_CMD_FLAG_INV_VMEM_L1;
+                                              RADV_CMD_FLAG_INV_L2 |
+                                              RADV_CMD_FLAG_INV_VCACHE;
+       if (cmd_buffer->device->physical_device->rad_info.chip_class >= GFX9) {
+               cmd_buffer->active_query_flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_CB |
+                                                      RADV_CMD_FLAG_FLUSH_AND_INV_DB;
+       }
 }
 
 void radv_CmdBeginQueryIndexedEXT(
@@ -1655,7 +1955,7 @@ void radv_CmdBeginQueryIndexedEXT(
 
        va += pool->stride * query;
 
-       emit_begin_query(cmd_buffer, va, pool->type, flags, index);
+       emit_begin_query(cmd_buffer, pool, va, pool->type, flags, index);
 }
 
 void radv_CmdBeginQuery(
@@ -1682,7 +1982,7 @@ void radv_CmdEndQueryIndexedEXT(
        /* Do not need to add the pool BO to the list because the query must
         * currently be active, which means the BO is already in the list.
         */
-       emit_end_query(cmd_buffer, va, avail_va, pool->type, index);
+       emit_end_query(cmd_buffer, pool, va, avail_va, pool->type, index);
 
        /*
         * For multiview we have to emit a query for each bit in the mask,
@@ -1699,8 +1999,8 @@ void radv_CmdEndQueryIndexedEXT(
                for (unsigned i = 1; i < util_bitcount(cmd_buffer->state.subpass->view_mask); i++) {
                        va += pool->stride;
                        avail_va += 4;
-                       emit_begin_query(cmd_buffer, va, pool->type, 0, 0);
-                       emit_end_query(cmd_buffer, va, avail_va, pool->type, 0);
+                       emit_begin_query(cmd_buffer, pool, va, pool->type, 0, 0);
+                       emit_end_query(cmd_buffer, pool, va, avail_va, pool->type, 0);
                }
        }
 }
@@ -1734,7 +2034,7 @@ void radv_CmdWriteTimestamp(
        if (cmd_buffer->state.subpass && cmd_buffer->state.subpass->view_mask)
                num_queries = util_bitcount(cmd_buffer->state.subpass->view_mask);
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 28 * num_queries);
+       ASSERTED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 28 * num_queries);
 
        for (unsigned i = 0; i < num_queries; i++) {
                switch(pipelineStage) {
@@ -1753,6 +2053,7 @@ void radv_CmdWriteTimestamp(
                                                   cmd_buffer->device->physical_device->rad_info.chip_class,
                                                   mec,
                                                   V_028A90_BOTTOM_OF_PIPE_TS, 0,
+                                                  EOP_DST_SEL_MEM,
                                                   EOP_DATA_SEL_TIMESTAMP,
                                                   query_va, 0,
                                                   cmd_buffer->gfx9_eop_bug_va);
@@ -1760,5 +2061,15 @@ void radv_CmdWriteTimestamp(
                }
                query_va += pool->stride;
        }
+
+       cmd_buffer->active_query_flush_bits |= RADV_CMD_FLAG_PS_PARTIAL_FLUSH |
+                                              RADV_CMD_FLAG_CS_PARTIAL_FLUSH |
+                                              RADV_CMD_FLAG_INV_L2 |
+                                              RADV_CMD_FLAG_INV_VCACHE;
+       if (cmd_buffer->device->physical_device->rad_info.chip_class >= GFX9) {
+               cmd_buffer->active_query_flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_CB |
+                                                      RADV_CMD_FLAG_FLUSH_AND_INV_DB;
+       }
+
        assert(cmd_buffer->cs->cdw <= cdw_max);
 }