diff options
author | Fernando Sahmkow <fsahmkow27@gmail.com> | 2020-02-27 15:47:02 +0100 |
---|---|---|
committer | Fernando Sahmkow <fsahmkow27@gmail.com> | 2020-06-27 17:35:18 +0200 |
commit | cc3aa959265480e1990b10ee37ebf1c0ade3da64 (patch) | |
tree | 0188b3adeab5cb83362f296e6894d70b53cb86e0 /src | |
parent | SVC: Add locks to the memory management. (diff) | |
download | yuzu-cc3aa959265480e1990b10ee37ebf1c0ade3da64.tar yuzu-cc3aa959265480e1990b10ee37ebf1c0ade3da64.tar.gz yuzu-cc3aa959265480e1990b10ee37ebf1c0ade3da64.tar.bz2 yuzu-cc3aa959265480e1990b10ee37ebf1c0ade3da64.tar.lz yuzu-cc3aa959265480e1990b10ee37ebf1c0ade3da64.tar.xz yuzu-cc3aa959265480e1990b10ee37ebf1c0ade3da64.tar.zst yuzu-cc3aa959265480e1990b10ee37ebf1c0ade3da64.zip |
Diffstat (limited to 'src')
-rw-r--r-- | src/core/hle/service/nvflinger/nvflinger.cpp | 2 | ||||
-rw-r--r-- | src/core/hle/service/nvflinger/nvflinger.h | 7 | ||||
-rw-r--r-- | src/core/hle/service/vi/vi.cpp | 2 |
3 files changed, 11 insertions, 0 deletions
diff --git a/src/core/hle/service/nvflinger/nvflinger.cpp b/src/core/hle/service/nvflinger/nvflinger.cpp index aaf28995d..b97f71350 100644 --- a/src/core/hle/service/nvflinger/nvflinger.cpp +++ b/src/core/hle/service/nvflinger/nvflinger.cpp @@ -36,10 +36,12 @@ NVFlinger::NVFlinger(Core::System& system) : system(system) { displays.emplace_back(2, "Edid", system); displays.emplace_back(3, "Internal", system); displays.emplace_back(4, "Null", system); + guard = std::make_shared<std::mutex>(); // Schedule the screen composition events composition_event = Core::Timing::CreateEvent("ScreenComposition", [this](u64 userdata, s64 ns_late) { + Lock(); Compose(); const auto ticks = GetNextTicks(); this->system.CoreTiming().ScheduleEvent(std::max<s64>(0LL, ticks - ns_late), diff --git a/src/core/hle/service/nvflinger/nvflinger.h b/src/core/hle/service/nvflinger/nvflinger.h index 57a21f33b..02c081494 100644 --- a/src/core/hle/service/nvflinger/nvflinger.h +++ b/src/core/hle/service/nvflinger/nvflinger.h @@ -5,6 +5,7 @@ #pragma once #include <memory> +#include <mutex> #include <optional> #include <string> #include <string_view> @@ -79,6 +80,10 @@ public: s64 GetNextTicks() const; + std::unique_lock<std::mutex> Lock() { + return std::unique_lock{*guard}; + } + private: /// Finds the display identified by the specified ID. VI::Display* FindDisplay(u64 display_id); @@ -108,6 +113,8 @@ private: /// Event that handles screen composition. std::shared_ptr<Core::Timing::EventType> composition_event; + std::shared_ptr<std::mutex> guard; + Core::System& system; }; diff --git a/src/core/hle/service/vi/vi.cpp b/src/core/hle/service/vi/vi.cpp index 46e14c2a3..157092074 100644 --- a/src/core/hle/service/vi/vi.cpp +++ b/src/core/hle/service/vi/vi.cpp @@ -511,6 +511,7 @@ private: LOG_DEBUG(Service_VI, "called. id=0x{:08X} transaction={:X}, flags=0x{:08X}", id, static_cast<u32>(transaction), flags); + nv_flinger->Lock(); auto& buffer_queue = nv_flinger->FindBufferQueue(id); switch (transaction) { @@ -550,6 +551,7 @@ private: [=](std::shared_ptr<Kernel::Thread> thread, Kernel::HLERequestContext& ctx, Kernel::ThreadWakeupReason reason) { // Repeat TransactParcel DequeueBuffer when a buffer is available + nv_flinger->Lock(); auto& buffer_queue = nv_flinger->FindBufferQueue(id); auto result = buffer_queue.DequeueBuffer(width, height); ASSERT_MSG(result != std::nullopt, "Could not dequeue buffer."); |