From 3b92f4928aeefae32569152ee3a746b76b37a803 Mon Sep 17 00:00:00 2001 From: Ali Homafar Date: Wed, 28 Aug 2024 13:28:55 -0400 Subject: [PATCH] Vulkan: Remove unnecessary command buffer execution When resizing the game window there is a chance that the swapchain state is bad. This will result in a deadlock on Linux + Nvidia's proprietary driver. This code can be removed and the next command buffer execution will simply occur when the swapchain has been recreated. The root issue may be a Linux Nvidia driver implementation problem. I wasn't able to reproduce the original problem on Windows, nor on my Steam Deck. I tested this change on Fedora 40+Nvidia & SteamOS+AMD. Fixes https://bugs.dolphin-emu.org/issues/13523 --- Source/Core/VideoBackends/Vulkan/VKGfx.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/Source/Core/VideoBackends/Vulkan/VKGfx.cpp b/Source/Core/VideoBackends/Vulkan/VKGfx.cpp index d65a5d4680..1134fb9014 100644 --- a/Source/Core/VideoBackends/Vulkan/VKGfx.cpp +++ b/Source/Core/VideoBackends/Vulkan/VKGfx.cpp @@ -260,9 +260,6 @@ bool VKGfx::BindBackbuffer(const ClearColor& clear_color) if (res != VK_SUCCESS) { - // Execute cmdbuffer before resizing, as the last frame could still be presenting. - ExecuteCommandBuffer(false, true); - // Was this a lost exclusive fullscreen? if (res == VK_ERROR_FULL_SCREEN_EXCLUSIVE_MODE_LOST_EXT) {