drm/v3d: Drop reservation of a shared slot in the dma-buf reservations.
authorEric Anholt <eric@anholt.net>
Tue, 16 Apr 2019 22:58:55 +0000 (15:58 -0700)
committerEric Anholt <eric@anholt.net>
Thu, 18 Apr 2019 16:54:13 +0000 (09:54 -0700)
We only set the excl (possible-writing) fence pointer and never add a
shared (read-only) fence.

Signed-off-by: Eric Anholt <eric@anholt.net>
Link: https://patchwork.freedesktop.org/patch/msgid/20190416225856.20264-5-eric@anholt.net
Acked-by: Rob Clark <robdclark@gmail.com>
drivers/gpu/drm/v3d/v3d_gem.c

index 8bd6fa69f566d039bdcffedfb8a3b2394c5523b5..6873b14a0d38af2024fb6c8a7508d69772090776 100644 (file)
@@ -253,18 +253,6 @@ v3d_lock_bo_reservations(struct drm_gem_object **bos,
        if (ret)
                return ret;
 
-       /* Reserve space for our shared (read-only) fence references,
-        * before we commit the CL to the hardware.
-        */
-       for (i = 0; i < bo_count; i++) {
-               ret = reservation_object_reserve_shared(bos[i]->resv, 1);
-               if (ret) {
-                       drm_gem_unlock_reservations(bos, bo_count,
-                                                   acquire_ctx);
-                       return ret;
-               }
-       }
-
        return 0;
 }