From f65bee7e85221ebbfa26168ca081ae20dc13a13b Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Thu, 12 Jul 2018 16:26:34 +0200 Subject: [PATCH] radv/winsys: use alloca() for semaphore dependencies Signed-off-by: Samuel Pitoiset Reviewed-by: Bas Nieuwenhuizen --- src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c index 5824a29b9e4..c94c0f339fd 100644 --- a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c +++ b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c @@ -1376,12 +1376,9 @@ static int radv_amdgpu_cs_submit(struct radv_amdgpu_ctx *ctx, } if (sem_info->wait.sem_count && sem_info->cs_emit_wait) { - sem_dependencies = malloc(sizeof(struct drm_amdgpu_cs_chunk_dep) * sem_info->wait.sem_count); - if (!sem_dependencies) { - r = -ENOMEM; - goto error_out; - } + sem_dependencies = alloca(sizeof(struct drm_amdgpu_cs_chunk_dep) * sem_info->wait.sem_count); int sem_count = 0; + for (unsigned j = 0; j < sem_info->wait.sem_count; j++) { sem = (struct amdgpu_cs_fence *)sem_info->wait.sem[j]; if (!sem->context) @@ -1420,7 +1417,6 @@ static int radv_amdgpu_cs_submit(struct radv_amdgpu_ctx *ctx, chunks, &request->seq_no); error_out: - free(sem_dependencies); free(wait_syncobj); free(signal_syncobj); return r; -- 2.30.2