diff options
| author | 2023-04-14 18:07:38 +0200 | |
|---|---|---|
| committer | 2023-04-29 00:18:21 +0200 | |
| commit | e3a2ca96bd2350471ebb6c2907c67b10254a4f7e (patch) | |
| tree | 5238364cdea97449adb0766df0d6263123ed06da /src/video_core/renderer_vulkan | |
| parent | TextureCache: refactor DMA downloads to allow multiple buffers. (diff) | |
| download | yuzu-e3a2ca96bd2350471ebb6c2907c67b10254a4f7e.tar.gz yuzu-e3a2ca96bd2350471ebb6c2907c67b10254a4f7e.tar.xz yuzu-e3a2ca96bd2350471ebb6c2907c67b10254a4f7e.zip | |
Accelerate DMA: Use texture cache async downloads to perform the copies
to host.
WIP
Diffstat (limited to 'src/video_core/renderer_vulkan')
| -rw-r--r-- | src/video_core/renderer_vulkan/vk_rasterizer.cpp | 6 | ||||
| -rw-r--r-- | src/video_core/renderer_vulkan/vk_texture_cache.cpp | 21 | ||||
| -rw-r--r-- | src/video_core/renderer_vulkan/vk_texture_cache.h | 2 |
3 files changed, 17 insertions, 12 deletions
diff --git a/src/video_core/renderer_vulkan/vk_rasterizer.cpp b/src/video_core/renderer_vulkan/vk_rasterizer.cpp index 8fc783cc0..2559a3aa7 100644 --- a/src/video_core/renderer_vulkan/vk_rasterizer.cpp +++ b/src/video_core/renderer_vulkan/vk_rasterizer.cpp | |||
| @@ -781,8 +781,7 @@ bool AccelerateDMA::DmaBufferImageCopy(const Tegra::DMA::ImageCopy& copy_info, | |||
| 781 | } | 781 | } |
| 782 | const u32 buffer_size = static_cast<u32>(buffer_operand.pitch * buffer_operand.height); | 782 | const u32 buffer_size = static_cast<u32>(buffer_operand.pitch * buffer_operand.height); |
| 783 | static constexpr auto sync_info = VideoCommon::ObtainBufferSynchronize::FullSynchronize; | 783 | static constexpr auto sync_info = VideoCommon::ObtainBufferSynchronize::FullSynchronize; |
| 784 | const auto post_op = IS_IMAGE_UPLOAD ? VideoCommon::ObtainBufferOperation::DoNothing | 784 | const auto post_op = VideoCommon::ObtainBufferOperation::DoNothing; |
| 785 | : VideoCommon::ObtainBufferOperation::MarkAsWritten; | ||
| 786 | const auto [buffer, offset] = | 785 | const auto [buffer, offset] = |
| 787 | buffer_cache.ObtainBuffer(buffer_operand.address, buffer_size, sync_info, post_op); | 786 | buffer_cache.ObtainBuffer(buffer_operand.address, buffer_size, sync_info, post_op); |
| 788 | 787 | ||
| @@ -793,7 +792,8 @@ bool AccelerateDMA::DmaBufferImageCopy(const Tegra::DMA::ImageCopy& copy_info, | |||
| 793 | if constexpr (IS_IMAGE_UPLOAD) { | 792 | if constexpr (IS_IMAGE_UPLOAD) { |
| 794 | image->UploadMemory(buffer->Handle(), offset, copy_span); | 793 | image->UploadMemory(buffer->Handle(), offset, copy_span); |
| 795 | } else { | 794 | } else { |
| 796 | texture_cache.DownloadImageIntoBuffer(image, buffer->Handle(), offset, copy_span); | 795 | texture_cache.DownloadImageIntoBuffer(image, buffer->Handle(), offset, copy_span, |
| 796 | buffer_operand.address, buffer_size); | ||
| 797 | } | 797 | } |
| 798 | return true; | 798 | return true; |
| 799 | } | 799 | } |
diff --git a/src/video_core/renderer_vulkan/vk_texture_cache.cpp b/src/video_core/renderer_vulkan/vk_texture_cache.cpp index e4d077e63..da3841bb3 100644 --- a/src/video_core/renderer_vulkan/vk_texture_cache.cpp +++ b/src/video_core/renderer_vulkan/vk_texture_cache.cpp | |||
| @@ -1342,17 +1342,19 @@ void Image::UploadMemory(const StagingBufferRef& map, std::span<const BufferImag | |||
| 1342 | UploadMemory(map.buffer, map.offset, copies); | 1342 | UploadMemory(map.buffer, map.offset, copies); |
| 1343 | } | 1343 | } |
| 1344 | 1344 | ||
| 1345 | void Image::DownloadMemory(std::span<VkBuffer> buffers_span, VkDeviceSize offset, | 1345 | void Image::DownloadMemory(std::span<VkBuffer> buffers_span, std::span<VkDeviceSize> offsets_span, |
| 1346 | std::span<const VideoCommon::BufferImageCopy> copies) { | 1346 | std::span<const VideoCommon::BufferImageCopy> copies) { |
| 1347 | const bool is_rescaled = True(flags & ImageFlagBits::Rescaled); | 1347 | const bool is_rescaled = True(flags & ImageFlagBits::Rescaled); |
| 1348 | if (is_rescaled) { | 1348 | if (is_rescaled) { |
| 1349 | ScaleDown(); | 1349 | ScaleDown(); |
| 1350 | } | 1350 | } |
| 1351 | boost::container::small_vector<VkBuffer, 1> buffers_vector{}; | 1351 | boost::container::small_vector<VkBuffer, 1> buffers_vector{}; |
| 1352 | for (auto& buffer : buffers_span) { | 1352 | boost::container::small_vector<std::vector<VkBufferImageCopy>, 1> vk_copies; |
| 1353 | buffers_vector.push_back(buffer); | 1353 | for (size_t index = 0; index < buffers_span.size(); index++) { |
| 1354 | buffers_vector.emplace_back(buffers_span[index]); | ||
| 1355 | vk_copies.emplace_back( | ||
| 1356 | TransformBufferImageCopies(copies, offsets_span[index], aspect_mask)); | ||
| 1354 | } | 1357 | } |
| 1355 | std::vector vk_copies = TransformBufferImageCopies(copies, offset, aspect_mask); | ||
| 1356 | scheduler->RequestOutsideRenderPassOperationContext(); | 1358 | scheduler->RequestOutsideRenderPassOperationContext(); |
| 1357 | scheduler->Record([buffers = std::move(buffers_vector), image = *original_image, | 1359 | scheduler->Record([buffers = std::move(buffers_vector), image = *original_image, |
| 1358 | aspect_mask = aspect_mask, vk_copies](vk::CommandBuffer cmdbuf) { | 1360 | aspect_mask = aspect_mask, vk_copies](vk::CommandBuffer cmdbuf) { |
| @@ -1377,9 +1379,9 @@ void Image::DownloadMemory(std::span<VkBuffer> buffers_span, VkDeviceSize offset | |||
| 1377 | cmdbuf.PipelineBarrier(VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, | 1379 | cmdbuf.PipelineBarrier(VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, |
| 1378 | 0, read_barrier); | 1380 | 0, read_barrier); |
| 1379 | 1381 | ||
| 1380 | for (auto buffer : buffers) { | 1382 | for (size_t index = 0; index < buffers.size(); index++) { |
| 1381 | cmdbuf.CopyImageToBuffer(image, VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL, buffer, | 1383 | cmdbuf.CopyImageToBuffer(image, VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL, buffers[index], |
| 1382 | vk_copies); | 1384 | vk_copies[index]); |
| 1383 | } | 1385 | } |
| 1384 | 1386 | ||
| 1385 | const VkMemoryBarrier memory_write_barrier{ | 1387 | const VkMemoryBarrier memory_write_barrier{ |
| @@ -1418,7 +1420,10 @@ void Image::DownloadMemory(const StagingBufferRef& map, std::span<const BufferIm | |||
| 1418 | std::array buffers{ | 1420 | std::array buffers{ |
| 1419 | map.buffer, | 1421 | map.buffer, |
| 1420 | }; | 1422 | }; |
| 1421 | DownloadMemory(buffers, map.offset, copies); | 1423 | std::array offsets{ |
| 1424 | map.offset, | ||
| 1425 | }; | ||
| 1426 | DownloadMemory(buffers, offsets, copies); | ||
| 1422 | } | 1427 | } |
| 1423 | 1428 | ||
| 1424 | bool Image::IsRescaled() const noexcept { | 1429 | bool Image::IsRescaled() const noexcept { |
diff --git a/src/video_core/renderer_vulkan/vk_texture_cache.h b/src/video_core/renderer_vulkan/vk_texture_cache.h index 422476188..bdaf43ba4 100644 --- a/src/video_core/renderer_vulkan/vk_texture_cache.h +++ b/src/video_core/renderer_vulkan/vk_texture_cache.h | |||
| @@ -138,7 +138,7 @@ public: | |||
| 138 | void UploadMemory(const StagingBufferRef& map, | 138 | void UploadMemory(const StagingBufferRef& map, |
| 139 | std::span<const VideoCommon::BufferImageCopy> copies); | 139 | std::span<const VideoCommon::BufferImageCopy> copies); |
| 140 | 140 | ||
| 141 | void DownloadMemory(std::span<VkBuffer> buffers, VkDeviceSize offset, | 141 | void DownloadMemory(std::span<VkBuffer> buffers, std::span<VkDeviceSize> offsets, |
| 142 | std::span<const VideoCommon::BufferImageCopy> copies); | 142 | std::span<const VideoCommon::BufferImageCopy> copies); |
| 143 | 143 | ||
| 144 | void DownloadMemory(const StagingBufferRef& map, | 144 | void DownloadMemory(const StagingBufferRef& map, |