diff options
Diffstat (limited to 'src/core')
| -rw-r--r-- | src/core/device_memory_manager.h | 3 | ||||
| -rw-r--r-- | src/core/device_memory_manager.inc | 11 |
2 files changed, 3 insertions, 11 deletions
diff --git a/src/core/device_memory_manager.h b/src/core/device_memory_manager.h index ffeed46cc..63823602c 100644 --- a/src/core/device_memory_manager.h +++ b/src/core/device_memory_manager.h | |||
| @@ -10,6 +10,7 @@ | |||
| 10 | #include <mutex> | 10 | #include <mutex> |
| 11 | 11 | ||
| 12 | #include "common/common_types.h" | 12 | #include "common/common_types.h" |
| 13 | #include "common/range_mutex.h" | ||
| 13 | #include "common/scratch_buffer.h" | 14 | #include "common/scratch_buffer.h" |
| 14 | #include "common/virtual_buffer.h" | 15 | #include "common/virtual_buffer.h" |
| 15 | 16 | ||
| @@ -204,7 +205,7 @@ private: | |||
| 204 | (1ULL << (device_virtual_bits - page_bits)) / subentries; | 205 | (1ULL << (device_virtual_bits - page_bits)) / subentries; |
| 205 | using CachedPages = std::array<CounterEntry, num_counter_entries>; | 206 | using CachedPages = std::array<CounterEntry, num_counter_entries>; |
| 206 | std::unique_ptr<CachedPages> cached_pages; | 207 | std::unique_ptr<CachedPages> cached_pages; |
| 207 | std::mutex counter_guard; | 208 | Common::RangeMutex counter_guard; |
| 208 | std::mutex mapping_guard; | 209 | std::mutex mapping_guard; |
| 209 | }; | 210 | }; |
| 210 | 211 | ||
diff --git a/src/core/device_memory_manager.inc b/src/core/device_memory_manager.inc index eab8a2731..0a59000aa 100644 --- a/src/core/device_memory_manager.inc +++ b/src/core/device_memory_manager.inc | |||
| @@ -508,12 +508,7 @@ void DeviceMemoryManager<Traits>::UnregisterProcess(Asid asid) { | |||
| 508 | 508 | ||
| 509 | template <typename Traits> | 509 | template <typename Traits> |
| 510 | void DeviceMemoryManager<Traits>::UpdatePagesCachedCount(DAddr addr, size_t size, s32 delta) { | 510 | void DeviceMemoryManager<Traits>::UpdatePagesCachedCount(DAddr addr, size_t size, s32 delta) { |
| 511 | std::unique_lock<std::mutex> lk(counter_guard, std::defer_lock); | 511 | Common::ScopedRangeLock lk(counter_guard, addr, size); |
| 512 | const auto Lock = [&] { | ||
| 513 | if (!lk) { | ||
| 514 | lk.lock(); | ||
| 515 | } | ||
| 516 | }; | ||
| 517 | u64 uncache_begin = 0; | 512 | u64 uncache_begin = 0; |
| 518 | u64 cache_begin = 0; | 513 | u64 cache_begin = 0; |
| 519 | u64 uncache_bytes = 0; | 514 | u64 uncache_bytes = 0; |
| @@ -548,7 +543,6 @@ void DeviceMemoryManager<Traits>::UpdatePagesCachedCount(DAddr addr, size_t size | |||
| 548 | } | 543 | } |
| 549 | uncache_bytes += Memory::YUZU_PAGESIZE; | 544 | uncache_bytes += Memory::YUZU_PAGESIZE; |
| 550 | } else if (uncache_bytes > 0) { | 545 | } else if (uncache_bytes > 0) { |
| 551 | Lock(); | ||
| 552 | MarkRegionCaching(memory_device_inter, uncache_begin << Memory::YUZU_PAGEBITS, | 546 | MarkRegionCaching(memory_device_inter, uncache_begin << Memory::YUZU_PAGEBITS, |
| 553 | uncache_bytes, false); | 547 | uncache_bytes, false); |
| 554 | uncache_bytes = 0; | 548 | uncache_bytes = 0; |
| @@ -559,7 +553,6 @@ void DeviceMemoryManager<Traits>::UpdatePagesCachedCount(DAddr addr, size_t size | |||
| 559 | } | 553 | } |
| 560 | cache_bytes += Memory::YUZU_PAGESIZE; | 554 | cache_bytes += Memory::YUZU_PAGESIZE; |
| 561 | } else if (cache_bytes > 0) { | 555 | } else if (cache_bytes > 0) { |
| 562 | Lock(); | ||
| 563 | MarkRegionCaching(memory_device_inter, cache_begin << Memory::YUZU_PAGEBITS, cache_bytes, | 556 | MarkRegionCaching(memory_device_inter, cache_begin << Memory::YUZU_PAGEBITS, cache_bytes, |
| 564 | true); | 557 | true); |
| 565 | cache_bytes = 0; | 558 | cache_bytes = 0; |
| @@ -567,12 +560,10 @@ void DeviceMemoryManager<Traits>::UpdatePagesCachedCount(DAddr addr, size_t size | |||
| 567 | vpage++; | 560 | vpage++; |
| 568 | } | 561 | } |
| 569 | if (uncache_bytes > 0) { | 562 | if (uncache_bytes > 0) { |
| 570 | Lock(); | ||
| 571 | MarkRegionCaching(memory_device_inter, uncache_begin << Memory::YUZU_PAGEBITS, uncache_bytes, | 563 | MarkRegionCaching(memory_device_inter, uncache_begin << Memory::YUZU_PAGEBITS, uncache_bytes, |
| 572 | false); | 564 | false); |
| 573 | } | 565 | } |
| 574 | if (cache_bytes > 0) { | 566 | if (cache_bytes > 0) { |
| 575 | Lock(); | ||
| 576 | MarkRegionCaching(memory_device_inter, cache_begin << Memory::YUZU_PAGEBITS, cache_bytes, | 567 | MarkRegionCaching(memory_device_inter, cache_begin << Memory::YUZU_PAGEBITS, cache_bytes, |
| 577 | true); | 568 | true); |
| 578 | } | 569 | } |