From cda9f3d8ecce663b53c982236557dcd1b70a25b0 Mon Sep 17 00:00:00 2001 From: Dave Airlie Date: Tue, 31 Jan 2017 15:18:33 +1000 Subject: [PATCH] radv: handle VK_QUEUE_FAMILY_IGNORED in image transitions (v3) The CTS tests at least are using this, and we were totally ignoring it. This hopefully fixes the bouncing multisample CTS tests. v2: get family mask in ignored case from command buffer. v3: only change things in one place, use logic from Bas. Reviewed-by: Bas Nieuwenhuizen Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_cmd_buffer.c | 12 ++++++------ src/amd/vulkan/radv_image.c | 11 +++++++---- src/amd/vulkan/radv_meta_clear.c | 2 +- src/amd/vulkan/radv_private.h | 2 +- 4 files changed, 15 insertions(+), 12 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 97c6b63ce01..f281f33dc73 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -38,8 +38,8 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer, struct radv_image *image, VkImageLayout src_layout, VkImageLayout dst_layout, - int src_family, - int dst_family, + uint32_t src_family, + uint32_t dst_family, VkImageSubresourceRange range, VkImageAspectFlags pending_clears); @@ -2719,8 +2719,8 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer, struct radv_image *image, VkImageLayout src_layout, VkImageLayout dst_layout, - int src_family, - int dst_family, + uint32_t src_family, + uint32_t dst_family, VkImageSubresourceRange range, VkImageAspectFlags pending_clears) { @@ -2741,8 +2741,8 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer, return; } - unsigned src_queue_mask = radv_image_queue_family_mask(image, src_family); - unsigned dst_queue_mask = radv_image_queue_family_mask(image, dst_family); + unsigned src_queue_mask = radv_image_queue_family_mask(image, src_family, cmd_buffer->queue_family_index); + unsigned dst_queue_mask = radv_image_queue_family_mask(image, dst_family, cmd_buffer->queue_family_index); if (image->htile.size) radv_handle_depth_image_transition(cmd_buffer, image, src_layout, diff --git a/src/amd/vulkan/radv_image.c b/src/amd/vulkan/radv_image.c index f75f0088495..99d17376cf6 100644 --- a/src/amd/vulkan/radv_image.c +++ b/src/amd/vulkan/radv_image.c @@ -905,10 +905,13 @@ bool radv_layout_can_fast_clear(const struct radv_image *image, } -unsigned radv_image_queue_family_mask(const struct radv_image *image, int family) { - if (image->exclusive) - return 1u <queue_family_mask; +unsigned radv_image_queue_family_mask(const struct radv_image *image, uint32_t family, uint32_t queue_family) +{ + if (!image->exclusive) + return image->queue_family_mask; + if (family == VK_QUEUE_FAMILY_IGNORED) + return 1u << queue_family; + return 1u << family; } VkResult diff --git a/src/amd/vulkan/radv_meta_clear.c b/src/amd/vulkan/radv_meta_clear.c index 957b3885939..a42e8345716 100644 --- a/src/amd/vulkan/radv_meta_clear.c +++ b/src/amd/vulkan/radv_meta_clear.c @@ -844,7 +844,7 @@ emit_fast_color_clear(struct radv_cmd_buffer *cmd_buffer, if (!(cmd_buffer->device->debug_flags & RADV_DEBUG_FAST_CLEARS)) return false; - if (!radv_layout_can_fast_clear(iview->image, image_layout, radv_image_queue_family_mask(iview->image, cmd_buffer->queue_family_index))) + if (!radv_layout_can_fast_clear(iview->image, image_layout, radv_image_queue_family_mask(iview->image, cmd_buffer->queue_family_index, cmd_buffer->queue_family_index))) goto fail; if (vk_format_get_blocksizebits(iview->image->vk_format) > 64) goto fail; diff --git a/src/amd/vulkan/radv_private.h b/src/amd/vulkan/radv_private.h index 57aa9ead9b7..69ff3578e34 100644 --- a/src/amd/vulkan/radv_private.h +++ b/src/amd/vulkan/radv_private.h @@ -1083,7 +1083,7 @@ bool radv_layout_can_fast_clear(const struct radv_image *image, unsigned queue_mask); -unsigned radv_image_queue_family_mask(const struct radv_image *image, int family); +unsigned radv_image_queue_family_mask(const struct radv_image *image, uint32_t family, uint32_t queue_family); static inline uint32_t radv_get_layerCount(const struct radv_image *image, -- 2.30.2