From 7e56af1b6d77cf8d39af12213a3f837597bd1fac Mon Sep 17 00:00:00 2001 From: Sagar Ghuge Date: Thu, 6 Nov 2025 11:52:11 -0800 Subject: [PATCH] anv: Drop unwanted untyped flush for AS query MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit CmdWriteAccelerationStructuresPropertiesKHR writes the data with MI commands, we no longer dispatch shaders to write the properties. As a result, we don't need to flush untyped cache. Fixes: f0e18c475b ("intel: remove GRL/intel-clc") Signed-off-by: Sagar Ghuge Reviewed-by: Lionel Landwerlin Reviewed-by: Tapani Pälli Part-of: (cherry picked from commit 14194e59a42e8899dd945ecf3a17a6fd995a6d85) --- .pick_status.json | 2 +- src/intel/vulkan/genX_query.c | 11 +---------- 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/.pick_status.json b/.pick_status.json index af8a4c63d28..df65c804003 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -914,7 +914,7 @@ "description": "anv: Drop unwanted untyped flush for AS query", "nominated": true, "nomination_type": 2, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": "f0e18c475b71c7804e893ab7c1b70bb548ee6917", "notes": null diff --git a/src/intel/vulkan/genX_query.c b/src/intel/vulkan/genX_query.c index 5d3324b9457..54474869ce2 100644 --- a/src/intel/vulkan/genX_query.c +++ b/src/intel/vulkan/genX_query.c @@ -1879,16 +1879,7 @@ copy_query_results_with_shader(struct anv_cmd_buffer *cmd_buffer, /* Flushes for the queries to complete */ if (flags & VK_QUERY_RESULT_WAIT_BIT) { - /* Some queries are done with shaders, so we need to have them flush - * high level caches writes. The L3 should be shared across the GPU. - */ - if (pool->vk.query_type == VK_QUERY_TYPE_ACCELERATION_STRUCTURE_COMPACTED_SIZE_KHR || - pool->vk.query_type == VK_QUERY_TYPE_ACCELERATION_STRUCTURE_SIZE_KHR || - pool->vk.query_type == VK_QUERY_TYPE_ACCELERATION_STRUCTURE_SERIALIZATION_SIZE_KHR || - pool->vk.query_type == VK_QUERY_TYPE_ACCELERATION_STRUCTURE_SERIALIZATION_BOTTOM_LEVEL_POINTERS_KHR) { - needed_flushes |= ANV_PIPE_UNTYPED_DATAPORT_CACHE_FLUSH_BIT; - } - /* And we need to stall for previous CS writes to land or the flushes to + /* We need to stall for previous CS writes to land or the flushes to * complete. */ needed_flushes |= ANV_PIPE_CS_STALL_BIT;