diff options
| author | bunnei <bunneidev@gmail.com> | 2018-11-23 12:58:55 -0500 |
|---|---|---|
| committer | bunnei <bunneidev@gmail.com> | 2018-11-23 12:58:55 -0500 |
| commit | 0b1842294f5f43ea2ee42bb8afd26e57c9250f4b (patch) | |
| tree | 97507dc052dbec1891184e512a666a791c89b633 /src/video_core/memory_manager.cpp | |
| parent | c267aea29b23f52add7c36578e55312b70480947 (diff) | |
memory_manager: Do not allow 0 to be a valid GPUVAddr.
- Fixes a bug with Undertale using 0 for a render target.
Diffstat (limited to 'src/video_core/memory_manager.cpp')
| -rw-r--r-- | src/video_core/memory_manager.cpp | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/video_core/memory_manager.cpp b/src/video_core/memory_manager.cpp index 77a20bb84..47247f097 100644 --- a/src/video_core/memory_manager.cpp +++ b/src/video_core/memory_manager.cpp @@ -9,6 +9,13 @@ namespace Tegra { +MemoryManager::MemoryManager() { + // Mark the first page as reserved, so that 0 is not a valid GPUVAddr. Otherwise, games might + // try to use 0 as a valid address, which is also used to mean nullptr. This fixes a bug with + // Undertale using 0 for a render target. + PageSlot(0) = static_cast<u64>(PageStatus::Reserved); +} + GPUVAddr MemoryManager::AllocateSpace(u64 size, u64 align) { const std::optional<GPUVAddr> gpu_addr{FindFreeBlock(0, size, align, PageStatus::Unmapped)}; |
