From edb5f855c28013e2782132251abd94bf1d040c13 Mon Sep 17 00:00:00 2001 From: Stenzek Date: Tue, 10 Oct 2017 23:52:17 +1000 Subject: VideoConfig: Prevent race condition on g_Config when refreshing There was a race condition between the video thread and the host thread, if corrections need to be made by VerifyValidity(). Briefly, the config will contain invalid values. Instead, pause emulation first, which will flush the video thread, update the config and correct it, then resume emulation, after which the video thread will detect the config has changed and act accordingly. --- Source/Core/VideoCommon/MainBase.cpp | 1 - 1 file changed, 1 deletion(-) (limited to 'Source/Core/VideoCommon/MainBase.cpp') diff --git a/Source/Core/VideoCommon/MainBase.cpp b/Source/Core/VideoCommon/MainBase.cpp index 5f5ce785ef..179efdbd41 100644 --- a/Source/Core/VideoCommon/MainBase.cpp +++ b/Source/Core/VideoCommon/MainBase.cpp @@ -192,7 +192,6 @@ void VideoBackendBase::InitializeShared() g_Config.Refresh(); g_Config.UpdateProjectionHack(); - g_Config.VerifyValidity(); UpdateActiveConfig(); } -- cgit v1.2.3