Re: [Intel-gfx] [PATCH] drm/i915: Prevent recursive deadlock on releasing a busy userptr

2014-09-03 Thread Jani Nikula
On Tue, 02 Sep 2014, Daniel Vetter wrote: > Queued for -next, thanks for the patch. Aside: Is there other userptr > stuff outstanding? I've definitely lost track of them :( Picked up for drm-intel-fixes instead. Thanks. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center ___

Re: [Intel-gfx] [PATCH] drm/i915: Prevent recursive deadlock on releasing a busy userptr

2014-09-02 Thread Jani Nikula
On Tue, 02 Sep 2014, Chris Wilson wrote: > On Tue, Sep 02, 2014 at 11:35:00AM +0200, Daniel Vetter wrote: >> On Mon, Sep 01, 2014 at 02:23:17PM +0100, Tvrtko Ursulin wrote: >> > >> > On 08/07/2014 02:20 PM, Chris Wilson wrote: >> > >During release of the GEM object we hold the struct_mutex. As th

Re: [Intel-gfx] [PATCH] drm/i915: Prevent recursive deadlock on releasing a busy userptr

2014-09-02 Thread Chris Wilson
On Tue, Sep 02, 2014 at 11:35:00AM +0200, Daniel Vetter wrote: > On Mon, Sep 01, 2014 at 02:23:17PM +0100, Tvrtko Ursulin wrote: > > > > On 08/07/2014 02:20 PM, Chris Wilson wrote: > > >During release of the GEM object we hold the struct_mutex. As the > > >object may be holding onto the last refer

Re: [Intel-gfx] [PATCH] drm/i915: Prevent recursive deadlock on releasing a busy userptr

2014-09-02 Thread Daniel Vetter
On Mon, Sep 01, 2014 at 02:23:17PM +0100, Tvrtko Ursulin wrote: > > On 08/07/2014 02:20 PM, Chris Wilson wrote: > >During release of the GEM object we hold the struct_mutex. As the > >object may be holding onto the last reference for the task->mm, > >calling mmput() may trigger exit_mmap() which c

[Intel-gfx] [PATCH] drm/i915: Prevent recursive deadlock on releasing a busy userptr

2014-09-02 Thread Chris Wilson
During release of the GEM object we hold the struct_mutex. As the object may be holding onto the last reference for the task->mm, calling mmput() may trigger exit_mmap() which close the vma which will call drm_gem_vm_close() and attempt to reacquire the struct_mutex. In order to avoid that recursio

Re: [Intel-gfx] [PATCH] drm/i915: Prevent recursive deadlock on releasing a busy userptr

2014-09-01 Thread Tvrtko Ursulin
On 08/07/2014 02:20 PM, Chris Wilson wrote: During release of the GEM object we hold the struct_mutex. As the object may be holding onto the last reference for the task->mm, calling mmput() may trigger exit_mmap() which close the vma which will call drm_gem_vm_close() and attempt to reacquire th

[Intel-gfx] [PATCH] drm/i915: Prevent recursive deadlock on releasing a busy userptr

2014-08-07 Thread Chris Wilson
During release of the GEM object we hold the struct_mutex. As the object may be holding onto the last reference for the task->mm, calling mmput() may trigger exit_mmap() which close the vma which will call drm_gem_vm_close() and attempt to reacquire the struct_mutex. In order to avoid that recursio