Use vkCmdBlit for texture copies when formats dont match

This commit is contained in:
Billy Laws 2022-11-06 20:31:23 +00:00
parent c8fc8f84ec
commit ac0e225114
3 changed files with 28 additions and 8 deletions

View File

@ -139,7 +139,7 @@ namespace skyline::gpu {
auto &presentSemaphore{presentSemaphores[nextImage.second]}; auto &presentSemaphore{presentSemaphores[nextImage.second]};
texture->SynchronizeHost(); texture->SynchronizeHost();
nextImageTexture->CopyFrom(texture, *acquireSemaphore, *presentSemaphore, vk::ImageSubresourceRange{ nextImageTexture->CopyFrom(texture, *acquireSemaphore, *presentSemaphore, swapchainFormat, vk::ImageSubresourceRange{
.aspectMask = vk::ImageAspectFlagBits::eColor, .aspectMask = vk::ImageAspectFlagBits::eColor,
.levelCount = 1, .levelCount = 1,
.layerCount = 1, .layerCount = 1,

View File

@ -865,7 +865,7 @@ namespace skyline::gpu {
return std::make_shared<TextureView>(shared_from_this(), type, range, pFormat, mapping); return std::make_shared<TextureView>(shared_from_this(), type, range, pFormat, mapping);
} }
void Texture::CopyFrom(std::shared_ptr<Texture> source, vk::Semaphore waitSemaphore, vk::Semaphore signalSemaphore, const vk::ImageSubresourceRange &subresource) { void Texture::CopyFrom(std::shared_ptr<Texture> source, vk::Semaphore waitSemaphore, vk::Semaphore signalSemaphore, texture::Format srcFormat, const vk::ImageSubresourceRange &subresource) {
if (cycle) if (cycle)
cycle->WaitSubmit(); cycle->WaitSubmit();
if (source->cycle) if (source->cycle)
@ -928,12 +928,32 @@ namespace skyline::gpu {
.baseArrayLayer = subresource.baseArrayLayer, .baseArrayLayer = subresource.baseArrayLayer,
.layerCount = subresource.layerCount == VK_REMAINING_ARRAY_LAYERS ? layerCount - subresource.baseArrayLayer : subresource.layerCount, .layerCount = subresource.layerCount == VK_REMAINING_ARRAY_LAYERS ? layerCount - subresource.baseArrayLayer : subresource.layerCount,
}; };
for (; subresourceLayers.mipLevel < (subresource.levelCount == VK_REMAINING_MIP_LEVELS ? levelCount - subresource.baseMipLevel : subresource.levelCount); subresourceLayers.mipLevel++) for (; subresourceLayers.mipLevel < (subresource.levelCount == VK_REMAINING_MIP_LEVELS ? levelCount - subresource.baseMipLevel : subresource.levelCount); subresourceLayers.mipLevel++) {
if (srcFormat != format) {
commandBuffer.blitImage(sourceBacking, vk::ImageLayout::eTransferSrcOptimal, destinationBacking, vk::ImageLayout::eTransferDstOptimal, vk::ImageBlit{
.srcSubresource = subresourceLayers,
.srcOffsets = std::array<vk::Offset3D, 2>{
vk::Offset3D{0, 0, 0},
vk::Offset3D{static_cast<i32>(dimensions.width),
static_cast<i32>(dimensions.height),
static_cast<i32>(subresourceLayers.layerCount)}
},
.dstSubresource = subresourceLayers,
.dstOffsets = std::array<vk::Offset3D, 2>{
vk::Offset3D{0, 0, 0},
vk::Offset3D{static_cast<i32>(dimensions.width),
static_cast<i32>(dimensions.height),
static_cast<i32>(subresourceLayers.layerCount)}
}
}, vk::Filter::eLinear);
} else {
commandBuffer.copyImage(sourceBacking, vk::ImageLayout::eTransferSrcOptimal, destinationBacking, vk::ImageLayout::eTransferDstOptimal, vk::ImageCopy{ commandBuffer.copyImage(sourceBacking, vk::ImageLayout::eTransferSrcOptimal, destinationBacking, vk::ImageLayout::eTransferDstOptimal, vk::ImageCopy{
.srcSubresource = subresourceLayers, .srcSubresource = subresourceLayers,
.dstSubresource = subresourceLayers, .dstSubresource = subresourceLayers,
.extent = dimensions, .extent = dimensions,
}); });
}
}
if (layout != vk::ImageLayout::eTransferDstOptimal) if (layout != vk::ImageLayout::eTransferDstOptimal)
commandBuffer.pipelineBarrier(vk::PipelineStageFlagBits::eAllCommands, vk::PipelineStageFlagBits::eAllCommands, {}, {}, {}, vk::ImageMemoryBarrier{ commandBuffer.pipelineBarrier(vk::PipelineStageFlagBits::eAllCommands, vk::PipelineStageFlagBits::eAllCommands, {}, {}, {}, vk::ImageMemoryBarrier{

View File

@ -581,7 +581,7 @@ namespace skyline::gpu {
/** /**
* @brief Copies the contents of the supplied source texture into the current texture * @brief Copies the contents of the supplied source texture into the current texture
*/ */
void CopyFrom(std::shared_ptr<Texture> source, vk::Semaphore waitSemaphore, vk::Semaphore signalSemaphore, const vk::ImageSubresourceRange &subresource = vk::ImageSubresourceRange{ void CopyFrom(std::shared_ptr<Texture> source, vk::Semaphore waitSemaphore, vk::Semaphore signalSemaphore, texture::Format srcFormat, const vk::ImageSubresourceRange &subresource = vk::ImageSubresourceRange{
.aspectMask = vk::ImageAspectFlagBits::eColor, .aspectMask = vk::ImageAspectFlagBits::eColor,
.levelCount = VK_REMAINING_MIP_LEVELS, .levelCount = VK_REMAINING_MIP_LEVELS,
.layerCount = VK_REMAINING_ARRAY_LAYERS, .layerCount = VK_REMAINING_ARRAY_LAYERS,