[PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-08 Thread Chris Wilson
On Wed, Aug 07, 2013 at 08:50:20PM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Aug 07, 2013 at 12:09:32PM +0200, Daniel Vetter wrote: > > This fixes a WARN in i915_gem_free_object when the > > obj->pages_pin_count isn't 0. > > > > v2: Add locking to unmap, noticed by Chris Wilson. Note that

[PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-07 Thread Konrad Rzeszutek Wilk
On Wed, Aug 07, 2013 at 12:09:32PM +0200, Daniel Vetter wrote: > This fixes a WARN in i915_gem_free_object when the > obj->pages_pin_count isn't 0. > > v2: Add locking to unmap, noticed by Chris Wilson. Note that even > though we call unmap with our own dev->struct_mutex held that won't > result

[PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-07 Thread Maarten Lankhorst
Op 07-08-13 12:09, Daniel Vetter schreef: > This fixes a WARN in i915_gem_free_object when the > obj->pages_pin_count isn't 0. > > v2: Add locking to unmap, noticed by Chris Wilson. Note that even > though we call unmap with our own dev->struct_mutex held that won't > result in an immediate

[PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-07 Thread Daniel Vetter
This fixes a WARN in i915_gem_free_object when the obj->pages_pin_count isn't 0. v2: Add locking to unmap, noticed by Chris Wilson. Note that even though we call unmap with our own dev->struct_mutex held that won't result in an immediate deadlock since we never go through the dma_buf interfaces

[PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-07 Thread Daniel Vetter
This fixes a WARN in i915_gem_free_object when the obj-pages_pin_count isn't 0. v2: Add locking to unmap, noticed by Chris Wilson. Note that even though we call unmap with our own dev-struct_mutex held that won't result in an immediate deadlock since we never go through the dma_buf interfaces for

Re: [PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-07 Thread Maarten Lankhorst
Op 07-08-13 12:09, Daniel Vetter schreef: This fixes a WARN in i915_gem_free_object when the obj-pages_pin_count isn't 0. v2: Add locking to unmap, noticed by Chris Wilson. Note that even though we call unmap with our own dev-struct_mutex held that won't result in an immediate deadlock since

Re: [PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-07 Thread Konrad Rzeszutek Wilk
On Wed, Aug 07, 2013 at 12:09:32PM +0200, Daniel Vetter wrote: This fixes a WARN in i915_gem_free_object when the obj-pages_pin_count isn't 0. v2: Add locking to unmap, noticed by Chris Wilson. Note that even though we call unmap with our own dev-struct_mutex held that won't result in an

Re: [PATCH 1/2] drm/i915: unpin backing storage in dmabuf_unmap

2013-08-07 Thread Chris Wilson
On Wed, Aug 07, 2013 at 08:50:20PM -0400, Konrad Rzeszutek Wilk wrote: On Wed, Aug 07, 2013 at 12:09:32PM +0200, Daniel Vetter wrote: This fixes a WARN in i915_gem_free_object when the obj-pages_pin_count isn't 0. v2: Add locking to unmap, noticed by Chris Wilson. Note that even though