[Intel-gfx] [PATCH 24/41] drm/i915: Use lockless object free

Joonas Lahtinen joonas.lahtinen at linux.intel.com
Tue Oct 18 08:50:33 UTC 2016


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 at 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.lahtinen at linux.intel.com>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation


More information about the Intel-gfx mailing list