On Wed, Aug 07, 2013 at 11:15:07AM +0200, Daniel Vetter wrote:
> This fixes a WARN in i915_gem_free_object when the
> obj->pages_pin_count isn't 0.
> 
> Reported-by: Maarten Lankhorst <maarten.lankho...@canonical.com>
> Cc: Maarten Lankhorst <maarten.lankho...@canonical.com>
> Signed-off-by: Daniel Vetter <daniel.vet...@ffwll.ch>

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=67391

> ---
>  drivers/gpu/drm/i915/i915_gem_dmabuf.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_gem_dmabuf.c 
> b/drivers/gpu/drm/i915/i915_gem_dmabuf.c
> index 63ee1a9..0bf3d51 100644
> --- a/drivers/gpu/drm/i915/i915_gem_dmabuf.c
> +++ b/drivers/gpu/drm/i915/i915_gem_dmabuf.c
> @@ -85,9 +85,13 @@ static void i915_gem_unmap_dma_buf(struct 
> dma_buf_attachment *attachment,
>                                  struct sg_table *sg,
>                                  enum dma_data_direction dir)
>  {
> +     struct drm_i915_gem_object *obj = attachment->dmabuf->priv;
> +
>       dma_unmap_sg(attachment->dev, sg->sgl, sg->nents, dir);
>       sg_free_table(sg);
>       kfree(sg);
> +
> +     i915_gem_object_unpin_pages(obj);
>  }
>  
>  static void *i915_gem_dmabuf_vmap(struct dma_buf *dma_buf)
> -- 
> 1.8.3.2
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to