[Mesa-dev] [PATCH 2/8] radv: handle multiple subpass dependencies with same dst

Bas Nieuwenhuizen bas at basnieuwenhuizen.nl
Tue Jun 26 23:10:38 UTC 2018


Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>

On Tue, Jun 26, 2018 at 11:49 PM, Fredrik Höglund <fredrik at kde.org> wrote:
> A subpass can have dependencies from multiple sources, and more
> than one subpass can have a dependency to external.
>
> Cc: <mesa-stable at lists.freedesktop.org>
> Signed-off-by: Fredrik Höglund <fredrik at kde.org>
> ---
>  src/amd/vulkan/radv_pass.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/src/amd/vulkan/radv_pass.c b/src/amd/vulkan/radv_pass.c
> index 7a0dca09496..c41d4c2ba92 100644
> --- a/src/amd/vulkan/radv_pass.c
> +++ b/src/amd/vulkan/radv_pass.c
> @@ -175,14 +175,14 @@ VkResult radv_CreateRenderPass(
>                 uint32_t dst = pCreateInfo->pDependencies[i].dstSubpass;
>                 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;
> -                       pass->end_barrier.src_access_mask = pCreateInfo->pDependencies[i].srcAccessMask;
> -                       pass->end_barrier.dst_access_mask = pCreateInfo->pDependencies[i].dstAccessMask;
> +                               pass->end_barrier.src_stage_mask |= pCreateInfo->pDependencies[i].srcStageMask;
> +                       pass->end_barrier.src_access_mask |= pCreateInfo->pDependencies[i].srcAccessMask;
> +                       pass->end_barrier.dst_access_mask |= pCreateInfo->pDependencies[i].dstAccessMask;
>                 } else {
>                         if (pCreateInfo->pDependencies[i].dstStageMask != VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT)
> -                               pass->subpasses[dst].start_barrier.src_stage_mask = pCreateInfo->pDependencies[i].srcStageMask;
> -                       pass->subpasses[dst].start_barrier.src_access_mask = pCreateInfo->pDependencies[i].srcAccessMask;
> -                       pass->subpasses[dst].start_barrier.dst_access_mask = pCreateInfo->pDependencies[i].dstAccessMask;
> +                               pass->subpasses[dst].start_barrier.src_stage_mask |= pCreateInfo->pDependencies[i].srcStageMask;
> +                       pass->subpasses[dst].start_barrier.src_access_mask |= pCreateInfo->pDependencies[i].srcAccessMask;
> +                       pass->subpasses[dst].start_barrier.dst_access_mask |= pCreateInfo->pDependencies[i].dstAccessMask;
>                 }
>         }
>
> --
> 2.17.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list