From 3f795016d33e14303726bfa15640b19f94e982fc Mon Sep 17 00:00:00 2001 From: Casey Bowman Date: Thu, 22 May 2025 15:26:32 -0700 Subject: [PATCH] anv: Remove vf_flush for start of command buffers These vertex fetch flushes aren't required in gen9+ because the display driver will take care of this invalidation on QueueSubmit. So let's remove it. Signed-off-by: Casey Bowman Reviewed-by: Lionel Landwerlin Part-of: --- src/intel/vulkan/genX_cmd_buffer.c | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index 4e298b61ab9..671a6ef30d9 100644 --- a/src/intel/vulkan/genX_cmd_buffer.c +++ b/src/intel/vulkan/genX_cmd_buffer.c @@ -3252,23 +3252,6 @@ genX(BeginCommandBuffer)( genX(cmd_buffer_emit_bt_pool_base_address)(cmd_buffer); } - /* We sometimes store vertex data in the dynamic state buffer for blorp - * operations and our dynamic state stream may re-use data from previous - * command buffers. In order to prevent stale cache data, we flush the VF - * cache. We could do this on every blorp call but that's not really - * needed as all of the data will get written by the CPU prior to the GPU - * executing anything. The chances are fairly high that they will use - * blorp at least once per primary command buffer so it shouldn't be - * wasted. - * - * There is also a workaround on gfx8 which requires us to invalidate the - * VF cache occasionally. It's easier if we can assume we start with a - * fresh cache (See also genX(cmd_buffer_set_binding_for_gfx8_vb_flush).) - */ - anv_add_pending_pipe_bits(cmd_buffer, - ANV_PIPE_VF_CACHE_INVALIDATE_BIT, - "new cmd buffer"); - /* Invalidate the aux table in every primary command buffer. This ensures * the command buffer see the last updates made by the host. */