anv: allow binding tables allocations on compute only queues
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Fri, 2 Jun 2023 07:02:29 +0000 (10:02 +0300)
committerMarge Bot <emma+marge@anholt.net>
Thu, 8 Jun 2023 02:37:44 +0000 (02:37 +0000)
COMPUTE_WALKER is using binding tables.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Fixes: eb2b309328 ("anv: defer binding table block allocation to when necessary")
Reviewed-by: Sagar Ghuge <sagar.ghuge@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23506>

src/intel/vulkan/anv_batch_chain.c

index 1880e63..a25f9cc 100644 (file)
@@ -425,8 +425,9 @@ anv_cmd_buffer_current_generation_batch_bo(struct anv_cmd_buffer *cmd_buffer)
 struct anv_address
 anv_cmd_buffer_surface_base_address(struct anv_cmd_buffer *cmd_buffer)
 {
-   /* Compute only queues don't need binding tables. */
-   if (!(cmd_buffer->queue_family->queueFlags & VK_QUEUE_GRAPHICS_BIT))
+   /* Only graphics & compute queues need binding tables. */
+   if (!(cmd_buffer->queue_family->queueFlags & (VK_QUEUE_GRAPHICS_BIT |
+                                                 VK_QUEUE_COMPUTE_BIT)))
       return ANV_NULL_ADDRESS;
 
    /* If we've never allocated a binding table block, do it now. Otherwise we