diff options
| author | 2022-04-07 19:31:32 +0100 | |
|---|---|---|
| committer | 2022-04-07 19:44:07 +0100 | |
| commit | 1f275eb0777d5274cea85585c7eb2f8afc63f925 (patch) | |
| tree | 9d54a4bf0e4585e25195791e88631054b00fbd02 | |
| parent | common: Replace lock_guard with scoped_lock (diff) | |
| download | yuzu-1f275eb0777d5274cea85585c7eb2f8afc63f925.tar.gz yuzu-1f275eb0777d5274cea85585c7eb2f8afc63f925.tar.xz yuzu-1f275eb0777d5274cea85585c7eb2f8afc63f925.zip | |
core/hle: Replace lock_guard with scoped_lock
| -rw-r--r-- | src/core/hle/kernel/kernel.cpp | 16 | ||||
| -rw-r--r-- | src/core/hle/kernel/time_manager.cpp | 4 | ||||
| -rw-r--r-- | src/core/hle/service/hid/controllers/npad.cpp | 2 | ||||
| -rw-r--r-- | src/core/hle/service/nvflinger/hos_binder_driver_server.cpp | 4 |
4 files changed, 13 insertions, 13 deletions
diff --git a/src/core/hle/kernel/kernel.cpp b/src/core/hle/kernel/kernel.cpp index 6387d0c29..134a0b8e9 100644 --- a/src/core/hle/kernel/kernel.cpp +++ b/src/core/hle/kernel/kernel.cpp | |||
| @@ -99,7 +99,7 @@ struct KernelCore::Impl { | |||
| 99 | // Close all open server ports. | 99 | // Close all open server ports. |
| 100 | std::unordered_set<KServerPort*> server_ports_; | 100 | std::unordered_set<KServerPort*> server_ports_; |
| 101 | { | 101 | { |
| 102 | std::lock_guard lk(server_ports_lock); | 102 | std::scoped_lock lk{server_ports_lock}; |
| 103 | server_ports_ = server_ports; | 103 | server_ports_ = server_ports; |
| 104 | server_ports.clear(); | 104 | server_ports.clear(); |
| 105 | } | 105 | } |
| @@ -157,7 +157,7 @@ struct KernelCore::Impl { | |||
| 157 | 157 | ||
| 158 | // Close kernel objects that were not freed on shutdown | 158 | // Close kernel objects that were not freed on shutdown |
| 159 | { | 159 | { |
| 160 | std::lock_guard lk(registered_in_use_objects_lock); | 160 | std::scoped_lock lk{registered_in_use_objects_lock}; |
| 161 | if (registered_in_use_objects.size()) { | 161 | if (registered_in_use_objects.size()) { |
| 162 | for (auto& object : registered_in_use_objects) { | 162 | for (auto& object : registered_in_use_objects) { |
| 163 | object->Close(); | 163 | object->Close(); |
| @@ -178,7 +178,7 @@ struct KernelCore::Impl { | |||
| 178 | 178 | ||
| 179 | // Track kernel objects that were not freed on shutdown | 179 | // Track kernel objects that were not freed on shutdown |
| 180 | { | 180 | { |
| 181 | std::lock_guard lk(registered_objects_lock); | 181 | std::scoped_lock lk{registered_objects_lock}; |
| 182 | if (registered_objects.size()) { | 182 | if (registered_objects.size()) { |
| 183 | LOG_DEBUG(Kernel, "{} kernel objects were dangling on shutdown!", | 183 | LOG_DEBUG(Kernel, "{} kernel objects were dangling on shutdown!", |
| 184 | registered_objects.size()); | 184 | registered_objects.size()); |
| @@ -660,7 +660,7 @@ struct KernelCore::Impl { | |||
| 660 | 660 | ||
| 661 | KClientPort* port = &search->second(system.ServiceManager(), system); | 661 | KClientPort* port = &search->second(system.ServiceManager(), system); |
| 662 | { | 662 | { |
| 663 | std::lock_guard lk(server_ports_lock); | 663 | std::scoped_lock lk{server_ports_lock}; |
| 664 | server_ports.insert(&port->GetParent()->GetServerPort()); | 664 | server_ports.insert(&port->GetParent()->GetServerPort()); |
| 665 | } | 665 | } |
| 666 | return port; | 666 | return port; |
| @@ -929,22 +929,22 @@ KClientPort* KernelCore::CreateNamedServicePort(std::string name) { | |||
| 929 | } | 929 | } |
| 930 | 930 | ||
| 931 | void KernelCore::RegisterKernelObject(KAutoObject* object) { | 931 | void KernelCore::RegisterKernelObject(KAutoObject* object) { |
| 932 | std::lock_guard lk(impl->registered_objects_lock); | 932 | std::scoped_lock lk{impl->registered_objects_lock}; |
| 933 | impl->registered_objects.insert(object); | 933 | impl->registered_objects.insert(object); |
| 934 | } | 934 | } |
| 935 | 935 | ||
| 936 | void KernelCore::UnregisterKernelObject(KAutoObject* object) { | 936 | void KernelCore::UnregisterKernelObject(KAutoObject* object) { |
| 937 | std::lock_guard lk(impl->registered_objects_lock); | 937 | std::scoped_lock lk{impl->registered_objects_lock}; |
| 938 | impl->registered_objects.erase(object); | 938 | impl->registered_objects.erase(object); |
| 939 | } | 939 | } |
| 940 | 940 | ||
| 941 | void KernelCore::RegisterInUseObject(KAutoObject* object) { | 941 | void KernelCore::RegisterInUseObject(KAutoObject* object) { |
| 942 | std::lock_guard lk(impl->registered_in_use_objects_lock); | 942 | std::scoped_lock lk{impl->registered_in_use_objects_lock}; |
| 943 | impl->registered_in_use_objects.insert(object); | 943 | impl->registered_in_use_objects.insert(object); |
| 944 | } | 944 | } |
| 945 | 945 | ||
| 946 | void KernelCore::UnregisterInUseObject(KAutoObject* object) { | 946 | void KernelCore::UnregisterInUseObject(KAutoObject* object) { |
| 947 | std::lock_guard lk(impl->registered_in_use_objects_lock); | 947 | std::scoped_lock lk{impl->registered_in_use_objects_lock}; |
| 948 | impl->registered_in_use_objects.erase(object); | 948 | impl->registered_in_use_objects.erase(object); |
| 949 | } | 949 | } |
| 950 | 950 | ||
diff --git a/src/core/hle/kernel/time_manager.cpp b/src/core/hle/kernel/time_manager.cpp index aa985d820..5b8fe8eae 100644 --- a/src/core/hle/kernel/time_manager.cpp +++ b/src/core/hle/kernel/time_manager.cpp | |||
| @@ -24,7 +24,7 @@ TimeManager::TimeManager(Core::System& system_) : system{system_} { | |||
| 24 | } | 24 | } |
| 25 | 25 | ||
| 26 | void TimeManager::ScheduleTimeEvent(KThread* thread, s64 nanoseconds) { | 26 | void TimeManager::ScheduleTimeEvent(KThread* thread, s64 nanoseconds) { |
| 27 | std::lock_guard lock{mutex}; | 27 | std::scoped_lock lock{mutex}; |
| 28 | if (nanoseconds > 0) { | 28 | if (nanoseconds > 0) { |
| 29 | ASSERT(thread); | 29 | ASSERT(thread); |
| 30 | ASSERT(thread->GetState() != ThreadState::Runnable); | 30 | ASSERT(thread->GetState() != ThreadState::Runnable); |
| @@ -35,7 +35,7 @@ void TimeManager::ScheduleTimeEvent(KThread* thread, s64 nanoseconds) { | |||
| 35 | } | 35 | } |
| 36 | 36 | ||
| 37 | void TimeManager::UnscheduleTimeEvent(KThread* thread) { | 37 | void TimeManager::UnscheduleTimeEvent(KThread* thread) { |
| 38 | std::lock_guard lock{mutex}; | 38 | std::scoped_lock lock{mutex}; |
| 39 | system.CoreTiming().UnscheduleEvent(time_manager_event_type, | 39 | system.CoreTiming().UnscheduleEvent(time_manager_event_type, |
| 40 | reinterpret_cast<uintptr_t>(thread)); | 40 | reinterpret_cast<uintptr_t>(thread)); |
| 41 | } | 41 | } |
diff --git a/src/core/hle/service/hid/controllers/npad.cpp b/src/core/hle/service/hid/controllers/npad.cpp index aa6cb34b7..4e17a952e 100644 --- a/src/core/hle/service/hid/controllers/npad.cpp +++ b/src/core/hle/service/hid/controllers/npad.cpp | |||
| @@ -318,7 +318,7 @@ void Controller_NPad::OnRelease() { | |||
| 318 | } | 318 | } |
| 319 | 319 | ||
| 320 | void Controller_NPad::RequestPadStateUpdate(Core::HID::NpadIdType npad_id) { | 320 | void Controller_NPad::RequestPadStateUpdate(Core::HID::NpadIdType npad_id) { |
| 321 | std::lock_guard lock{mutex}; | 321 | std::scoped_lock lock{mutex}; |
| 322 | auto& controller = GetControllerFromNpadIdType(npad_id); | 322 | auto& controller = GetControllerFromNpadIdType(npad_id); |
| 323 | const auto controller_type = controller.device->GetNpadStyleIndex(); | 323 | const auto controller_type = controller.device->GetNpadStyleIndex(); |
| 324 | if (!controller.device->IsConnected()) { | 324 | if (!controller.device->IsConnected()) { |
diff --git a/src/core/hle/service/nvflinger/hos_binder_driver_server.cpp b/src/core/hle/service/nvflinger/hos_binder_driver_server.cpp index 0c937d682..094ba2542 100644 --- a/src/core/hle/service/nvflinger/hos_binder_driver_server.cpp +++ b/src/core/hle/service/nvflinger/hos_binder_driver_server.cpp | |||
| @@ -14,7 +14,7 @@ HosBinderDriverServer::HosBinderDriverServer(Core::System& system_) | |||
| 14 | HosBinderDriverServer::~HosBinderDriverServer() {} | 14 | HosBinderDriverServer::~HosBinderDriverServer() {} |
| 15 | 15 | ||
| 16 | u64 HosBinderDriverServer::RegisterProducer(std::unique_ptr<android::IBinder>&& binder) { | 16 | u64 HosBinderDriverServer::RegisterProducer(std::unique_ptr<android::IBinder>&& binder) { |
| 17 | std::lock_guard lk{lock}; | 17 | std::scoped_lock lk{lock}; |
| 18 | 18 | ||
| 19 | last_id++; | 19 | last_id++; |
| 20 | 20 | ||
| @@ -24,7 +24,7 @@ u64 HosBinderDriverServer::RegisterProducer(std::unique_ptr<android::IBinder>&& | |||
| 24 | } | 24 | } |
| 25 | 25 | ||
| 26 | android::IBinder* HosBinderDriverServer::TryGetProducer(u64 id) { | 26 | android::IBinder* HosBinderDriverServer::TryGetProducer(u64 id) { |
| 27 | std::lock_guard lk{lock}; | 27 | std::scoped_lock lk{lock}; |
| 28 | 28 | ||
| 29 | if (auto search = producers.find(id); search != producers.end()) { | 29 | if (auto search = producers.find(id); search != producers.end()) { |
| 30 | return search->second.get(); | 30 | return search->second.get(); |