radv: only pass the global BO list at submit time if enabled
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Thu, 17 May 2018 09:36:09 +0000 (11:36 +0200)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Thu, 17 May 2018 11:48:27 +0000 (13:48 +0200)
That way the winsys might use a faster path when the global
BO list is NULL.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
src/amd/vulkan/radv_device.c

index 2ce0c9dbd0fcc14b6776659d51fb74625aa80f5d..e24b8c2a7662e5b02d799a00c198818cc6f5c2d8 100644 (file)
@@ -2472,6 +2472,8 @@ VkResult radv_QueueSubmit(
 
                for (uint32_t j = 0; j < pSubmits[i].commandBufferCount; j += advance) {
                        struct radeon_winsys_cs *initial_preamble = (do_flush && !j) ? initial_flush_preamble_cs : initial_preamble_cs;
+                       const struct radv_winsys_bo_list *bo_list = NULL;
+
                        advance = MIN2(max_cs_submission,
                                       pSubmits[i].commandBufferCount - j);
 
@@ -2481,12 +2483,14 @@ VkResult radv_QueueSubmit(
                        sem_info.cs_emit_wait = j == 0;
                        sem_info.cs_emit_signal = j + advance == pSubmits[i].commandBufferCount;
 
-                       if (unlikely(queue->device->use_global_bo_list))
+                       if (unlikely(queue->device->use_global_bo_list)) {
                                pthread_mutex_lock(&queue->device->bo_list.mutex);
+                               bo_list = &queue->device->bo_list.list;
+                       }
 
                        ret = queue->device->ws->cs_submit(ctx, queue->queue_idx, cs_array + j,
                                                        advance, initial_preamble, continue_preamble_cs,
-                                                       &sem_info, &queue->device->bo_list.list,
+                                                       &sem_info, bo_list,
                                                        can_patch, base_fence);
 
                        if (unlikely(queue->device->use_global_bo_list))