From: Bas Nieuwenhuizen Date: Sun, 5 Mar 2017 21:25:20 +0000 (+0100) Subject: radv: Cache command buffers in command pool. X-Git-Tag: upstream/17.1.0~1702 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=682248db451fa164d964863339ca873a1f85aec9;p=platform%2Fupstream%2Fmesa.git radv: Cache command buffers in command pool. So that we don't keep allocating BOs for the IBs and upload buffers. We run some risk of memory increase with e.g. a bimodal size distribution of command buffers, but I haven't noticed a significant increase with dota2 and talos. Signed-off-by: Bas Nieuwenhuizen Reviewed-by: Dave Airlie --- diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 709ae8b..dec3e9f 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -187,6 +187,51 @@ fail: return result; } +static void +radv_cmd_buffer_destroy(struct radv_cmd_buffer *cmd_buffer) +{ + list_del(&cmd_buffer->pool_link); + + list_for_each_entry_safe(struct radv_cmd_buffer_upload, up, + &cmd_buffer->upload.list, list) { + cmd_buffer->device->ws->buffer_destroy(up->upload_bo); + list_del(&up->list); + free(up); + } + + if (cmd_buffer->upload.upload_bo) + cmd_buffer->device->ws->buffer_destroy(cmd_buffer->upload.upload_bo); + cmd_buffer->device->ws->cs_destroy(cmd_buffer->cs); + vk_free(&cmd_buffer->pool->alloc, cmd_buffer); +} + +static void radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer) +{ + + cmd_buffer->device->ws->cs_reset(cmd_buffer->cs); + + list_for_each_entry_safe(struct radv_cmd_buffer_upload, up, + &cmd_buffer->upload.list, list) { + cmd_buffer->device->ws->buffer_destroy(up->upload_bo); + list_del(&up->list); + free(up); + } + + cmd_buffer->scratch_size_needed = 0; + cmd_buffer->compute_scratch_size_needed = 0; + cmd_buffer->esgs_ring_size_needed = 0; + cmd_buffer->gsvs_ring_size_needed = 0; + + if (cmd_buffer->upload.upload_bo) + cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs, + cmd_buffer->upload.upload_bo, 8); + cmd_buffer->upload.offset = 0; + + cmd_buffer->record_fail = false; + + cmd_buffer->ring_offsets_idx = -1; +} + static bool radv_cmd_buffer_resize_upload_buf(struct radv_cmd_buffer *cmd_buffer, uint64_t min_needed) @@ -1589,8 +1634,22 @@ VkResult radv_AllocateCommandBuffers( sizeof(*pCommandBuffers)*pAllocateInfo->commandBufferCount); for (i = 0; i < pAllocateInfo->commandBufferCount; i++) { - result = radv_create_cmd_buffer(device, pool, pAllocateInfo->level, - &pCommandBuffers[i]); + + if (!list_empty(&pool->free_cmd_buffers)) { + struct radv_cmd_buffer *cmd_buffer = list_first_entry(&pool->free_cmd_buffers, struct radv_cmd_buffer, pool_link); + + list_del(&cmd_buffer->pool_link); + list_addtail(&cmd_buffer->pool_link, &pool->cmd_buffers); + + radv_reset_cmd_buffer(cmd_buffer); + cmd_buffer->level = pAllocateInfo->level; + + pCommandBuffers[i] = radv_cmd_buffer_to_handle(cmd_buffer); + result = VK_SUCCESS; + } else { + result = radv_create_cmd_buffer(device, pool, pAllocateInfo->level, + &pCommandBuffers[i]); + } if (result != VK_SUCCESS) break; } @@ -1602,24 +1661,6 @@ VkResult radv_AllocateCommandBuffers( return result; } -static void -radv_cmd_buffer_destroy(struct radv_cmd_buffer *cmd_buffer) -{ - list_del(&cmd_buffer->pool_link); - - list_for_each_entry_safe(struct radv_cmd_buffer_upload, up, - &cmd_buffer->upload.list, list) { - cmd_buffer->device->ws->buffer_destroy(up->upload_bo); - list_del(&up->list); - free(up); - } - - if (cmd_buffer->upload.upload_bo) - cmd_buffer->device->ws->buffer_destroy(cmd_buffer->upload.upload_bo); - cmd_buffer->device->ws->cs_destroy(cmd_buffer->cs); - vk_free(&cmd_buffer->pool->alloc, cmd_buffer); -} - void radv_FreeCommandBuffers( VkDevice device, VkCommandPool commandPool, @@ -1629,36 +1670,15 @@ void radv_FreeCommandBuffers( for (uint32_t i = 0; i < commandBufferCount; i++) { RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, pCommandBuffers[i]); - if (cmd_buffer) - radv_cmd_buffer_destroy(cmd_buffer); - } -} + if (cmd_buffer) { + if (cmd_buffer->pool) { + list_del(&cmd_buffer->pool_link); + list_addtail(&cmd_buffer->pool_link, &cmd_buffer->pool->free_cmd_buffers); + } else + radv_cmd_buffer_destroy(cmd_buffer); -static void radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer) -{ - - cmd_buffer->device->ws->cs_reset(cmd_buffer->cs); - - list_for_each_entry_safe(struct radv_cmd_buffer_upload, up, - &cmd_buffer->upload.list, list) { - cmd_buffer->device->ws->buffer_destroy(up->upload_bo); - list_del(&up->list); - free(up); + } } - - cmd_buffer->scratch_size_needed = 0; - cmd_buffer->compute_scratch_size_needed = 0; - cmd_buffer->esgs_ring_size_needed = 0; - cmd_buffer->gsvs_ring_size_needed = 0; - - if (cmd_buffer->upload.upload_bo) - cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs, - cmd_buffer->upload.upload_bo, 8); - cmd_buffer->upload.offset = 0; - - cmd_buffer->record_fail = false; - - cmd_buffer->ring_offsets_idx = -1; } VkResult radv_ResetCommandBuffer( @@ -2140,6 +2160,7 @@ VkResult radv_CreateCommandPool( pool->alloc = device->alloc; list_inithead(&pool->cmd_buffers); + list_inithead(&pool->free_cmd_buffers); pool->queue_family_index = pCreateInfo->queueFamilyIndex; @@ -2165,6 +2186,11 @@ void radv_DestroyCommandPool( radv_cmd_buffer_destroy(cmd_buffer); } + list_for_each_entry_safe(struct radv_cmd_buffer, cmd_buffer, + &pool->free_cmd_buffers, pool_link) { + radv_cmd_buffer_destroy(cmd_buffer); + } + vk_free2(&device->alloc, pAllocator, pool); } @@ -2188,6 +2214,15 @@ void radv_TrimCommandPoolKHR( VkCommandPool commandPool, VkCommandPoolTrimFlagsKHR flags) { + RADV_FROM_HANDLE(radv_cmd_pool, pool, commandPool); + + if (!pool) + return; + + list_for_each_entry_safe(struct radv_cmd_buffer, cmd_buffer, + &pool->free_cmd_buffers, pool_link) { + radv_cmd_buffer_destroy(cmd_buffer); + } } void radv_CmdBeginRenderPass( diff --git a/src/amd/vulkan/radv_private.h b/src/amd/vulkan/radv_private.h index 37549c5..bc8e447 100644 --- a/src/amd/vulkan/radv_private.h +++ b/src/amd/vulkan/radv_private.h @@ -710,6 +710,7 @@ struct radv_cmd_state { struct radv_cmd_pool { VkAllocationCallbacks alloc; struct list_head cmd_buffers; + struct list_head free_cmd_buffers; uint32_t queue_family_index; };