On ti, 2016-09-20 at 09:29 +0100, Chris Wilson wrote: > +static struct intel_engine_cs * > +last_write_engine(struct drm_i915_gem_object *obj) This function is repeated below, better move it to header as __i915_gem_obj_last_write_engine? I'd say there is not an ABI change now as non-i915 fences were not supported before either. So with the duplicate code removed, 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