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.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx