The-Homebrew-Cloud/skyline
2
0
Fork 0
mirror of https://github.com/skyline-emu/skyline.git synced 2024-06-14 20:48:48 +02:00
Code Issues Packages Projects Releases Wiki Activity

Use pipeline barriers, as opposed to an ext dependency for RP barrier

Browse Source 
Allows for waiting on compute shaders, which are not a graphics stage.
This commit is contained in:
Billy Laws 2023-02-13 18:01:02 +00:00
parent ee68facc5d
commit 6ee8a919e5
2 changed files with 24 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
app/src/main/cpp/skyline/gpu/interconnect/command_nodes.cpp
View File

@ -6,14 +6,7 @@
#include <vulkan/vulkan_enums.hpp>
namespace skyline::gpu::interconnect::node {
RenderPassNode::RenderPassNode(vk::Rect2D renderArea)
: externalDependency{vk::SubpassDependency{
.srcSubpass = VK_SUBPASS_EXTERNAL,
.dstSubpass = 0,
.srcAccessMask = vk::AccessFlagBits::eMemoryWrite,
.dstAccessMask = vk::AccessFlagBits::eMemoryRead | vk::AccessFlagBits::eMemoryWrite,
}},
renderArea{renderArea} {}
RenderPassNode::RenderPassNode(vk::Rect2D renderArea) : renderArea{renderArea} {}
u32 RenderPassNode::AddAttachment(TextureView *view, GPU &gpu) {
auto vkView{view->GetView()};
@ -41,15 +34,18 @@ namespace skyline::gpu::interconnect::node {
});
if (auto usage{view->texture->GetLastRenderPassUsage()}; usage != texture::RenderPassUsage::None) {
vk::PipelineStageFlags attachmentDstStageMask{};
if (view->format->vkAspect & vk::ImageAspectFlagBits::eColor)
externalDependency.dstStageMask |= vk::PipelineStageFlagBits::eColorAttachmentOutput;
attachmentDstStageMask |= vk::PipelineStageFlagBits::eColorAttachmentOutput;
else if (view->format->vkAspect & (vk::ImageAspectFlagBits::eDepth | vk::ImageAspectFlagBits::eStencil))
externalDependency.dstStageMask |= vk::PipelineStageFlagBits::eEarlyFragmentTests | vk::PipelineStageFlagBits::eLateFragmentTests;
attachmentDstStageMask |= vk::PipelineStageFlagBits::eEarlyFragmentTests | vk::PipelineStageFlagBits::eLateFragmentTests;
dependencyDstStageMask |= attachmentDstStageMask;
if (usage == texture::RenderPassUsage::RenderTarget)
externalDependency.srcStageMask |= externalDependency.dstStageMask;
dependencySrcStageMask |= attachmentDstStageMask;
else if (usage == texture::RenderPassUsage::Sampled)
externalDependency.srcStageMask |= vk::PipelineStageFlagBits::eAllGraphics;
dependencySrcStageMask |= view->texture->GetReadStageMask();
}
return static_cast<u32>(attachments.size() - 1);
@ -173,8 +169,8 @@ namespace skyline::gpu::interconnect::node {
}
void RenderPassNode::UpdateDependency(vk::PipelineStageFlags srcStageMask, vk::PipelineStageFlags dstStageMask) {
externalDependency.srcStageMask |= srcStageMask;
externalDependency.dstStageMask |= dstStageMask;
dependencySrcStageMask |= srcStageMask;
dependencyDstStageMask |= dstStageMask;
}
bool RenderPassNode::ClearColorAttachment(u32 colorAttachment, const vk::ClearColorValue &value, GPU& gpu) {
@ -240,8 +236,12 @@ namespace skyline::gpu::interconnect::node {
preserveAttachmentIt++;
}
if (externalDependency.srcStageMask && externalDependency.dstStageMask)
subpassDependencies.push_back(externalDependency);
if (dependencyDstStageMask && dependencySrcStageMask) {
commandBuffer.pipelineBarrier(dependencySrcStageMask, dependencyDstStageMask, {}, {vk::MemoryBarrier{
.srcAccessMask = vk::AccessFlagBits::eMemoryWrite,
.dstAccessMask = vk::AccessFlagBits::eMemoryWrite | vk::AccessFlagBits::eMemoryRead,
}}, {}, {});
}
auto renderPass{gpu.renderPassCache.GetRenderPass(vk::RenderPassCreateInfo{
.attachmentCount = static_cast<u32>(attachmentDescriptions.size()),

10
app/src/main/cpp/skyline/gpu/interconnect/command_nodes.h
View File

@ -48,7 +48,8 @@ namespace skyline::gpu::interconnect::node {
public:
std::vector<vk::SubpassDescription> subpassDescriptions;
std::vector<vk::SubpassDependency> subpassDependencies;
vk::SubpassDependency externalDependency;
vk::PipelineStageFlags dependencySrcStageMask;
vk::PipelineStageFlags dependencyDstStageMask;
vk::Rect2D renderArea;
std::vector<vk::ClearValue> clearValues;
@ -64,7 +65,12 @@ namespace skyline::gpu::interconnect::node {
/**
* @brief Creates a subpass with the attachments bound in the specified order
*/
void AddSubpass(span<TextureView *> inputAttachments, span<TextureView *> colorAttachments, TextureView *depthStencilAttachment, GPU &gpu, vk::PipelineStageFlags srcStageMask, vk::PipelineStageFlags dstStageMask);
void AddSubpass(span<TextureView *> inputAttachments, span<TextureView *> colorAttachments, TextureView *depthStencilAttachment, GPU &gpu);
/**
* @brief Updates the dependency barrier for the renderpass
*/
void UpdateDependency(vk::PipelineStageFlags srcStageMask, vk::PipelineStageFlags dstStageMask);
/**
* @brief Clears a color attachment in the current subpass with VK_ATTACHMENT_LOAD_OP_CLEAR