From: Jason Ekstrand Date: Tue, 26 Jul 2016 22:29:40 +0000 (-0700) Subject: anv/image: Don't create invalid render target surfaces X-Git-Tag: upstream/17.1.0~7108 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=954c0bfb209c96b05f01bac8cdbead8d306505bb;p=platform%2Fupstream%2Fmesa.git anv/image: Don't create invalid render target surfaces Signed-off-by: Jason Ekstrand --- diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index 7fd8268..af6b4ca 100644 --- a/src/intel/vulkan/anv_image.c +++ b/src/intel/vulkan/anv_image.c @@ -456,7 +456,18 @@ anv_image_view_init(struct anv_image_view *iview, iview->sampler_surface_state.alloc_size = 0; } - if (image->usage & usage_mask & VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT) { + /* This is kind-of hackish. It is possible, due to get_full_usage above, + * to get a surface state with a non-renderable format but with + * VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT. This happens in particular for + * formats which aren't renderable but where we want to use Vulkan copy + * commands so VK_IMAGE_USAGE_TRANSFER_DST_BIT is set. In the case of a + * copy, meta will use a format that we can render to, but most of the rest + * of the time, we don't want to create those surface states. Once we + * start using blorp for copies, this problem will go away and we can + * remove a lot of hacks. + */ + if ((image->usage & usage_mask & VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT) && + isl_format_supports_rendering(&device->info, isl_view.format)) { iview->color_rt_surface_state = alloc_surface_state(device, cmd_buffer); isl_view.usage = cube_usage | ISL_SURF_USAGE_RENDER_TARGET_BIT;