summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/core/hle/service/nvdrv/devices/nvhost_as_gpu.cpp5
-rw-r--r--src/core/hle/service/nvdrv/devices/nvmap.cpp13
-rw-r--r--src/core/hle/service/nvdrv/devices/nvmap.h2
-rw-r--r--src/video_core/engines/maxwell_dma.cpp6
-rw-r--r--src/video_core/memory_manager.cpp4
-rw-r--r--src/video_core/renderer_opengl/gl_rasterizer.cpp6
-rw-r--r--src/video_core/renderer_opengl/gl_rasterizer_cache.cpp5
7 files changed, 33 insertions, 8 deletions
diff --git a/src/core/hle/service/nvdrv/devices/nvhost_as_gpu.cpp b/src/core/hle/service/nvdrv/devices/nvhost_as_gpu.cpp
index c1eea861d..7d8ed6920 100644
--- a/src/core/hle/service/nvdrv/devices/nvhost_as_gpu.cpp
+++ b/src/core/hle/service/nvdrv/devices/nvhost_as_gpu.cpp
@@ -8,6 +8,8 @@
8#include "core/core.h" 8#include "core/core.h"
9#include "core/hle/service/nvdrv/devices/nvhost_as_gpu.h" 9#include "core/hle/service/nvdrv/devices/nvhost_as_gpu.h"
10#include "core/hle/service/nvdrv/devices/nvmap.h" 10#include "core/hle/service/nvdrv/devices/nvmap.h"
11#include "video_core/renderer_base.h"
12#include "video_core/video_core.h"
11 13
12namespace Service::Nvidia::Devices { 14namespace Service::Nvidia::Devices {
13 15
@@ -154,6 +156,9 @@ u32 nvhost_as_gpu::UnmapBuffer(const std::vector<u8>& input, std::vector<u8>& ou
154 156
155 ASSERT_MSG(itr != buffer_mappings.end(), "Tried to unmap invalid mapping"); 157 ASSERT_MSG(itr != buffer_mappings.end(), "Tried to unmap invalid mapping");
156 158
159 // Remove this memory region from the rasterizer cache.
160 VideoCore::g_renderer->Rasterizer()->FlushAndInvalidateRegion(params.offset, itr->second.size);
161
157 params.offset = gpu.memory_manager->UnmapBuffer(params.offset, itr->second.size); 162 params.offset = gpu.memory_manager->UnmapBuffer(params.offset, itr->second.size);
158 163
159 buffer_mappings.erase(itr->second.offset); 164 buffer_mappings.erase(itr->second.offset);
diff --git a/src/core/hle/service/nvdrv/devices/nvmap.cpp b/src/core/hle/service/nvdrv/devices/nvmap.cpp
index 23fe98190..2fc7c87e0 100644
--- a/src/core/hle/service/nvdrv/devices/nvmap.cpp
+++ b/src/core/hle/service/nvdrv/devices/nvmap.cpp
@@ -148,6 +148,7 @@ u32 nvmap::IocParam(const std::vector<u8>& input, std::vector<u8>& output) {
148} 148}
149 149
150u32 nvmap::IocFree(const std::vector<u8>& input, std::vector<u8>& output) { 150u32 nvmap::IocFree(const std::vector<u8>& input, std::vector<u8>& output) {
151 // TODO(Subv): These flags are unconfirmed.
151 enum FreeFlags { 152 enum FreeFlags {
152 Freed = 0, 153 Freed = 0,
153 NotFreedYet = 1, 154 NotFreedYet = 1,
@@ -161,15 +162,21 @@ u32 nvmap::IocFree(const std::vector<u8>& input, std::vector<u8>& output) {
161 auto itr = handles.find(params.handle); 162 auto itr = handles.find(params.handle);
162 ASSERT(itr != handles.end()); 163 ASSERT(itr != handles.end());
163 164
165 ASSERT(itr->second->refcount > 0);
166
164 itr->second->refcount--; 167 itr->second->refcount--;
165 168
166 params.refcount = itr->second->refcount;
167 params.size = itr->second->size; 169 params.size = itr->second->size;
168 170
169 if (itr->second->refcount == 0) 171 if (itr->second->refcount == 0) {
170 params.flags = Freed; 172 params.flags = Freed;
171 else 173 // The address of the nvmap is written to the output if we're finally freeing it, otherwise
174 // 0 is written.
175 params.address = itr->second->addr;
176 } else {
172 params.flags = NotFreedYet; 177 params.flags = NotFreedYet;
178 params.address = 0;
179 }
173 180
174 handles.erase(params.handle); 181 handles.erase(params.handle);
175 182
diff --git a/src/core/hle/service/nvdrv/devices/nvmap.h b/src/core/hle/service/nvdrv/devices/nvmap.h
index 39fafaa7c..f2eec6409 100644
--- a/src/core/hle/service/nvdrv/devices/nvmap.h
+++ b/src/core/hle/service/nvdrv/devices/nvmap.h
@@ -94,7 +94,7 @@ private:
94 struct IocFreeParams { 94 struct IocFreeParams {
95 u32_le handle; 95 u32_le handle;
96 INSERT_PADDING_BYTES(4); 96 INSERT_PADDING_BYTES(4);
97 u64_le refcount; 97 u64_le address;
98 u32_le size; 98 u32_le size;
99 u32_le flags; 99 u32_le flags;
100 }; 100 };
diff --git a/src/video_core/engines/maxwell_dma.cpp b/src/video_core/engines/maxwell_dma.cpp
index 442138988..c298f0bfb 100644
--- a/src/video_core/engines/maxwell_dma.cpp
+++ b/src/video_core/engines/maxwell_dma.cpp
@@ -49,7 +49,11 @@ void MaxwellDMA::HandleCopy() {
49 ASSERT(regs.src_params.pos_y == 0); 49 ASSERT(regs.src_params.pos_y == 0);
50 ASSERT(regs.dst_params.pos_x == 0); 50 ASSERT(regs.dst_params.pos_x == 0);
51 ASSERT(regs.dst_params.pos_y == 0); 51 ASSERT(regs.dst_params.pos_y == 0);
52 ASSERT(regs.exec.is_dst_linear != regs.exec.is_src_linear); 52
53 if (regs.exec.is_dst_linear == regs.exec.is_src_linear) {
54 Memory::CopyBlock(dest_cpu, source_cpu, regs.x_count * regs.y_count);
55 return;
56 }
53 57
54 u8* src_buffer = Memory::GetPointer(source_cpu); 58 u8* src_buffer = Memory::GetPointer(source_cpu);
55 u8* dst_buffer = Memory::GetPointer(dest_cpu); 59 u8* dst_buffer = Memory::GetPointer(dest_cpu);
diff --git a/src/video_core/memory_manager.cpp b/src/video_core/memory_manager.cpp
index 5cefce9fc..2f814a184 100644
--- a/src/video_core/memory_manager.cpp
+++ b/src/video_core/memory_manager.cpp
@@ -100,9 +100,9 @@ boost::optional<GPUVAddr> MemoryManager::FindFreeBlock(u64 size, u64 align) {
100 100
101boost::optional<VAddr> MemoryManager::GpuToCpuAddress(GPUVAddr gpu_addr) { 101boost::optional<VAddr> MemoryManager::GpuToCpuAddress(GPUVAddr gpu_addr) {
102 VAddr base_addr = PageSlot(gpu_addr); 102 VAddr base_addr = PageSlot(gpu_addr);
103 ASSERT(base_addr != static_cast<u64>(PageStatus::Unmapped));
104 103
105 if (base_addr == static_cast<u64>(PageStatus::Allocated)) { 104 if (base_addr == static_cast<u64>(PageStatus::Allocated) ||
105 base_addr == static_cast<u64>(PageStatus::Unmapped)) {
106 return {}; 106 return {};
107 } 107 }
108 108
diff --git a/src/video_core/renderer_opengl/gl_rasterizer.cpp b/src/video_core/renderer_opengl/gl_rasterizer.cpp
index b10a860a6..324c28d1b 100644
--- a/src/video_core/renderer_opengl/gl_rasterizer.cpp
+++ b/src/video_core/renderer_opengl/gl_rasterizer.cpp
@@ -638,7 +638,11 @@ u32 RasterizerOpenGL::SetupTextures(Maxwell::ShaderStage stage, GLuint program,
638 glProgramUniform1i(program, uniform, current_bindpoint); 638 glProgramUniform1i(program, uniform, current_bindpoint);
639 639
640 const auto texture = maxwell3d.GetStageTexture(entry.GetStage(), entry.GetOffset()); 640 const auto texture = maxwell3d.GetStageTexture(entry.GetStage(), entry.GetOffset());
641 ASSERT(texture.enabled); 641
642 if (!texture.enabled) {
643 state.texture_units[current_bindpoint].texture_2d = 0;
644 continue;
645 }
642 646
643 texture_samplers[current_bindpoint].SyncWithConfig(texture.tsc); 647 texture_samplers[current_bindpoint].SyncWithConfig(texture.tsc);
644 Surface surface = res_cache.GetTextureSurface(texture); 648 Surface surface = res_cache.GetTextureSurface(texture);
diff --git a/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp b/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp
index ae48378f3..9410ddb4e 100644
--- a/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp
+++ b/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp
@@ -461,6 +461,11 @@ Surface RasterizerCacheOpenGL::GetSurface(const SurfaceParams& params) {
461 return {}; 461 return {};
462 } 462 }
463 463
464 const auto& gpu = Core::System::GetInstance().GPU();
465 // Don't try to create any entries in the cache if the address of the texture is invalid.
466 if (gpu.memory_manager->GpuToCpuAddress(params.addr) == boost::none)
467 return {};
468
464 // Check for an exact match in existing surfaces 469 // Check for an exact match in existing surfaces
465 const auto& surface_key{SurfaceKey::Create(params)}; 470 const auto& surface_key{SurfaceKey::Create(params)};
466 const auto& search{surface_cache.find(surface_key)}; 471 const auto& search{surface_cache.find(surface_key)};