drm/i915: Mark up potential allocation paths within i915_sw_fence as might_sleep
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 12 Dec 2017 18:06:50 +0000 (18:06 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Wed, 13 Dec 2017 13:17:37 +0000 (13:17 +0000)
As kmalloc is allowed to block (if given the right flags), mark up the
two i915_sw_fence routines that may call kmalloc as potential sleeping
routines.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20171212180652.22061-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_sw_fence.c

index ac236b88c99ca0fb07c49ec5d9487b1e0b38adfe..3669f5eeb91efa709ea2d6f9f83525f98230f7c5 100644 (file)
@@ -303,6 +303,7 @@ static int __i915_sw_fence_await_sw_fence(struct i915_sw_fence *fence,
        int pending;
 
        debug_fence_assert(fence);
+       might_sleep_if(gfpflags_allow_blocking(gfp));
 
        if (i915_sw_fence_done(signaler))
                return 0;
@@ -419,6 +420,7 @@ int i915_sw_fence_await_dma_fence(struct i915_sw_fence *fence,
        int ret;
 
        debug_fence_assert(fence);
+       might_sleep_if(gfpflags_allow_blocking(gfp));
 
        if (dma_fence_is_signaled(dma))
                return 0;
@@ -465,6 +467,7 @@ int i915_sw_fence_await_reservation(struct i915_sw_fence *fence,
        int ret = 0, pending;
 
        debug_fence_assert(fence);
+       might_sleep_if(gfpflags_allow_blocking(gfp));
 
        if (write) {
                struct dma_fence **shared;