Mesa (master): radv: Generate storage image descriptors unconditionally

Dave Airlie airlied at kemper.freedesktop.org
Fri Jul 21 20:40:42 UTC 2017


Module: Mesa
Branch: master
Commit: af9d6a8a99fa67bade628028b7265ae21445a930
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=af9d6a8a99fa67bade628028b7265ae21445a930

Author: Alex Smith <asmith at feralinteractive.com>
Date:   Fri Jul 21 17:00:00 2017 +0100

radv: Generate storage image descriptors unconditionally

We can also use storage images internally for resolves, which don't
require TRANSFER_DST usage on the image, so currently we may not create
the needed descriptors.

Just create these descriptors unconditionally.

Fixes: 0e1886efb9e ("radv: Fix descriptors for cube images with VK_IMAGE_USAGE_STORAGE_BIT")
Reported-by: Grazvydas Ignotas <notasas at gmail.com>
Signed-off-by: Alex Smith <asmith at feralinteractive.com>
Signed-off-by: Dave Airlie <airlied at redhat.com>

---

 src/amd/vulkan/radv_image.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/amd/vulkan/radv_image.c b/src/amd/vulkan/radv_image.c
index f5fb790dda..d8b7e8f421 100644
--- a/src/amd/vulkan/radv_image.c
+++ b/src/amd/vulkan/radv_image.c
@@ -945,10 +945,7 @@ radv_image_view_init(struct radv_image_view *iview,
 	iview->base_mip = range->baseMipLevel;
 
 	radv_image_view_make_descriptor(iview, device, pCreateInfo, false);
-
-	/* For transfers we may use the image as a storage image. */
-	if (image->usage & (VK_IMAGE_USAGE_STORAGE_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT))
-		radv_image_view_make_descriptor(iview, device, pCreateInfo, true);
+	radv_image_view_make_descriptor(iview, device, pCreateInfo, true);
 }
 
 bool radv_layout_has_htile(const struct radv_image *image,




More information about the mesa-commit mailing list