Reviewed-by: Bas Nieuwenhuizen <[email protected]> for patch 3-4. Not sure they should go to stable though, since they are optimizations?
On Tue, Jun 26, 2018 at 11:49 PM, Fredrik Höglund <[email protected]> wrote: > This is to avoid repeating dependencies when more than one subpass > has a dependency from external. > > Cc: <[email protected]> > Signed-off-by: Fredrik Höglund <[email protected]> > --- > src/amd/vulkan/radv_pass.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/amd/vulkan/radv_pass.c b/src/amd/vulkan/radv_pass.c > index 2827f5f1a8d..7e6fd84af55 100644 > --- a/src/amd/vulkan/radv_pass.c > +++ b/src/amd/vulkan/radv_pass.c > @@ -179,6 +179,10 @@ VkResult radv_CreateRenderPass( > if (src == dst) > continue; > > + if (src == VK_SUBPASS_EXTERNAL) { > + /* Add all dependencies from external to the first > subpass */ > + dst = 0; > + } > if (dst == VK_SUBPASS_EXTERNAL) { > if (pCreateInfo->pDependencies[i].dstStageMask != > VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT) > pass->end_barrier.src_stage_mask |= > pCreateInfo->pDependencies[i].srcStageMask; > -- > 2.17.0 > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
