Re: [Intel-gfx] [PATCH] drm/i915: fix locking around punit access in cur_delayinfo for VLV

2013-04-23 Thread Jani Nikula
On Tue, 23 Apr 2013, Jesse Barnes jbar...@virtuousgeek.org wrote: We need to hold the rps lock around punit access. Reviewed-by: Jani Nikula jani.nik...@intel.com And a semi-related question while at it... we will need punit access also for non-rps stuff. Shall we just bundle them under the

Re: [Intel-gfx] [PATCH] drm/i915: fix locking around punit access in cur_delayinfo for VLV

2013-04-23 Thread Jesse Barnes
On Tue, 23 Apr 2013 10:51:28 +0300 Jani Nikula jani.nik...@linux.intel.com wrote: On Tue, 23 Apr 2013, Jesse Barnes jbar...@virtuousgeek.org wrote: We need to hold the rps lock around punit access. Reviewed-by: Jani Nikula jani.nik...@intel.com And a semi-related question while at it...

[Intel-gfx] [PATCH] drm/i915: fix locking around punit access in cur_delayinfo for VLV

2013-04-22 Thread Jesse Barnes
We need to hold the rps lock around punit access. Reported-by: Kenneth Graunke kenn...@whitecape.org Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org --- drivers/gpu/drm/i915/i915_debugfs.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_debugfs.c