radv: Add logic for multisample format descriptions.
[mesa.git] / src / amd / vulkan / radv_pass.c
index f8e5ea40954921ede53973bb4bfd478f6819b2d7..4d1e38a780e89e3092665e0e9b8c433a50bd206a 100644 (file)
 
 #include "vk_util.h"
 
+static void
+radv_render_pass_add_subpass_dep(struct radv_render_pass *pass,
+                                const VkSubpassDependency2KHR *dep)
+{
+       uint32_t src = dep->srcSubpass;
+       uint32_t dst = dep->dstSubpass;
+
+       /* Ignore subpass self-dependencies as they allow the app to call
+        * vkCmdPipelineBarrier() inside the render pass and the driver should
+        * only do the barrier when called, not when starting the render pass.
+        */
+       if (src == dst)
+               return;
+
+       /* Accumulate all ingoing external dependencies to the first subpass. */
+       if (src == VK_SUBPASS_EXTERNAL)
+               dst = 0;
+
+       if (dst == VK_SUBPASS_EXTERNAL) {
+               if (dep->dstStageMask != VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT)
+                       pass->end_barrier.src_stage_mask |= dep->srcStageMask;
+               pass->end_barrier.src_access_mask |= dep->srcAccessMask;
+               pass->end_barrier.dst_access_mask |= dep->dstAccessMask;
+       } else {
+               if (dep->dstStageMask != VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT)
+                       pass->subpasses[dst].start_barrier.src_stage_mask |= dep->srcStageMask;
+               pass->subpasses[dst].start_barrier.src_access_mask |= dep->srcAccessMask;
+               pass->subpasses[dst].start_barrier.dst_access_mask |= dep->dstAccessMask;
+       }
+}
+
+static void
+radv_render_pass_compile(struct radv_render_pass *pass)
+{
+       for (uint32_t i = 0; i < pass->subpass_count; i++) {
+               struct radv_subpass *subpass = &pass->subpasses[i];
+               uint32_t color_sample_count = 1, depth_sample_count = 1;
+
+               /* We don't allow depth_stencil_attachment to be non-NULL and
+                * be VK_ATTACHMENT_UNUSED.  This way something can just check
+                * for NULL and be guaranteed that they have a valid
+                * attachment.
+                */
+               if (subpass->depth_stencil_attachment &&
+                   subpass->depth_stencil_attachment->attachment == VK_ATTACHMENT_UNUSED)
+                       subpass->depth_stencil_attachment = NULL;
+
+               for (uint32_t j = 0; j < subpass->attachment_count; j++) {
+                       struct radv_subpass_attachment *subpass_att =
+                               &subpass->attachments[j];
+                       if (subpass_att->attachment == VK_ATTACHMENT_UNUSED)
+                               continue;
+
+                       struct radv_render_pass_attachment *pass_att =
+                               &pass->attachments[subpass_att->attachment];
+
+                       pass_att->last_subpass_idx = i;
+               }
+
+               subpass->has_color_att = false;
+               for (uint32_t j = 0; j < subpass->color_count; j++) {
+                       struct radv_subpass_attachment *subpass_att =
+                               &subpass->color_attachments[j];
+                       if (subpass_att->attachment == VK_ATTACHMENT_UNUSED)
+                               continue;
+
+                       subpass->has_color_att = true;
+
+                       struct radv_render_pass_attachment *pass_att =
+                               &pass->attachments[subpass_att->attachment];
+
+                       color_sample_count = pass_att->samples;
+               }
+
+               if (subpass->depth_stencil_attachment) {
+                       const uint32_t a =
+                               subpass->depth_stencil_attachment->attachment;
+                       struct radv_render_pass_attachment *pass_att =
+                               &pass->attachments[a];
+                       depth_sample_count = pass_att->samples;
+               }
+
+               subpass->max_sample_count = MAX2(color_sample_count,
+                                                depth_sample_count);
+
+               /* We have to handle resolve attachments specially */
+               subpass->has_resolve = false;
+               if (subpass->resolve_attachments) {
+                       for (uint32_t j = 0; j < subpass->color_count; j++) {
+                               struct radv_subpass_attachment *resolve_att =
+                                       &subpass->resolve_attachments[j];
+
+                               if (resolve_att->attachment == VK_ATTACHMENT_UNUSED)
+                                       continue;
+
+                               subpass->has_resolve = true;
+                       }
+               }
+       }
+}
+
+static unsigned
+radv_num_subpass_attachments(const VkSubpassDescription *desc)
+{
+       return desc->inputAttachmentCount +
+              desc->colorAttachmentCount +
+              (desc->pResolveAttachments ? desc->colorAttachmentCount : 0) +
+              (desc->pDepthStencilAttachment != NULL);
+}
+
 VkResult radv_CreateRenderPass(
        VkDevice                                    _device,
        const VkRenderPassCreateInfo*               pCreateInfo,
@@ -38,7 +148,7 @@ VkResult radv_CreateRenderPass(
        struct radv_render_pass *pass;
        size_t size;
        size_t attachments_offset;
-       VkRenderPassMultiviewCreateInfoKHR *multiview_info = NULL;
+       VkRenderPassMultiviewCreateInfo *multiview_info = NULL;
 
        assert(pCreateInfo->sType == VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO);
 
@@ -59,8 +169,8 @@ VkResult radv_CreateRenderPass(
 
        vk_foreach_struct(ext, pCreateInfo->pNext) {
                switch(ext->sType) {
-               case  VK_STRUCTURE_TYPE_RENDER_PASS_MULTIVIEW_CREATE_INFO_KHR:
-                       multiview_info = ( VkRenderPassMultiviewCreateInfoKHR*)ext;
+               case  VK_STRUCTURE_TYPE_RENDER_PASS_MULTIVIEW_CREATE_INFO:
+                       multiview_info = (VkRenderPassMultiviewCreateInfo*)ext;
                        break;
                default:
                        break;
@@ -82,13 +192,8 @@ VkResult radv_CreateRenderPass(
        uint32_t subpass_attachment_count = 0;
        struct radv_subpass_attachment *p;
        for (uint32_t i = 0; i < pCreateInfo->subpassCount; i++) {
-               const VkSubpassDescription *desc = &pCreateInfo->pSubpasses[i];
-
                subpass_attachment_count +=
-                       desc->inputAttachmentCount +
-                       desc->colorAttachmentCount +
-                       (desc->pResolveAttachments ? desc->colorAttachmentCount : 0) +
-                       (desc->pDepthStencilAttachment != NULL);
+                       radv_num_subpass_attachments(&pCreateInfo->pSubpasses[i]);
        }
 
        if (subpass_attachment_count) {
@@ -106,11 +211,13 @@ VkResult radv_CreateRenderPass(
        p = pass->subpass_attachments;
        for (uint32_t i = 0; i < pCreateInfo->subpassCount; i++) {
                const VkSubpassDescription *desc = &pCreateInfo->pSubpasses[i];
-               uint32_t color_sample_count = 1, depth_sample_count = 1;
                struct radv_subpass *subpass = &pass->subpasses[i];
 
                subpass->input_count = desc->inputAttachmentCount;
                subpass->color_count = desc->colorAttachmentCount;
+               subpass->attachment_count = radv_num_subpass_attachments(desc);
+               subpass->attachments = p;
+
                if (multiview_info)
                        subpass->view_mask = multiview_info->pViewMasks[i];
 
@@ -123,8 +230,6 @@ VkResult radv_CreateRenderPass(
                                        .attachment = desc->pInputAttachments[j].attachment,
                                        .layout = desc->pInputAttachments[j].layout,
                                };
-                               if (desc->pInputAttachments[j].attachment != VK_ATTACHMENT_UNUSED)
-                                       pass->attachments[desc->pInputAttachments[j].attachment].view_mask |= subpass->view_mask;
                        }
                }
 
@@ -137,66 +242,61 @@ VkResult radv_CreateRenderPass(
                                        .attachment = desc->pColorAttachments[j].attachment,
                                        .layout = desc->pColorAttachments[j].layout,
                                };
-                               if (desc->pColorAttachments[j].attachment != VK_ATTACHMENT_UNUSED) {
-                                       pass->attachments[desc->pColorAttachments[j].attachment].view_mask |= subpass->view_mask;
-                                       color_sample_count = pCreateInfo->pAttachments[desc->pColorAttachments[j].attachment].samples;
-                               }
                        }
                }
 
-               subpass->has_resolve = false;
                if (desc->pResolveAttachments) {
                        subpass->resolve_attachments = p;
                        p += desc->colorAttachmentCount;
 
                        for (uint32_t j = 0; j < desc->colorAttachmentCount; j++) {
-                               uint32_t a = desc->pResolveAttachments[j].attachment;
                                subpass->resolve_attachments[j] = (struct radv_subpass_attachment) {
                                        .attachment = desc->pResolveAttachments[j].attachment,
                                        .layout = desc->pResolveAttachments[j].layout,
                                };
-                               if (a != VK_ATTACHMENT_UNUSED) {
-                                       subpass->has_resolve = true;
-                                       pass->attachments[desc->pResolveAttachments[j].attachment].view_mask |= subpass->view_mask;
-                               }
                        }
                }
 
                if (desc->pDepthStencilAttachment) {
-                       subpass->depth_stencil_attachment = (struct radv_subpass_attachment) {
+                       subpass->depth_stencil_attachment = p++;
+
+                       *subpass->depth_stencil_attachment = (struct radv_subpass_attachment) {
                                .attachment = desc->pDepthStencilAttachment->attachment,
                                .layout = desc->pDepthStencilAttachment->layout,
                        };
-                       if (desc->pDepthStencilAttachment->attachment != VK_ATTACHMENT_UNUSED) {
-                               pass->attachments[desc->pDepthStencilAttachment->attachment].view_mask |= subpass->view_mask;
-                               depth_sample_count = pCreateInfo->pAttachments[desc->pDepthStencilAttachment->attachment].samples;
-                       }
-               } else {
-                       subpass->depth_stencil_attachment.attachment = VK_ATTACHMENT_UNUSED;
                }
-
-               subpass->max_sample_count = MAX2(color_sample_count,
-                                                depth_sample_count);
        }
 
        for (unsigned i = 0; i < pCreateInfo->dependencyCount; ++i) {
-               uint32_t dst = pCreateInfo->pDependencies[i].dstSubpass;
-               if (dst == VK_SUBPASS_EXTERNAL) {
-                       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 {
-                       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;
-               }
+               /* Convert to a Dependency2KHR */
+               struct VkSubpassDependency2KHR dep2 = {
+                       .srcSubpass       = pCreateInfo->pDependencies[i].srcSubpass,
+                       .dstSubpass       = pCreateInfo->pDependencies[i].dstSubpass,
+                       .srcStageMask     = pCreateInfo->pDependencies[i].srcStageMask,
+                       .dstStageMask     = pCreateInfo->pDependencies[i].dstStageMask,
+                       .srcAccessMask    = pCreateInfo->pDependencies[i].srcAccessMask,
+                       .dstAccessMask    = pCreateInfo->pDependencies[i].dstAccessMask,
+                       .dependencyFlags  = pCreateInfo->pDependencies[i].dependencyFlags,
+               };
+               radv_render_pass_add_subpass_dep(pass, &dep2);
        }
 
+       radv_render_pass_compile(pass);
+
        *pRenderPass = radv_render_pass_to_handle(pass);
 
        return VK_SUCCESS;
 }
 
+static unsigned
+radv_num_subpass_attachments2(const VkSubpassDescription2KHR *desc)
+{
+       return desc->inputAttachmentCount +
+              desc->colorAttachmentCount +
+              (desc->pResolveAttachments ? desc->colorAttachmentCount : 0) +
+              (desc->pDepthStencilAttachment != NULL);
+}
+
 VkResult radv_CreateRenderPass2KHR(
     VkDevice                                    _device,
     const VkRenderPassCreateInfo2KHR*           pCreateInfo,
@@ -240,13 +340,8 @@ VkResult radv_CreateRenderPass2KHR(
        uint32_t subpass_attachment_count = 0;
        struct radv_subpass_attachment *p;
        for (uint32_t i = 0; i < pCreateInfo->subpassCount; i++) {
-               const VkSubpassDescription2KHR *desc = &pCreateInfo->pSubpasses[i];
-
                subpass_attachment_count +=
-                       desc->inputAttachmentCount +
-                       desc->colorAttachmentCount +
-                       (desc->pResolveAttachments ? desc->colorAttachmentCount : 0) +
-                       (desc->pDepthStencilAttachment != NULL);
+                       radv_num_subpass_attachments2(&pCreateInfo->pSubpasses[i]);
        }
 
        if (subpass_attachment_count) {
@@ -264,11 +359,12 @@ VkResult radv_CreateRenderPass2KHR(
        p = pass->subpass_attachments;
        for (uint32_t i = 0; i < pCreateInfo->subpassCount; i++) {
                const VkSubpassDescription2KHR *desc = &pCreateInfo->pSubpasses[i];
-               uint32_t color_sample_count = 1, depth_sample_count = 1;
                struct radv_subpass *subpass = &pass->subpasses[i];
 
                subpass->input_count = desc->inputAttachmentCount;
                subpass->color_count = desc->colorAttachmentCount;
+               subpass->attachment_count = radv_num_subpass_attachments2(desc);
+               subpass->attachments = p;
                subpass->view_mask = desc->viewMask;
 
                if (desc->inputAttachmentCount > 0) {
@@ -280,8 +376,6 @@ VkResult radv_CreateRenderPass2KHR(
                                        .attachment = desc->pInputAttachments[j].attachment,
                                        .layout = desc->pInputAttachments[j].layout,
                                };
-                               if (desc->pInputAttachments[j].attachment != VK_ATTACHMENT_UNUSED)
-                                       pass->attachments[desc->pInputAttachments[j].attachment].view_mask |= subpass->view_mask;
                        }
                }
 
@@ -294,71 +388,38 @@ VkResult radv_CreateRenderPass2KHR(
                                        .attachment = desc->pColorAttachments[j].attachment,
                                        .layout = desc->pColorAttachments[j].layout,
                                };
-                               if (desc->pColorAttachments[j].attachment != VK_ATTACHMENT_UNUSED) {
-                                       pass->attachments[desc->pColorAttachments[j].attachment].view_mask |= subpass->view_mask;
-                                       color_sample_count = pCreateInfo->pAttachments[desc->pColorAttachments[j].attachment].samples;
-                               }
                        }
                }
 
-               subpass->has_resolve = false;
                if (desc->pResolveAttachments) {
                        subpass->resolve_attachments = p;
                        p += desc->colorAttachmentCount;
 
                        for (uint32_t j = 0; j < desc->colorAttachmentCount; j++) {
-                               uint32_t a = desc->pResolveAttachments[j].attachment;
                                subpass->resolve_attachments[j] = (struct radv_subpass_attachment) {
                                        .attachment = desc->pResolveAttachments[j].attachment,
                                        .layout = desc->pResolveAttachments[j].layout,
                                };
-                               if (a != VK_ATTACHMENT_UNUSED) {
-                                       subpass->has_resolve = true;
-                                       pass->attachments[desc->pResolveAttachments[j].attachment].view_mask |= subpass->view_mask;
-                               }
                        }
                }
 
                if (desc->pDepthStencilAttachment) {
-                       subpass->depth_stencil_attachment = (struct radv_subpass_attachment) {
+                       subpass->depth_stencil_attachment = p++;
+
+                       *subpass->depth_stencil_attachment = (struct radv_subpass_attachment) {
                                .attachment = desc->pDepthStencilAttachment->attachment,
                                .layout = desc->pDepthStencilAttachment->layout,
                        };
-                       if (desc->pDepthStencilAttachment->attachment != VK_ATTACHMENT_UNUSED) {
-                               pass->attachments[desc->pDepthStencilAttachment->attachment].view_mask |= subpass->view_mask;
-                               depth_sample_count = pCreateInfo->pAttachments[desc->pDepthStencilAttachment->attachment].samples;
-                       }
-               } else {
-                       subpass->depth_stencil_attachment.attachment = VK_ATTACHMENT_UNUSED;
                }
-
-               subpass->max_sample_count = MAX2(color_sample_count,
-                                                depth_sample_count);
        }
 
        for (unsigned i = 0; i < pCreateInfo->dependencyCount; ++i) {
-               uint32_t src = pCreateInfo->pDependencies[i].srcSubpass;
-               uint32_t dst = pCreateInfo->pDependencies[i].dstSubpass;
-
-               /* Ignore subpass self-dependencies as they allow the app to
-                * call vkCmdPipelineBarrier() inside the render pass and the
-                * driver should only do the barrier when called, not when
-                * starting the render pass.
-                */
-               if (src == dst)
-                       continue;
-
-               if (dst == VK_SUBPASS_EXTERNAL) {
-                       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 {
-                       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;
-               }
+               radv_render_pass_add_subpass_dep(pass,
+                                                &pCreateInfo->pDependencies[i]);
        }
 
+       radv_render_pass_compile(pass);
+
        *pRenderPass = radv_render_pass_to_handle(pass);
 
        return VK_SUCCESS;