On Tue, Dec 17, 2013 at 11:42:11PM +0100, Daniel Vetter wrote:
> But only when we indeed set up a gtt mapping. We need this since the
> vma also holds a pages_pin_count, on top of the unconditional
> pages_pin_count we grab for all stolen objects (to avoid swap-out).
> 
> This should avoid a pages_pin_count underrun when cleaning up
> framebuffers objects taken over from the BIOS.
> 
> Reported-by: Jesse Barnes <jbar...@virtuousgeek.org>
> Cc: Jesse Barnes <jbar...@virtuousgeek.org>
> Cc: Ben Widawsky <benjamin.widaw...@intel.com>
> Signed-off-by: Daniel Vetter <daniel.vet...@ffwll.ch>

The bug is older than the introduction of vma.
Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to