From ef390c5c40af42bffdda3264ec3e10c1e827a229 Mon Sep 17 00:00:00 2001 From: "Unknown W. Brackets" Date: Thu, 19 Jun 2014 22:52:45 -0700 Subject: [PATCH] Double check upload/download are VRAM addresses. --- GPU/GLES/Framebuffer.cpp | 2 +- GPU/GLES/GLES_GPU.cpp | 10 ++++++++-- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/GPU/GLES/Framebuffer.cpp b/GPU/GLES/Framebuffer.cpp index 6fc83e0e90..e342ac084d 100644 --- a/GPU/GLES/Framebuffer.cpp +++ b/GPU/GLES/Framebuffer.cpp @@ -982,7 +982,7 @@ void FramebufferManager::DoSetRenderFrameBuffer() { u32 byteSize = FramebufferByteSize(vfb); u32 fb_address_mem = (fb_address & 0x3FFFFFFF) | 0x04000000; - if (fb_address_mem + byteSize > framebufRangeEnd_) { + if (Memory::IsVRAMAddress(fb_address_mem) && fb_address_mem + byteSize > framebufRangeEnd_) { framebufRangeEnd_ = fb_address_mem + byteSize; } diff --git a/GPU/GLES/GLES_GPU.cpp b/GPU/GLES/GLES_GPU.cpp index 78b03da430..d38a423a1e 100644 --- a/GPU/GLES/GLES_GPU.cpp +++ b/GPU/GLES/GLES_GPU.cpp @@ -2061,13 +2061,19 @@ bool GLES_GPU::PerformMemorySet(u32 dest, u8 v, int size) { bool GLES_GPU::PerformMemoryDownload(u32 dest, int size) { // Cheat a bit to force a download of the framebuffer. // VRAM + 0x00400000 is simply a VRAM mirror. - return gpu->PerformMemoryCopy(dest ^ 0x00400000, dest, size); + if (Memory::IsVRAMAddress(dest)) { + return gpu->PerformMemoryCopy(dest ^ 0x00400000, dest, size); + } + return false; } bool GLES_GPU::PerformMemoryUpload(u32 dest, int size) { // Cheat a bit to force an upload of the framebuffer. // VRAM + 0x00400000 is simply a VRAM mirror. - return gpu->PerformMemoryCopy(dest, dest ^ 0x00400000, size); + if (Memory::IsVRAMAddress(dest)) { + return gpu->PerformMemoryCopy(dest, dest ^ 0x00400000, size); + } + return false; } bool GLES_GPU::PerformStencilUpload(u32 dest, int size) {