dma-buf/sync_file: cleanup fence merging a bit
authorChristian König <christian.koenig@amd.com>
Fri, 22 Apr 2022 12:38:47 +0000 (14:38 +0200)
committerChristian König <christian.koenig@amd.com>
Wed, 4 May 2022 10:09:53 +0000 (12:09 +0200)
krealloc_array() ignores attempts to reduce the array size, so the attempt
to save memory is completely pointless here.

Also move testing for the no fence case into sync_file_set_fence(), this
way we don't even touch the fence array when we don't have any fences.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20220426124637.329764-1-christian.koenig@amd.com
drivers/dma-buf/sync_file.c

index 514d213261df3d8f579dedec1269c353b84a7f0a..0fe56453916673be4a06cb2b11e6023c845f288c 100644 (file)
@@ -157,9 +157,15 @@ static int sync_file_set_fence(struct sync_file *sync_file,
         * we already own a new reference to the fence. For num_fence > 1
         * we own the reference of the dma_fence_array creation.
         */
-       if (num_fences == 1) {
+
+       if (num_fences == 0) {
+               sync_file->fence = dma_fence_get_stub();
+               kfree(fences);
+
+       } else if (num_fences == 1) {
                sync_file->fence = fences[0];
                kfree(fences);
+
        } else {
                array = dma_fence_array_create(num_fences, fences,
                                               dma_fence_context_alloc(1),
@@ -261,19 +267,6 @@ static struct sync_file *sync_file_merge(const char *name, struct sync_file *a,
                }
        }
 
-       if (index == 0)
-               fences[index++] = dma_fence_get_stub();
-
-       if (num_fences > index) {
-               struct dma_fence **tmp;
-
-               /* Keep going even when reducing the size failed */
-               tmp = krealloc_array(fences, index, sizeof(*fences),
-                                    GFP_KERNEL);
-               if (tmp)
-                       fences = tmp;
-       }
-
        if (sync_file_set_fence(sync_file, fences, index) < 0)
                goto err_put_fences;