[PATCH] drm/nouveau: add lockdep annotations

2013-02-07 Thread Maarten Lankhorst
Hey, Op 05-02-13 21:52, Ben Skeggs schreef: > On Mon, Feb 04, 2013 at 10:59:28PM +0100, Maarten Lankhorst wrote: >> Op 04-02-13 22:30, Marcin Slusarz schreef: >>> 1) Lockdep thinks all nouveau subdevs belong to the same class and can be >>> locked in arbitrary order, which is not true (at least in

Re: [PATCH] drm/nouveau: add lockdep annotations

2013-02-07 Thread Maarten Lankhorst
Hey, Op 05-02-13 21:52, Ben Skeggs schreef: > On Mon, Feb 04, 2013 at 10:59:28PM +0100, Maarten Lankhorst wrote: >> Op 04-02-13 22:30, Marcin Slusarz schreef: >>> 1) Lockdep thinks all nouveau subdevs belong to the same class and can be >>> locked in arbitrary order, which is not true (at least in

[PATCH] drm/nouveau: add lockdep annotations

2013-02-06 Thread Ben Skeggs
On Mon, Feb 04, 2013 at 10:59:28PM +0100, Maarten Lankhorst wrote: > Op 04-02-13 22:30, Marcin Slusarz schreef: > > 1) Lockdep thinks all nouveau subdevs belong to the same class and can be > > locked in arbitrary order, which is not true (at least in general case). > > Tell it to distinguish subde

Re: [PATCH] drm/nouveau: add lockdep annotations

2013-02-05 Thread Ben Skeggs
On Mon, Feb 04, 2013 at 10:59:28PM +0100, Maarten Lankhorst wrote: > Op 04-02-13 22:30, Marcin Slusarz schreef: > > 1) Lockdep thinks all nouveau subdevs belong to the same class and can be > > locked in arbitrary order, which is not true (at least in general case). > > Tell it to distinguish subde

Re: [PATCH] drm/nouveau: add lockdep annotations

2013-02-04 Thread Peter Hurley
Hi Maarten On Mon, 2013-02-04 at 22:59 +0100, Maarten Lankhorst wrote: > Op 04-02-13 22:30, Marcin Slusarz schreef: > > 1) Lockdep thinks all nouveau subdevs belong to the same class and can be > > locked in arbitrary order, which is not true (at least in general case). > > Tell it to distinguish

[PATCH] drm/nouveau: add lockdep annotations

2013-02-04 Thread Maarten Lankhorst
Op 04-02-13 22:30, Marcin Slusarz schreef: > 1) Lockdep thinks all nouveau subdevs belong to the same class and can be > locked in arbitrary order, which is not true (at least in general case). > Tell it to distinguish subdevs by (o)class type. Apart from this specific case, is there any other reas

[PATCH] drm/nouveau: add lockdep annotations

2013-02-04 Thread Marcin Slusarz
1) Lockdep thinks all nouveau subdevs belong to the same class and can be locked in arbitrary order, which is not true (at least in general case). Tell it to distinguish subdevs by (o)class type. 2) DRM client can be locked under user client lock - tell lockdep to put DRM client lock in a separate

[PATCH] drm/nouveau: add lockdep annotations

2013-02-04 Thread Peter Hurley
Hi Maarten On Mon, 2013-02-04 at 22:59 +0100, Maarten Lankhorst wrote: > Op 04-02-13 22:30, Marcin Slusarz schreef: > > 1) Lockdep thinks all nouveau subdevs belong to the same class and can be > > locked in arbitrary order, which is not true (at least in general case). > > Tell it to distinguish

Re: [PATCH] drm/nouveau: add lockdep annotations

2013-02-04 Thread Maarten Lankhorst
Op 04-02-13 22:30, Marcin Slusarz schreef: > 1) Lockdep thinks all nouveau subdevs belong to the same class and can be > locked in arbitrary order, which is not true (at least in general case). > Tell it to distinguish subdevs by (o)class type. Apart from this specific case, is there any other reas

[PATCH] drm/nouveau: add lockdep annotations

2013-02-04 Thread Marcin Slusarz
1) Lockdep thinks all nouveau subdevs belong to the same class and can be locked in arbitrary order, which is not true (at least in general case). Tell it to distinguish subdevs by (o)class type. 2) DRM client can be locked under user client lock - tell lockdep to put DRM client lock in a separate