turnip: CmdClearAttachments fixes
authorJonathan Marek <jonathan@marek.ca>
Thu, 12 Dec 2019 19:02:49 +0000 (14:02 -0500)
committerJonathan Marek <jonathan@marek.ca>
Fri, 13 Dec 2019 01:33:17 +0000 (20:33 -0500)
Partial depth/stencil clear and skipping unused attachments.

Signed-off-by: Jonathan Marek <jonathan@marek.ca>
Reviewed-by: Eric Anholt <eric@anholt.net>
src/freedreno/vulkan/tu_meta_clear.c

index c539a1b65c21ce8eabd01dfe5b3fc3c9320fa3a0..5f5fe44c6764cce907640fa6da9a59eccbaa05b2 100644 (file)
@@ -127,11 +127,20 @@ tu_CmdClearAttachments(VkCommandBuffer commandBuffer,
 
       for (unsigned j = 0; j < attachmentCount; j++) {
          uint32_t a;
-         if (pAttachments[j].aspectMask & VK_IMAGE_ASPECT_COLOR_BIT)
+         unsigned clear_mask = 0;
+         if (pAttachments[j].aspectMask & VK_IMAGE_ASPECT_COLOR_BIT) {
+            clear_mask = 0xf;
             a = subpass->color_attachments[pAttachments[j].colorAttachment].attachment;
-         else
+         } else {
             a = subpass->depth_stencil_attachment.attachment;
-         /* TODO: partial depth/stencil clear? */
+            if (pAttachments[j].aspectMask & VK_IMAGE_ASPECT_DEPTH_BIT)
+               clear_mask |= 1;
+            if (pAttachments[j].aspectMask & VK_IMAGE_ASPECT_STENCIL_BIT)
+               clear_mask |= 2;
+         }
+
+         if (a == VK_ATTACHMENT_UNUSED)
+               continue;
 
          VkFormat fmt = cmd->state.pass->attachments[a].format;
          const struct tu_native_format *format = tu6_get_native_format(fmt);
@@ -141,7 +150,7 @@ tu_CmdClearAttachments(VkCommandBuffer commandBuffer,
          tu_cs_emit(cs, A6XX_RB_BLIT_DST_INFO_COLOR_FORMAT(format->rb));
 
          tu_cs_emit_pkt4(cs, REG_A6XX_RB_BLIT_INFO, 1);
-         tu_cs_emit(cs, A6XX_RB_BLIT_INFO_GMEM | A6XX_RB_BLIT_INFO_CLEAR_MASK(0xf));
+         tu_cs_emit(cs, A6XX_RB_BLIT_INFO_GMEM | A6XX_RB_BLIT_INFO_CLEAR_MASK(clear_mask));
 
          tu_cs_emit_pkt4(cs, REG_A6XX_RB_BLIT_BASE_GMEM, 1);
          tu_cs_emit(cs, cmd->state.tiling_config.gmem_offsets[a]);