On Tue, 18 Dec 2012 15:24:37 +0100, Daniel Vetter <[email protected]> 
wrote:
> We need to clean up the overlay first, before taking down the
> stolen memory allocator.
> 
> This regression has been introducec in
> 
> commit 8040513870399f1cb032cb8bc805df5042fedcdf
> Author: Chris Wilson <[email protected]>
> Date:   Thu Nov 15 11:32:29 2012 +0000
> 
>     drm/i915: Allocate overlay registers from stolen memory
> 
> Note: This is just a quick hack to shut up a warning in the module
> unload code, so that I can check again whether we don't leak any
> framebuffers.
> 
> v2: Rework the patch a bit as suggested by Chris Wilson:
> - move the overlay teardown up, into the modeset cleanup
> - move the stolen mm takedown into i915_gem_cleanup_stolen
> 
> Cc: Chris Wilson <[email protected]>
> Signed-off-by: Daniel Vetter <[email protected]>

That organisation indeed makes more sense.

Reviewed-by: Chris Wilson <[email protected]>
-Chris
-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to