[PATCH 2/2] drm/i915: lock event_lock for drm_vblank_off()

2012-11-01 Thread Imre Deak
On Wed, 2012-10-31 at 23:46 +0100, Daniel Vetter wrote: > On Thu, Nov 01, 2012 at 12:03:39AM +0200, Imre Deak wrote: > > drm_vblank_off() requires callers to hold the event_lock. > > > > Signed-off-by: Imre Deak > > Hm, do we put this code through its paces already in flip_test by > scheduling

[PATCH 2/2] drm/i915: lock event_lock for drm_vblank_off()

2012-11-01 Thread Imre Deak
drm_vblank_off() requires callers to hold the event_lock. Signed-off-by: Imre Deak --- drivers/gpu/drm/i915/intel_display.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index b453901..56f1119

[PATCH 2/2] drm/i915: lock event_lock for drm_vblank_off()

2012-11-01 Thread Daniel Vetter
On Thu, Nov 01, 2012 at 12:03:39AM +0200, Imre Deak wrote: > drm_vblank_off() requires callers to hold the event_lock. > > Signed-off-by: Imre Deak Hm, do we put this code through its paces already in flip_test by scheduling a vblank wait in the future (a second or so), and then disabling the

Re: [PATCH 2/2] drm/i915: lock event_lock for drm_vblank_off()

2012-11-01 Thread Imre Deak
On Wed, 2012-10-31 at 23:46 +0100, Daniel Vetter wrote: On Thu, Nov 01, 2012 at 12:03:39AM +0200, Imre Deak wrote: drm_vblank_off() requires callers to hold the event_lock. Signed-off-by: Imre Deak imre.d...@intel.com Hm, do we put this code through its paces already in flip_test by

[PATCH 2/2] drm/i915: lock event_lock for drm_vblank_off()

2012-10-31 Thread Imre Deak
drm_vblank_off() requires callers to hold the event_lock. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/intel_display.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c

Re: [PATCH 2/2] drm/i915: lock event_lock for drm_vblank_off()

2012-10-31 Thread Daniel Vetter
On Thu, Nov 01, 2012 at 12:03:39AM +0200, Imre Deak wrote: drm_vblank_off() requires callers to hold the event_lock. Signed-off-by: Imre Deak imre.d...@intel.com Hm, do we put this code through its paces already in flip_test by scheduling a vblank wait in the future (a second or so), and then