Re: [Intel-gfx] [PATCH] drm: Paper over locking inversion after registration rework

2016-08-05 Thread Daniel Vetter
On Fri, Aug 05, 2016 at 10:57:58AM +0200, Jiri Kosina wrote: > On Thu, 4 Aug 2016, Daniel Vetter wrote: > > > drm_connector_register_all requires a few too many locks because our > > connector_list locking is busted. Add another FIXME+hack to work > > around this. This should address the below loc

Re: [Intel-gfx] [PATCH] drm: Paper over locking inversion after registration rework

2016-08-05 Thread Jiri Kosina
On Thu, 4 Aug 2016, Daniel Vetter wrote: > drm_connector_register_all requires a few too many locks because our > connector_list locking is busted. Add another FIXME+hack to work > around this. This should address the below lockdep splat: [ ... snip ... ] > v2: Rebase onto the right branch (hand-e

Re: [Intel-gfx] [PATCH] drm: Paper over locking inversion after registration rework

2016-08-05 Thread Jani Nikula
On Thu, 04 Aug 2016, Daniel Vetter wrote: > drm_connector_register_all requires a few too many locks because our > connector_list locking is busted. Add another FIXME+hack to work > around this. This should address the below lockdep splat: Jiri, a Tested-by on this would be appreciated. BR, Jani

[Intel-gfx] [PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Daniel Vetter
drm_connector_register_all requires a few too many locks because our connector_list locking is busted. Add another FIXME+hack to work around this. This should address the below lockdep splat: == [ INFO: possible circular locking dependency detect

Re: [Intel-gfx] [PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Chris Wilson
On Thu, Aug 04, 2016 at 12:15:14PM +0200, Daniel Vetter wrote: > drm_connector_register_all requires a few too many locks because our > connector_list locking is busted. Add another FIXME+hack to work > around this. This should address the below lockdep splat: > > Cc: Imre Deak > Cc: Chris Wilson

Re: [Intel-gfx] [PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Jani Nikula
On Thu, 04 Aug 2016, Daniel Vetter wrote: > drm_connector_register_all requires a few too many locks because our > connector_list locking is busted. Add another FIXME+hack to work > around this. This should address the below lockdep splat: > > ==

[Intel-gfx] [PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Daniel Vetter
drm_connector_register_all requires a few too many locks because our connector_list locking is busted. Add another FIXME+hack to work around this. This should address the below lockdep splat: == [ INFO: possible circular locking dependency detect