Module: Mesa
Branch: master
Commit: f65bee7e85221ebbfa26168ca081ae20dc13a13b
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=f65bee7e85221ebbfa26168ca081ae20dc13a13b

Author: Samuel Pitoiset <[email protected]>
Date:   Thu Jul 12 16:26:34 2018 +0200

radv/winsys: use alloca() for semaphore dependencies

Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Bas Nieuwenhuizen <[email protected]>

---

 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 5824a29b9e..c94c0f339f 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;

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to