On pe, 2016-10-14 at 13:18 +0100, Chris Wilson wrote: > Having moved the locked phase of freeing an object to a separate worker, > we can now declare to the core that we only need the unlocked variant of > driver->gem_free_object, and can use the simple unreference internally. > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Now has a commit message as suggested by John. With that, R-b from John too? > @@ -1562,7 +1562,7 @@ i915_gem_set_domain_ioctl(struct drm_device *dev, void *data, > out_pages: > i915_gem_object_unpin_pages(obj); > out_unlocked: "out_put:" or simply "out:"? With the "_unlocked" eliminated from the label; 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