On ti, 2016-09-20 at 09:29 +0100, Chris Wilson wrote:
> +     if (!fence_is_array(fence)) {
> +             ret = i915_sw_fence_await_dma_fence(&req->submit,
> +                                                 fence, 10*HZ,

#define I915_MAX_USERSPACE_WAIT or something. We already have two
instances of it with only one explanation. Like mentinoed in the
previous review.

> +                                                 GFP_KERNEL);
> +             return ret < 0 ? ret : 0;
> +     }
> +

With the #define added;

Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to