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 <[email protected]>

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 <[email protected]>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to