From 60b6e98529b2dba0de600e5ec5cdefa9c597aac7 Mon Sep 17 00:00:00 2001 From: Robin Kertels Date: Wed, 21 Jun 2023 16:00:51 +0200 Subject: [PATCH] [d3d9] Transition DS too if there's a feedback loop --- src/d3d9/d3d9_device.cpp | 32 ++++++++++++++++++++++++++++---- 1 file changed, 28 insertions(+), 4 deletions(-) diff --git a/src/d3d9/d3d9_device.cpp b/src/d3d9/d3d9_device.cpp index 0c844b0f..b6bd6141 100644 --- a/src/d3d9/d3d9_device.cpp +++ b/src/d3d9/d3d9_device.cpp @@ -5592,10 +5592,25 @@ namespace dxvk { void D3D9DeviceEx::MarkRenderHazards() { - EmitCs([](DxvkContext* ctx) { + EmitCs([ + cActiveHazardsRT = m_activeHazardsRT, + cActiveHazardsDS = m_activeHazardsDS + ](DxvkContext* ctx) { + VkPipelineStageFlags srcStages = 0; + VkAccessFlags srcAccess = 0; + + if (cActiveHazardsRT != 0) { + srcStages |= VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT; + srcAccess |= VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT; + } + if (cActiveHazardsDS != 0) { + srcStages |= VK_PIPELINE_STAGE_EARLY_FRAGMENT_TESTS_BIT | VK_PIPELINE_STAGE_LATE_FRAGMENT_TESTS_BIT; + srcAccess |= VK_ACCESS_DEPTH_STENCIL_ATTACHMENT_WRITE_BIT; + } + ctx->emitGraphicsBarrier( - VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, - VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT, + srcStages, + srcAccess, VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT, VK_ACCESS_SHADER_READ_BIT); }); @@ -5608,6 +5623,15 @@ namespace dxvk { m_flags.set(D3D9DeviceFlag::DirtyFramebuffer); } } + + for (uint32_t samplerIdx : bit::BitMask(m_activeHazardsDS)) { + // Guaranteed to not be nullptr... + auto tex = GetCommonTexture(m_state.textures[samplerIdx]); + if (unlikely(!tex->MarkHazardous())) { + TransitionImage(tex, m_hazardLayout); + m_flags.set(D3D9DeviceFlag::DirtyFramebuffer); + } + } } @@ -6400,7 +6424,7 @@ namespace dxvk { void D3D9DeviceEx::PrepareDraw(D3DPRIMITIVETYPE PrimitiveType) { - if (unlikely(m_activeHazardsRT != 0)) + if (unlikely(m_activeHazardsRT != 0 || m_activeHazardsDS != 0)) MarkRenderHazards(); if (unlikely((!m_lastHazardsDS) != (!m_activeHazardsDS))