[PATCH 1/2] drm: fix order of event_lock wrt. vblank_time_clock

2012-11-01 Thread Imre Deak
On Thu, 2012-11-01 at 00:03 +0200, Imre Deak wrote: > drm_vblank_off() requires callers to hold the event_lock, while itself > locking vbl_time and vblank_time_lock. This defines a dependency chain > that conflicts with the one in drm_handle_vblank() where we first lock > vblank_time_lock and then

Re: [PATCH 1/2] drm: fix order of event_lock wrt. vblank_time_clock

2012-11-01 Thread Imre Deak
On Thu, 2012-11-01 at 00:03 +0200, Imre Deak wrote: > drm_vblank_off() requires callers to hold the event_lock, while itself > locking vbl_time and vblank_time_lock. This defines a dependency chain > that conflicts with the one in drm_handle_vblank() where we first lock > vblank_time_lock and then

[PATCH 1/2] drm: fix order of event_lock wrt. vblank_time_clock

2012-11-01 Thread Imre Deak
drm_vblank_off() requires callers to hold the event_lock, while itself locking vbl_time and vblank_time_lock. This defines a dependency chain that conflicts with the one in drm_handle_vblank() where we first lock vblank_time_lock and then the event_lock. Fix this by reversing the locking order in d

[PATCH 1/2] drm: fix order of event_lock wrt. vblank_time_clock

2012-10-31 Thread Imre Deak
drm_vblank_off() requires callers to hold the event_lock, while itself locking vbl_time and vblank_time_lock. This defines a dependency chain that conflicts with the one in drm_handle_vblank() where we first lock vblank_time_lock and then the event_lock. Fix this by reversing the locking order in d