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 <ch...@chris-wilson.co.uk>

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.lahti...@linux.intel.com>

Regards, Joonas
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
Intel-gfx mailing list

Reply via email to