summaryrefslogtreecommitdiff
path: root/src/video_core/renderer_vulkan
diff options
context:
space:
mode:
authorGravatar ReinUsesLisp2021-04-06 20:14:55 -0300
committerGravatar ameerj2021-07-22 21:51:26 -0400
commite9a91bc5cc2c39b476ba8946f66930f5ab5608b2 (patch)
treead75617771ba4093c5609a505e3c8926668eb533 /src/video_core/renderer_vulkan
parentshader: Fix F2I (diff)
downloadyuzu-e9a91bc5cc2c39b476ba8946f66930f5ab5608b2.tar.gz
yuzu-e9a91bc5cc2c39b476ba8946f66930f5ab5608b2.tar.xz
yuzu-e9a91bc5cc2c39b476ba8946f66930f5ab5608b2.zip
shader: Interact texture buffers with buffer cache
Diffstat (limited to 'src/video_core/renderer_vulkan')
-rw-r--r--src/video_core/renderer_vulkan/pipeline_helper.h26
-rw-r--r--src/video_core/renderer_vulkan/vk_buffer_cache.cpp57
-rw-r--r--src/video_core/renderer_vulkan/vk_buffer_cache.h18
-rw-r--r--src/video_core/renderer_vulkan/vk_compute_pipeline.cpp30
-rw-r--r--src/video_core/renderer_vulkan/vk_graphics_pipeline.cpp31
-rw-r--r--src/video_core/renderer_vulkan/vk_texture_cache.cpp63
-rw-r--r--src/video_core/renderer_vulkan/vk_texture_cache.h30
7 files changed, 140 insertions, 115 deletions
diff --git a/src/video_core/renderer_vulkan/pipeline_helper.h b/src/video_core/renderer_vulkan/pipeline_helper.h
index decf0d32c..cff93cc60 100644
--- a/src/video_core/renderer_vulkan/pipeline_helper.h
+++ b/src/video_core/renderer_vulkan/pipeline_helper.h
@@ -24,7 +24,8 @@ struct TextureHandle {
24 [[likely]] if (via_header_index) { 24 [[likely]] if (via_header_index) {
25 image = data; 25 image = data;
26 sampler = data; 26 sampler = data;
27 } else { 27 }
28 else {
28 const Tegra::Texture::TextureHandle handle{data}; 29 const Tegra::Texture::TextureHandle handle{data};
29 image = handle.tic_id; 30 image = handle.tic_id;
30 sampler = via_header_index ? image : handle.tsc_id.Value(); 31 sampler = via_header_index ? image : handle.tsc_id.Value();
@@ -90,12 +91,12 @@ public:
90 for ([[maybe_unused]] const auto& desc : info.storage_buffers_descriptors) { 91 for ([[maybe_unused]] const auto& desc : info.storage_buffers_descriptors) {
91 Add(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER, stage); 92 Add(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER, stage);
92 } 93 }
94 for ([[maybe_unused]] const auto& desc : info.texture_buffer_descriptors) {
95 Add(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER, stage);
96 }
93 for ([[maybe_unused]] const auto& desc : info.texture_descriptors) { 97 for ([[maybe_unused]] const auto& desc : info.texture_descriptors) {
94 Add(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, stage); 98 Add(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, stage);
95 } 99 }
96 for (const auto& desc : info.texture_buffer_descriptors) {
97 Add(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER, stage);
98 }
99 } 100 }
100 101
101private: 102private:
@@ -156,20 +157,15 @@ inline VideoCommon::ImageViewType CastType(Shader::TextureType type) {
156 return {}; 157 return {};
157} 158}
158 159
159inline void PushImageDescriptors(const Shader::Info& info, const VkSampler* samplers, 160inline void PushImageDescriptors(const Shader::Info& info, const VkSampler*& samplers,
160 const ImageId* image_view_ids, TextureCache& texture_cache, 161 const ImageId*& image_view_ids, TextureCache& texture_cache,
161 VKUpdateDescriptorQueue& update_descriptor_queue, size_t& index) { 162 VKUpdateDescriptorQueue& update_descriptor_queue) {
163 image_view_ids += info.texture_buffer_descriptors.size();
162 for (const auto& desc : info.texture_descriptors) { 164 for (const auto& desc : info.texture_descriptors) {
163 const VkSampler sampler{samplers[index]}; 165 const VkSampler sampler{*(samplers++)};
164 ImageView& image_view{texture_cache.GetImageView(image_view_ids[index])}; 166 ImageView& image_view{texture_cache.GetImageView(*(image_view_ids++))};
165 const VkImageView vk_image_view{image_view.Handle(CastType(desc.type))}; 167 const VkImageView vk_image_view{image_view.Handle(CastType(desc.type))};
166 update_descriptor_queue.AddSampledImage(vk_image_view, sampler); 168 update_descriptor_queue.AddSampledImage(vk_image_view, sampler);
167 ++index;
168 }
169 for (const auto& desc : info.texture_buffer_descriptors) {
170 ImageView& image_view{texture_cache.GetImageView(image_view_ids[index])};
171 update_descriptor_queue.AddTexelBuffer(image_view.BufferView());
172 ++index;
173 } 169 }
174} 170}
175 171
diff --git a/src/video_core/renderer_vulkan/vk_buffer_cache.cpp b/src/video_core/renderer_vulkan/vk_buffer_cache.cpp
index 0def1e769..cdda56ab1 100644
--- a/src/video_core/renderer_vulkan/vk_buffer_cache.cpp
+++ b/src/video_core/renderer_vulkan/vk_buffer_cache.cpp
@@ -67,25 +67,50 @@ Buffer::Buffer(BufferCacheRuntime&, VideoCommon::NullBufferParams null_params)
67 67
68Buffer::Buffer(BufferCacheRuntime& runtime, VideoCore::RasterizerInterface& rasterizer_, 68Buffer::Buffer(BufferCacheRuntime& runtime, VideoCore::RasterizerInterface& rasterizer_,
69 VAddr cpu_addr_, u64 size_bytes_) 69 VAddr cpu_addr_, u64 size_bytes_)
70 : VideoCommon::BufferBase<VideoCore::RasterizerInterface>(rasterizer_, cpu_addr_, size_bytes_) { 70 : VideoCommon::BufferBase<VideoCore::RasterizerInterface>(rasterizer_, cpu_addr_, size_bytes_),
71 buffer = runtime.device.GetLogical().CreateBuffer(VkBufferCreateInfo{ 71 device{&runtime.device},
72 .sType = VK_STRUCTURE_TYPE_BUFFER_CREATE_INFO, 72 buffer{device->GetLogical().CreateBuffer({
73 .pNext = nullptr, 73 .sType = VK_STRUCTURE_TYPE_BUFFER_CREATE_INFO,
74 .flags = 0, 74 .pNext = nullptr,
75 .size = SizeBytes(), 75 .flags = 0,
76 .usage = VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT | 76 .size = SizeBytes(),
77 VK_BUFFER_USAGE_UNIFORM_TEXEL_BUFFER_BIT | 77 .usage = VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT |
78 VK_BUFFER_USAGE_STORAGE_TEXEL_BUFFER_BIT | VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT | 78 VK_BUFFER_USAGE_UNIFORM_TEXEL_BUFFER_BIT |
79 VK_BUFFER_USAGE_STORAGE_BUFFER_BIT | VK_BUFFER_USAGE_INDEX_BUFFER_BIT | 79 VK_BUFFER_USAGE_STORAGE_TEXEL_BUFFER_BIT | VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT |
80 VK_BUFFER_USAGE_VERTEX_BUFFER_BIT, 80 VK_BUFFER_USAGE_STORAGE_BUFFER_BIT | VK_BUFFER_USAGE_INDEX_BUFFER_BIT |
81 .sharingMode = VK_SHARING_MODE_EXCLUSIVE, 81 VK_BUFFER_USAGE_VERTEX_BUFFER_BIT,
82 .queueFamilyIndexCount = 0, 82 .sharingMode = VK_SHARING_MODE_EXCLUSIVE,
83 .pQueueFamilyIndices = nullptr, 83 .queueFamilyIndexCount = 0,
84 }); 84 .pQueueFamilyIndices = nullptr,
85 })},
86 commit{runtime.memory_allocator.Commit(buffer, MemoryUsage::DeviceLocal)} {
85 if (runtime.device.HasDebuggingToolAttached()) { 87 if (runtime.device.HasDebuggingToolAttached()) {
86 buffer.SetObjectNameEXT(fmt::format("Buffer 0x{:x}", CpuAddr()).c_str()); 88 buffer.SetObjectNameEXT(fmt::format("Buffer 0x{:x}", CpuAddr()).c_str());
87 } 89 }
88 commit = runtime.memory_allocator.Commit(buffer, MemoryUsage::DeviceLocal); 90}
91
92VkBufferView Buffer::View(u32 offset, u32 size, VideoCore::Surface::PixelFormat format) {
93 const auto it{std::ranges::find_if(views, [offset, size, format](const BufferView& view) {
94 return offset == view.offset && size == view.size && format == view.format;
95 })};
96 if (it != views.end()) {
97 return *it->handle;
98 }
99 views.push_back({
100 .offset = offset,
101 .size = size,
102 .format = format,
103 .handle = device->GetLogical().CreateBufferView({
104 .sType = VK_STRUCTURE_TYPE_BUFFER_VIEW_CREATE_INFO,
105 .pNext = nullptr,
106 .flags = 0,
107 .buffer = *buffer,
108 .format = MaxwellToVK::SurfaceFormat(*device, FormatType::Buffer, false, format).format,
109 .offset = offset,
110 .range = size,
111 }),
112 });
113 return *views.back().handle;
89} 114}
90 115
91BufferCacheRuntime::BufferCacheRuntime(const Device& device_, MemoryAllocator& memory_allocator_, 116BufferCacheRuntime::BufferCacheRuntime(const Device& device_, MemoryAllocator& memory_allocator_,
diff --git a/src/video_core/renderer_vulkan/vk_buffer_cache.h b/src/video_core/renderer_vulkan/vk_buffer_cache.h
index 3bb81d5b3..ea17406dc 100644
--- a/src/video_core/renderer_vulkan/vk_buffer_cache.h
+++ b/src/video_core/renderer_vulkan/vk_buffer_cache.h
@@ -9,6 +9,7 @@
9#include "video_core/renderer_vulkan/vk_compute_pass.h" 9#include "video_core/renderer_vulkan/vk_compute_pass.h"
10#include "video_core/renderer_vulkan/vk_staging_buffer_pool.h" 10#include "video_core/renderer_vulkan/vk_staging_buffer_pool.h"
11#include "video_core/renderer_vulkan/vk_update_descriptor.h" 11#include "video_core/renderer_vulkan/vk_update_descriptor.h"
12#include "video_core/surface.h"
12#include "video_core/vulkan_common/vulkan_memory_allocator.h" 13#include "video_core/vulkan_common/vulkan_memory_allocator.h"
13#include "video_core/vulkan_common/vulkan_wrapper.h" 14#include "video_core/vulkan_common/vulkan_wrapper.h"
14 15
@@ -26,6 +27,8 @@ public:
26 explicit Buffer(BufferCacheRuntime& runtime, VideoCore::RasterizerInterface& rasterizer_, 27 explicit Buffer(BufferCacheRuntime& runtime, VideoCore::RasterizerInterface& rasterizer_,
27 VAddr cpu_addr_, u64 size_bytes_); 28 VAddr cpu_addr_, u64 size_bytes_);
28 29
30 [[nodiscard]] VkBufferView View(u32 offset, u32 size, VideoCore::Surface::PixelFormat format);
31
29 [[nodiscard]] VkBuffer Handle() const noexcept { 32 [[nodiscard]] VkBuffer Handle() const noexcept {
30 return *buffer; 33 return *buffer;
31 } 34 }
@@ -35,8 +38,17 @@ public:
35 } 38 }
36 39
37private: 40private:
41 struct BufferView {
42 u32 offset;
43 u32 size;
44 VideoCore::Surface::PixelFormat format;
45 vk::BufferView handle;
46 };
47
48 const Device* device{};
38 vk::Buffer buffer; 49 vk::Buffer buffer;
39 MemoryCommit commit; 50 MemoryCommit commit;
51 std::vector<BufferView> views;
40}; 52};
41 53
42class BufferCacheRuntime { 54class BufferCacheRuntime {
@@ -87,6 +99,11 @@ public:
87 BindBuffer(buffer, offset, size); 99 BindBuffer(buffer, offset, size);
88 } 100 }
89 101
102 void BindTextureBuffer(Buffer& buffer, u32 offset, u32 size,
103 VideoCore::Surface::PixelFormat format) {
104 update_descriptor_queue.AddTexelBuffer(buffer.View(offset, size, format));
105 }
106
90private: 107private:
91 void BindBuffer(VkBuffer buffer, u32 offset, u32 size) { 108 void BindBuffer(VkBuffer buffer, u32 offset, u32 size) {
92 update_descriptor_queue.AddBuffer(buffer, offset, size); 109 update_descriptor_queue.AddBuffer(buffer, offset, size);
@@ -123,6 +140,7 @@ struct BufferCacheParams {
123 static constexpr bool HAS_FULL_INDEX_AND_PRIMITIVE_SUPPORT = false; 140 static constexpr bool HAS_FULL_INDEX_AND_PRIMITIVE_SUPPORT = false;
124 static constexpr bool NEEDS_BIND_UNIFORM_INDEX = false; 141 static constexpr bool NEEDS_BIND_UNIFORM_INDEX = false;
125 static constexpr bool NEEDS_BIND_STORAGE_INDEX = false; 142 static constexpr bool NEEDS_BIND_STORAGE_INDEX = false;
143 static constexpr bool NEEDS_BIND_TEXTURE_BUFFER_INDEX = false;
126 static constexpr bool USE_MEMORY_MAPS = true; 144 static constexpr bool USE_MEMORY_MAPS = true;
127}; 145};
128 146
diff --git a/src/video_core/renderer_vulkan/vk_compute_pipeline.cpp b/src/video_core/renderer_vulkan/vk_compute_pipeline.cpp
index 9922cbd0f..ac47b1f3c 100644
--- a/src/video_core/renderer_vulkan/vk_compute_pipeline.cpp
+++ b/src/video_core/renderer_vulkan/vk_compute_pipeline.cpp
@@ -80,8 +80,6 @@ void ComputePipeline::Configure(Tegra::Engines::KeplerCompute& kepler_compute,
80 desc.is_written); 80 desc.is_written);
81 ++ssbo_index; 81 ++ssbo_index;
82 } 82 }
83 buffer_cache.UpdateComputeBuffers();
84 buffer_cache.BindHostComputeBuffers();
85 83
86 texture_cache.SynchronizeComputeDescriptors(); 84 texture_cache.SynchronizeComputeDescriptors();
87 85
@@ -99,6 +97,10 @@ void ComputePipeline::Configure(Tegra::Engines::KeplerCompute& kepler_compute,
99 const u32 raw_handle{gpu_memory.Read<u32>(addr)}; 97 const u32 raw_handle{gpu_memory.Read<u32>(addr)};
100 return TextureHandle(raw_handle, via_header_index); 98 return TextureHandle(raw_handle, via_header_index);
101 }}; 99 }};
100 for (const auto& desc : info.texture_buffer_descriptors) {
101 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)};
102 image_view_indices.push_back(handle.image);
103 }
102 for (const auto& desc : info.texture_descriptors) { 104 for (const auto& desc : info.texture_descriptors) {
103 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)}; 105 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)};
104 image_view_indices.push_back(handle.image); 106 image_view_indices.push_back(handle.image);
@@ -106,16 +108,26 @@ void ComputePipeline::Configure(Tegra::Engines::KeplerCompute& kepler_compute,
106 Sampler* const sampler = texture_cache.GetComputeSampler(handle.sampler); 108 Sampler* const sampler = texture_cache.GetComputeSampler(handle.sampler);
107 samplers.push_back(sampler->Handle()); 109 samplers.push_back(sampler->Handle());
108 } 110 }
109 for (const auto& desc : info.texture_buffer_descriptors) {
110 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)};
111 image_view_indices.push_back(handle.image);
112 }
113 const std::span indices_span(image_view_indices.data(), image_view_indices.size()); 111 const std::span indices_span(image_view_indices.data(), image_view_indices.size());
114 texture_cache.FillComputeImageViews(indices_span, image_view_ids); 112 texture_cache.FillComputeImageViews(indices_span, image_view_ids);
115 113
116 size_t image_index{}; 114 buffer_cache.UnbindComputeTextureBuffers();
117 PushImageDescriptors(info, samplers.data(), image_view_ids.data(), texture_cache, 115 ImageId* texture_buffer_ids{image_view_ids.data()};
118 update_descriptor_queue, image_index); 116 size_t index{};
117 for (const auto& desc : info.texture_buffer_descriptors) {
118 ASSERT(desc.count == 1);
119 ImageView& image_view = texture_cache.GetImageView(*texture_buffer_ids);
120 buffer_cache.BindComputeTextureBuffer(index, image_view.GpuAddr(), image_view.BufferSize(),
121 image_view.format);
122 ++texture_buffer_ids;
123 ++index;
124 }
125 buffer_cache.UpdateComputeBuffers();
126 buffer_cache.BindHostComputeBuffers();
127
128 const VkSampler* samplers_it{samplers.data()};
129 const ImageId* views_it{image_view_ids.data()};
130 PushImageDescriptors(info, samplers_it, views_it, texture_cache, update_descriptor_queue);
119 131
120 if (!is_built.load(std::memory_order::relaxed)) { 132 if (!is_built.load(std::memory_order::relaxed)) {
121 // Wait for the pipeline to be built 133 // Wait for the pipeline to be built
diff --git a/src/video_core/renderer_vulkan/vk_graphics_pipeline.cpp b/src/video_core/renderer_vulkan/vk_graphics_pipeline.cpp
index afdd8b371..893258b4a 100644
--- a/src/video_core/renderer_vulkan/vk_graphics_pipeline.cpp
+++ b/src/video_core/renderer_vulkan/vk_graphics_pipeline.cpp
@@ -175,6 +175,10 @@ void GraphicsPipeline::Configure(bool is_indexed) {
175 const u32 raw_handle{gpu_memory.Read<u32>(addr)}; 175 const u32 raw_handle{gpu_memory.Read<u32>(addr)};
176 return TextureHandle(raw_handle, via_header_index); 176 return TextureHandle(raw_handle, via_header_index);
177 }}; 177 }};
178 for (const auto& desc : info.texture_buffer_descriptors) {
179 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)};
180 image_view_indices.push_back(handle.image);
181 }
178 for (const auto& desc : info.texture_descriptors) { 182 for (const auto& desc : info.texture_descriptors) {
179 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)}; 183 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)};
180 image_view_indices.push_back(handle.image); 184 image_view_indices.push_back(handle.image);
@@ -182,24 +186,37 @@ void GraphicsPipeline::Configure(bool is_indexed) {
182 Sampler* const sampler{texture_cache.GetGraphicsSampler(handle.sampler)}; 186 Sampler* const sampler{texture_cache.GetGraphicsSampler(handle.sampler)};
183 samplers.push_back(sampler->Handle()); 187 samplers.push_back(sampler->Handle());
184 } 188 }
189 }
190 const std::span indices_span(image_view_indices.data(), image_view_indices.size());
191 texture_cache.FillGraphicsImageViews(indices_span, image_view_ids);
192
193 ImageId* texture_buffer_index{image_view_ids.data()};
194 for (size_t stage = 0; stage < Maxwell::MaxShaderStage; ++stage) {
195 const Shader::Info& info{stage_infos[stage]};
196 buffer_cache.UnbindGraphicsTextureBuffers(stage);
197 size_t index{};
185 for (const auto& desc : info.texture_buffer_descriptors) { 198 for (const auto& desc : info.texture_buffer_descriptors) {
186 const TextureHandle handle{read_handle(desc.cbuf_index, desc.cbuf_offset)}; 199 ASSERT(desc.count == 1);
187 image_view_indices.push_back(handle.image); 200 ImageView& image_view = texture_cache.GetImageView(*texture_buffer_index);
201 buffer_cache.BindGraphicsTextureBuffer(stage, index, image_view.GpuAddr(),
202 image_view.BufferSize(), image_view.format);
203 ++index;
204 ++texture_buffer_index;
188 } 205 }
206 texture_buffer_index += info.texture_descriptors.size();
189 } 207 }
190 const std::span indices_span(image_view_indices.data(), image_view_indices.size());
191 buffer_cache.UpdateGraphicsBuffers(is_indexed); 208 buffer_cache.UpdateGraphicsBuffers(is_indexed);
192 texture_cache.FillGraphicsImageViews(indices_span, image_view_ids);
193 209
194 buffer_cache.BindHostGeometryBuffers(is_indexed); 210 buffer_cache.BindHostGeometryBuffers(is_indexed);
195 211
196 update_descriptor_queue.Acquire(); 212 update_descriptor_queue.Acquire();
197 213
198 size_t index{}; 214 const VkSampler* samplers_it{samplers.data()};
215 const ImageId* views_it{image_view_ids.data()};
199 for (size_t stage = 0; stage < Maxwell::MaxShaderStage; ++stage) { 216 for (size_t stage = 0; stage < Maxwell::MaxShaderStage; ++stage) {
200 buffer_cache.BindHostStageBuffers(stage); 217 buffer_cache.BindHostStageBuffers(stage);
201 PushImageDescriptors(stage_infos[stage], samplers.data(), image_view_ids.data(), 218 PushImageDescriptors(stage_infos[stage], samplers_it, views_it, texture_cache,
202 texture_cache, update_descriptor_queue, index); 219 update_descriptor_queue);
203 } 220 }
204 texture_cache.UpdateRenderTargets(false); 221 texture_cache.UpdateRenderTargets(false);
205 scheduler.RequestRenderpass(texture_cache.GetFramebuffer()); 222 scheduler.RequestRenderpass(texture_cache.GetFramebuffer());
diff --git a/src/video_core/renderer_vulkan/vk_texture_cache.cpp b/src/video_core/renderer_vulkan/vk_texture_cache.cpp
index 1bbc542a1..e42b091c5 100644
--- a/src/video_core/renderer_vulkan/vk_texture_cache.cpp
+++ b/src/video_core/renderer_vulkan/vk_texture_cache.cpp
@@ -15,10 +15,10 @@
15#include "video_core/renderer_vulkan/maxwell_to_vk.h" 15#include "video_core/renderer_vulkan/maxwell_to_vk.h"
16#include "video_core/renderer_vulkan/vk_compute_pass.h" 16#include "video_core/renderer_vulkan/vk_compute_pass.h"
17#include "video_core/renderer_vulkan/vk_rasterizer.h" 17#include "video_core/renderer_vulkan/vk_rasterizer.h"
18#include "video_core/renderer_vulkan/vk_render_pass_cache.h"
18#include "video_core/renderer_vulkan/vk_scheduler.h" 19#include "video_core/renderer_vulkan/vk_scheduler.h"
19#include "video_core/renderer_vulkan/vk_staging_buffer_pool.h" 20#include "video_core/renderer_vulkan/vk_staging_buffer_pool.h"
20#include "video_core/renderer_vulkan/vk_texture_cache.h" 21#include "video_core/renderer_vulkan/vk_texture_cache.h"
21#include "video_core/renderer_vulkan/vk_render_pass_cache.h"
22#include "video_core/vulkan_common/vulkan_device.h" 22#include "video_core/vulkan_common/vulkan_device.h"
23#include "video_core/vulkan_common/vulkan_memory_allocator.h" 23#include "video_core/vulkan_common/vulkan_memory_allocator.h"
24#include "video_core/vulkan_common/vulkan_wrapper.h" 24#include "video_core/vulkan_common/vulkan_wrapper.h"
@@ -162,25 +162,6 @@ constexpr VkBorderColor ConvertBorderColor(const std::array<float, 4>& color) {
162 return device.GetLogical().CreateImage(MakeImageCreateInfo(device, info)); 162 return device.GetLogical().CreateImage(MakeImageCreateInfo(device, info));
163} 163}
164 164
165[[nodiscard]] vk::Buffer MakeBuffer(const Device& device, const ImageInfo& info) {
166 if (info.type != ImageType::Buffer) {
167 return vk::Buffer{};
168 }
169 const size_t bytes_per_block = VideoCore::Surface::BytesPerBlock(info.format);
170 return device.GetLogical().CreateBuffer(VkBufferCreateInfo{
171 .sType = VK_STRUCTURE_TYPE_BUFFER_CREATE_INFO,
172 .pNext = nullptr,
173 .flags = 0,
174 .size = info.size.width * bytes_per_block,
175 .usage = VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT |
176 VK_BUFFER_USAGE_UNIFORM_TEXEL_BUFFER_BIT |
177 VK_BUFFER_USAGE_STORAGE_TEXEL_BUFFER_BIT,
178 .sharingMode = VK_SHARING_MODE_EXCLUSIVE,
179 .queueFamilyIndexCount = 0,
180 .pQueueFamilyIndices = nullptr,
181 });
182}
183
184[[nodiscard]] VkImageAspectFlags ImageAspectMask(PixelFormat format) { 165[[nodiscard]] VkImageAspectFlags ImageAspectMask(PixelFormat format) {
185 switch (VideoCore::Surface::GetFormatType(format)) { 166 switch (VideoCore::Surface::GetFormatType(format)) {
186 case VideoCore::Surface::SurfaceType::ColorTexture: 167 case VideoCore::Surface::SurfaceType::ColorTexture:
@@ -813,13 +794,9 @@ u64 TextureCacheRuntime::GetDeviceLocalMemory() const {
813Image::Image(TextureCacheRuntime& runtime, const ImageInfo& info_, GPUVAddr gpu_addr_, 794Image::Image(TextureCacheRuntime& runtime, const ImageInfo& info_, GPUVAddr gpu_addr_,
814 VAddr cpu_addr_) 795 VAddr cpu_addr_)
815 : VideoCommon::ImageBase(info_, gpu_addr_, cpu_addr_), scheduler{&runtime.scheduler}, 796 : VideoCommon::ImageBase(info_, gpu_addr_, cpu_addr_), scheduler{&runtime.scheduler},
816 image(MakeImage(runtime.device, info)), buffer(MakeBuffer(runtime.device, info)), 797 image(MakeImage(runtime.device, info)),
798 commit(runtime.memory_allocator.Commit(image, MemoryUsage::DeviceLocal)),
817 aspect_mask(ImageAspectMask(info.format)) { 799 aspect_mask(ImageAspectMask(info.format)) {
818 if (image) {
819 commit = runtime.memory_allocator.Commit(image, MemoryUsage::DeviceLocal);
820 } else {
821 commit = runtime.memory_allocator.Commit(buffer, MemoryUsage::DeviceLocal);
822 }
823 if (IsPixelFormatASTC(info.format) && !runtime.device.IsOptimalAstcSupported()) { 800 if (IsPixelFormatASTC(info.format) && !runtime.device.IsOptimalAstcSupported()) {
824 if (Settings::values.accelerate_astc.GetValue()) { 801 if (Settings::values.accelerate_astc.GetValue()) {
825 flags |= VideoCommon::ImageFlagBits::AcceleratedUpload; 802 flags |= VideoCommon::ImageFlagBits::AcceleratedUpload;
@@ -828,11 +805,7 @@ Image::Image(TextureCacheRuntime& runtime, const ImageInfo& info_, GPUVAddr gpu_
828 } 805 }
829 } 806 }
830 if (runtime.device.HasDebuggingToolAttached()) { 807 if (runtime.device.HasDebuggingToolAttached()) {
831 if (image) { 808 image.SetObjectNameEXT(VideoCommon::Name(*this).c_str());
832 image.SetObjectNameEXT(VideoCommon::Name(*this).c_str());
833 } else {
834 buffer.SetObjectNameEXT(VideoCommon::Name(*this).c_str());
835 }
836 } 809 }
837 static constexpr VkImageViewUsageCreateInfo storage_image_view_usage_create_info{ 810 static constexpr VkImageViewUsageCreateInfo storage_image_view_usage_create_info{
838 .sType = VK_STRUCTURE_TYPE_IMAGE_VIEW_USAGE_CREATE_INFO, 811 .sType = VK_STRUCTURE_TYPE_IMAGE_VIEW_USAGE_CREATE_INFO,
@@ -884,19 +857,6 @@ void Image::UploadMemory(const StagingBufferRef& map, std::span<const BufferImag
884 }); 857 });
885} 858}
886 859
887void Image::UploadMemory(const StagingBufferRef& map,
888 std::span<const VideoCommon::BufferCopy> copies) {
889 // TODO: Move this to another API
890 scheduler->RequestOutsideRenderPassOperationContext();
891 std::vector vk_copies = TransformBufferCopies(copies, map.offset);
892 const VkBuffer src_buffer = map.buffer;
893 const VkBuffer dst_buffer = *buffer;
894 scheduler->Record([src_buffer, dst_buffer, vk_copies](vk::CommandBuffer cmdbuf) {
895 // TODO: Barriers
896 cmdbuf.CopyBuffer(src_buffer, dst_buffer, vk_copies);
897 });
898}
899
900void Image::DownloadMemory(const StagingBufferRef& map, std::span<const BufferImageCopy> copies) { 860void Image::DownloadMemory(const StagingBufferRef& map, std::span<const BufferImageCopy> copies) {
901 std::vector vk_copies = TransformBufferImageCopies(copies, map.offset, aspect_mask); 861 std::vector vk_copies = TransformBufferImageCopies(copies, map.offset, aspect_mask);
902 scheduler->RequestOutsideRenderPassOperationContext(); 862 scheduler->RequestOutsideRenderPassOperationContext();
@@ -1032,19 +992,16 @@ ImageView::ImageView(TextureCacheRuntime& runtime, const VideoCommon::ImageViewI
1032 UNIMPLEMENTED(); 992 UNIMPLEMENTED();
1033 break; 993 break;
1034 case VideoCommon::ImageViewType::Buffer: 994 case VideoCommon::ImageViewType::Buffer:
1035 buffer_view = device->GetLogical().CreateBufferView(VkBufferViewCreateInfo{ 995 UNREACHABLE();
1036 .sType = VK_STRUCTURE_TYPE_BUFFER_VIEW_CREATE_INFO,
1037 .pNext = nullptr,
1038 .flags = 0,
1039 .buffer = image.Buffer(),
1040 .format = format_info.format,
1041 .offset = 0, // TODO: Redesign buffer cache to support this
1042 .range = image.guest_size_bytes,
1043 });
1044 break; 996 break;
1045 } 997 }
1046} 998}
1047 999
1000ImageView::ImageView(TextureCacheRuntime&, const VideoCommon::ImageInfo& info,
1001 const VideoCommon::ImageViewInfo& view_info, GPUVAddr gpu_addr_)
1002 : VideoCommon::ImageViewBase{info, view_info}, gpu_addr{gpu_addr_},
1003 buffer_size{VideoCommon::CalculateGuestSizeInBytes(info)} {}
1004
1048ImageView::ImageView(TextureCacheRuntime&, const VideoCommon::NullImageParams& params) 1005ImageView::ImageView(TextureCacheRuntime&, const VideoCommon::NullImageParams& params)
1049 : VideoCommon::ImageViewBase{params} {} 1006 : VideoCommon::ImageViewBase{params} {}
1050 1007
diff --git a/src/video_core/renderer_vulkan/vk_texture_cache.h b/src/video_core/renderer_vulkan/vk_texture_cache.h
index 189ee5a68..498e76a1c 100644
--- a/src/video_core/renderer_vulkan/vk_texture_cache.h
+++ b/src/video_core/renderer_vulkan/vk_texture_cache.h
@@ -41,9 +41,9 @@ struct TextureCacheRuntime {
41 41
42 void Finish(); 42 void Finish();
43 43
44 [[nodiscard]] StagingBufferRef UploadStagingBuffer(size_t size); 44 StagingBufferRef UploadStagingBuffer(size_t size);
45 45
46 [[nodiscard]] StagingBufferRef DownloadStagingBuffer(size_t size); 46 StagingBufferRef DownloadStagingBuffer(size_t size);
47 47
48 void BlitImage(Framebuffer* dst_framebuffer, ImageView& dst, ImageView& src, 48 void BlitImage(Framebuffer* dst_framebuffer, ImageView& dst, ImageView& src,
49 const Region2D& dst_region, const Region2D& src_region, 49 const Region2D& dst_region, const Region2D& src_region,
@@ -54,7 +54,7 @@ struct TextureCacheRuntime {
54 54
55 void ConvertImage(Framebuffer* dst, ImageView& dst_view, ImageView& src_view); 55 void ConvertImage(Framebuffer* dst, ImageView& dst_view, ImageView& src_view);
56 56
57 [[nodiscard]] bool CanAccelerateImageUpload(Image&) const noexcept { 57 bool CanAccelerateImageUpload(Image&) const noexcept {
58 return false; 58 return false;
59 } 59 }
60 60
@@ -92,8 +92,6 @@ public:
92 void UploadMemory(const StagingBufferRef& map, 92 void UploadMemory(const StagingBufferRef& map,
93 std::span<const VideoCommon::BufferImageCopy> copies); 93 std::span<const VideoCommon::BufferImageCopy> copies);
94 94
95 void UploadMemory(const StagingBufferRef& map, std::span<const VideoCommon::BufferCopy> copies);
96
97 void DownloadMemory(const StagingBufferRef& map, 95 void DownloadMemory(const StagingBufferRef& map,
98 std::span<const VideoCommon::BufferImageCopy> copies); 96 std::span<const VideoCommon::BufferImageCopy> copies);
99 97
@@ -101,10 +99,6 @@ public:
101 return *image; 99 return *image;
102 } 100 }
103 101
104 [[nodiscard]] VkBuffer Buffer() const noexcept {
105 return *buffer;
106 }
107
108 [[nodiscard]] VkImageAspectFlags AspectMask() const noexcept { 102 [[nodiscard]] VkImageAspectFlags AspectMask() const noexcept {
109 return aspect_mask; 103 return aspect_mask;
110 } 104 }
@@ -121,7 +115,6 @@ public:
121private: 115private:
122 VKScheduler* scheduler; 116 VKScheduler* scheduler;
123 vk::Image image; 117 vk::Image image;
124 vk::Buffer buffer;
125 MemoryCommit commit; 118 MemoryCommit commit;
126 vk::ImageView image_view; 119 vk::ImageView image_view;
127 std::vector<vk::ImageView> storage_image_views; 120 std::vector<vk::ImageView> storage_image_views;
@@ -132,6 +125,8 @@ private:
132class ImageView : public VideoCommon::ImageViewBase { 125class ImageView : public VideoCommon::ImageViewBase {
133public: 126public:
134 explicit ImageView(TextureCacheRuntime&, const VideoCommon::ImageViewInfo&, ImageId, Image&); 127 explicit ImageView(TextureCacheRuntime&, const VideoCommon::ImageViewInfo&, ImageId, Image&);
128 explicit ImageView(TextureCacheRuntime&, const VideoCommon::ImageInfo&,
129 const VideoCommon::ImageViewInfo&, GPUVAddr);
135 explicit ImageView(TextureCacheRuntime&, const VideoCommon::NullImageParams&); 130 explicit ImageView(TextureCacheRuntime&, const VideoCommon::NullImageParams&);
136 131
137 [[nodiscard]] VkImageView DepthView(); 132 [[nodiscard]] VkImageView DepthView();
@@ -142,10 +137,6 @@ public:
142 return *image_views[static_cast<size_t>(query_type)]; 137 return *image_views[static_cast<size_t>(query_type)];
143 } 138 }
144 139
145 [[nodiscard]] VkBufferView BufferView() const noexcept {
146 return *buffer_view;
147 }
148
149 [[nodiscard]] VkImage ImageHandle() const noexcept { 140 [[nodiscard]] VkImage ImageHandle() const noexcept {
150 return image_handle; 141 return image_handle;
151 } 142 }
@@ -162,6 +153,14 @@ public:
162 return samples; 153 return samples;
163 } 154 }
164 155
156 [[nodiscard]] GPUVAddr GpuAddr() const noexcept {
157 return gpu_addr;
158 }
159
160 [[nodiscard]] u32 BufferSize() const noexcept {
161 return buffer_size;
162 }
163
165private: 164private:
166 [[nodiscard]] vk::ImageView MakeDepthStencilView(VkImageAspectFlags aspect_mask); 165 [[nodiscard]] vk::ImageView MakeDepthStencilView(VkImageAspectFlags aspect_mask);
167 166
@@ -169,11 +168,12 @@ private:
169 std::array<vk::ImageView, VideoCommon::NUM_IMAGE_VIEW_TYPES> image_views; 168 std::array<vk::ImageView, VideoCommon::NUM_IMAGE_VIEW_TYPES> image_views;
170 vk::ImageView depth_view; 169 vk::ImageView depth_view;
171 vk::ImageView stencil_view; 170 vk::ImageView stencil_view;
172 vk::BufferView buffer_view;
173 VkImage image_handle = VK_NULL_HANDLE; 171 VkImage image_handle = VK_NULL_HANDLE;
174 VkImageView render_target = VK_NULL_HANDLE; 172 VkImageView render_target = VK_NULL_HANDLE;
175 PixelFormat image_format = PixelFormat::Invalid; 173 PixelFormat image_format = PixelFormat::Invalid;
176 VkSampleCountFlagBits samples = VK_SAMPLE_COUNT_1_BIT; 174 VkSampleCountFlagBits samples = VK_SAMPLE_COUNT_1_BIT;
175 GPUVAddr gpu_addr = 0;
176 u32 buffer_size = 0;
177}; 177};
178 178
179class ImageAlloc : public VideoCommon::ImageAllocBase {}; 179class ImageAlloc : public VideoCommon::ImageAllocBase {};