[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-10 Thread Daniel Vetter
On Thu, Aug 06, 2015 at 03:06:40PM +0200, Daniel Vetter wrote: > We want to make sure that no one tries to acquire more locks and > states, and ww mutexes provide debug facilities for that. So use them. > > v2: Only call acquire_done when ->atomic_check was successful to avoid > falling over an -E

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-06 Thread Daniel Vetter
On Thu, Aug 06, 2015 at 06:17:28AM +0200, Maarten Lankhorst wrote: > Op 06-08-15 om 00:25 schreef Daniel Vetter: > > On Wed, Aug 5, 2015 at 8:13 PM, Maarten Lankhorst > > wrote: > >> Op 05-08-15 om 17:03 schreef Daniel Vetter: > >>> On Wed, Aug 5, 2015 at 4:57 PM, Maarten Lankhorst > >>> wrote: >

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-06 Thread Daniel Vetter
We want to make sure that no one tries to acquire more locks and states, and ww mutexes provide debug facilities for that. So use them. v2: Only call acquire_done when ->atomic_check was successful to avoid falling over an -EDEADLK (spotted by Maarten). Cc: Rob Clark Cc: Maarten Lankhorst Signe

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-06 Thread Maarten Lankhorst
Op 06-08-15 om 00:25 schreef Daniel Vetter: > On Wed, Aug 5, 2015 at 8:13 PM, Maarten Lankhorst > wrote: >> Op 05-08-15 om 17:03 schreef Daniel Vetter: >>> On Wed, Aug 5, 2015 at 4:57 PM, Maarten Lankhorst >>> wrote: Op 05-08-15 om 15:08 schreef Daniel Vetter: > We want to make sure that

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-06 Thread Daniel Vetter
On Wed, Aug 5, 2015 at 8:13 PM, Maarten Lankhorst wrote: > Op 05-08-15 om 17:03 schreef Daniel Vetter: >> On Wed, Aug 5, 2015 at 4:57 PM, Maarten Lankhorst >> wrote: >>> Op 05-08-15 om 15:08 schreef Daniel Vetter: We want to make sure that no one tries to acquire more locks and states,

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-05 Thread Maarten Lankhorst
Op 05-08-15 om 17:03 schreef Daniel Vetter: > On Wed, Aug 5, 2015 at 4:57 PM, Maarten Lankhorst > wrote: >> Op 05-08-15 om 15:08 schreef Daniel Vetter: >>> We want to make sure that no one tries to acquire more locks and >>> states, and ww mutexes provide debug facilities for that. So use them. >>

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-05 Thread Daniel Vetter
On Wed, Aug 5, 2015 at 4:57 PM, Maarten Lankhorst wrote: > > Op 05-08-15 om 15:08 schreef Daniel Vetter: >> We want to make sure that no one tries to acquire more locks and >> states, and ww mutexes provide debug facilities for that. So use them. >> >> Cc: Rob Clark >> Cc: Maarten Lankhorst >> S

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-05 Thread Maarten Lankhorst
Hey, Op 05-08-15 om 15:08 schreef Daniel Vetter: > We want to make sure that no one tries to acquire more locks and > states, and ww mutexes provide debug facilities for that. So use them. > > Cc: Rob Clark > Cc: Maarten Lankhorst > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/drm_atom

[PATCH] drm/atomic: Call ww_acquire_done after check phase is complete

2015-08-05 Thread Daniel Vetter
We want to make sure that no one tries to acquire more locks and states, and ww mutexes provide debug facilities for that. So use them. Cc: Rob Clark Cc: Maarten Lankhorst Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_atomic.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/driver