[Mesa-dev] [PATCH] radv/meta: missing initialisations in create_pass().

Bas Nieuwenhuizen bas at basnieuwenhuizen.nl
Thu Aug 24 21:44:31 UTC 2017


On Thu, Aug 24, 2017, at 11:10, Xavier Bouchoux wrote:
> Otherwise radv_cmd_state_setup_attachments() will complain it has no
> clearvalues,
> when called via radv_process_depth_image_inplace().
> 
> Signed-off-by: Xavier Bouchoux <xavierb at gmail.com>
> ---
>  src/amd/vulkan/radv_meta_decompress.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/amd/vulkan/radv_meta_decompress.c
> b/src/amd/vulkan/radv_meta_decompress.c
> index f68ce8d2b0..f932b4c492 100644
> --- a/src/amd/vulkan/radv_meta_decompress.c
> +++ b/src/amd/vulkan/radv_meta_decompress.c
> @@ -38,10 +38,13 @@ create_pass(struct radv_device *device,
>  	const VkAllocationCallbacks *alloc = &device->meta_state.alloc;
>  	VkAttachmentDescription attachment;
>  
> +       attachment.flags = 0;
>  	attachment.format = VK_FORMAT_D32_SFLOAT_S8_UINT;
>  	attachment.samples = samples;
>  	attachment.loadOp = VK_ATTACHMENT_LOAD_OP_LOAD;
>  	attachment.storeOp = VK_ATTACHMENT_STORE_OP_STORE;
> +       attachment.stencilLoadOp = VK_ATTACHMENT_LOAD_OP_DONT_CARE;
> +       attachment.stencilStoreOp = VK_ATTACHMENT_STORE_OP_DONT_CARE;

I think we should make these LOAD/STORE instead of DONT_CARE, since
HTILE decompression needs to preserve stencil.

>  	attachment.initialLayout = VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL;
>  	attachment.finalLayout = VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL;
>  
> -- 
> 2.14.1
> 
> _______________________________________________
> 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