Merge pull request #38 from Djihads80/nx2-accuracy-switch

nx2: Fix the current crash when switching from fast to accurate and vice versa

1 files changed, 12 insertions, 4 deletions

diff --git a/src/video_core/gpu_split.cpp b/src/video_core/gpu_split.cpp
index 2bc8e29470..ac0d3f25f1 100644
--- a/src/video_core/gpu_split.cpp
+++ b/src/video_core/gpu_split.cpp
@@ -90,15 +90,23 @@ void GpuSplit::MaybeStart(Core::System& system_) {
         return value != nullptr ? value[0] : '2';
     }();
     const bool wanted = mode == '1' || mode == '2';
-    if (active || !wanted || !Tegra::g_nx2_ampere.load(std::memory_order_relaxed)) {
+    if (!wanted || !Tegra::g_nx2_ampere.load(std::memory_order_relaxed)) {
         return;
     }
     // The front reads command lists straight from guest memory; a read that flushes the caches
-    // first (safe DMA, accurate GPU) would touch back-owned state.
+    // first (safe DMA, accurate GPU) would touch back-owned state. The settings can change
+    // mid-game: hand the engines back to the front (and start again when they allow it).
     const bool safe_reads = Settings::IsDMALevelDefault() ? Settings::IsGPULevelHigh()
                                                           : Settings::IsDMALevelSafe();
-    if (safe_reads || Settings::IsGPULevelHigh() ||
-        Settings::values.renderer_backend.GetValue() != Settings::RendererBackend::Vulkan) {
+    const bool allowed = !safe_reads && !Settings::IsGPULevelHigh() &&
+                         Settings::values.renderer_backend.GetValue() == Settings::RendererBackend::Vulkan;
+    if (active && !allowed) {
+        Drain();
+        Stop();
+        LOG_INFO(HW_GPU, "GPU thread split: stopped, the accuracy settings need the caches on the front");
+        return;
+    }
+    if (active || !allowed) {
         return;
     }
     system = &system_;