From 9666bd1245071e627301a60bf7fd73fa37f3f0a2 Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Thu, 13 Nov 2025 14:12:50 +0100 Subject: [PATCH] radv: remove unnecessary handling of SDMA in radv_cs_emit_write_event_eop() This function is only called for GFX or ACE. SDMA uses are already handled before. Signed-off-by: Samuel Pitoiset Part-of: --- src/amd/vulkan/radv_cs.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/amd/vulkan/radv_cs.c b/src/amd/vulkan/radv_cs.c index 8d374f13ba9..db899a89ce9 100644 --- a/src/amd/vulkan/radv_cs.c +++ b/src/amd/vulkan/radv_cs.c @@ -21,10 +21,7 @@ radv_cs_emit_write_event_eop(struct radv_cmd_stream *cs, enum amd_gfx_level gfx_ unsigned event_flags, unsigned dst_sel, unsigned int_sel, unsigned data_sel, uint64_t va, uint32_t new_fence, uint64_t gfx9_eop_bug_va) { - if (cs->hw_ip == AMD_IP_SDMA) { - ac_emit_sdma_fence(cs->b, va, new_fence); - return; - } + assert(cs->hw_ip == AMD_IP_GFX || cs->hw_ip == AMD_IP_COMPUTE); /* The EOP bug is specific to GFX9. Though, RadeonSI also implements it for GFX6-8 but it * shouldn't be necessary because it's using SURFACE_SYNC to flush L2. See