Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-27 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: > amdgpu gained dev->struct_mutex usage, and that's because it's walking > the dev->filelist list. Protect that list with it's own lock to take > one more step towards getting rid of struct_mutex usage in drivers > once and for all. >

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-27 Thread Chris Wilson
On Wed, Apr 27, 2016 at 10:01:16AM +0200, Daniel Vetter wrote: > On Wed, Apr 27, 2016 at 08:17:08AM +0100, Chris Wilson wrote: > > drm/i915: s/struct_mutex/bfg9000/ > > Can't do that yet because holding struct_mutex prevents objects from > disappearing, speficially all our mm and lru lists only

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-27 Thread Daniel Vetter
On Wed, Apr 27, 2016 at 08:17:08AM +0100, Chris Wilson wrote: > On Wed, Apr 27, 2016 at 09:06:09AM +0200, Daniel Vetter wrote: > > On Tue, Apr 26, 2016 at 05:45:44PM -0400, Alex Deucher wrote: > > > On Tue, Apr 26, 2016 at 4:52 PM, Chris Wilson > > > wrote: > > > > On

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-27 Thread Deucher, Alexander
niel Vetter > Subject: Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own > mutex > > On Wed, Apr 27, 2016 at 09:06:09AM +0200, Daniel Vetter wrote: > > On Tue, Apr 26, 2016 at 05:45:44PM -0400, Alex Deucher wrote: > > > On Tue, Apr 26, 2016 at 4:52

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-27 Thread Chris Wilson
On Wed, Apr 27, 2016 at 09:06:09AM +0200, Daniel Vetter wrote: > On Tue, Apr 26, 2016 at 05:45:44PM -0400, Alex Deucher wrote: > > On Tue, Apr 26, 2016 at 4:52 PM, Chris Wilson > > wrote: > > > On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: > > >> amdgpu

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-27 Thread Daniel Vetter
On Wed, Apr 27, 2016 at 09:06:09AM +0200, Daniel Vetter wrote: > On Tue, Apr 26, 2016 at 05:45:44PM -0400, Alex Deucher wrote: > > On Tue, Apr 26, 2016 at 4:52 PM, Chris Wilson > > wrote: > > > On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: > > >> amdgpu

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-27 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 05:45:44PM -0400, Alex Deucher wrote: > On Tue, Apr 26, 2016 at 4:52 PM, Chris Wilson > wrote: > > On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: > >> amdgpu gained dev->struct_mutex usage, and that's because it's walking > >> the

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 4:52 PM, Chris Wilson wrote: > On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: >> amdgpu gained dev->struct_mutex usage, and that's because it's walking >> the dev->filelist list. Protect that list with it's own lock to take >> one

Re: [Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: > amdgpu gained dev->struct_mutex usage, and that's because it's walking > the dev->filelist list. Protect that list with it's own lock to take > one more step towards getting rid of struct_mutex usage in drivers > once and for all. >

[Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-26 Thread Daniel Vetter
amdgpu gained dev->struct_mutex usage, and that's because it's walking the dev->filelist list. Protect that list with it's own lock to take one more step towards getting rid of struct_mutex usage in drivers once and for all. While doing the conversion I noticed that 2 debugfs files in i915