drm/i915: move getting struct_mutex lower in the callstack during GPU reset
authorImre Deak <imre.deak@intel.com>
Mon, 14 Apr 2014 17:24:28 +0000 (20:24 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 5 May 2014 07:08:56 +0000 (09:08 +0200)
Getting struct_mutex around the whole intel_enable_gt_powersave()
function is not necessary, since it's only needed for the ILK path
therein.

This will make intel_enable_gt_powersave() useable on the RPM resume
path for >=GEN6 (added in an upcoming patch to reset the RPS state
during RPM resume), where we can't (and need not) get this mutex.

Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_drv.c
drivers/gpu/drm/i915/intel_pm.c

index 743cddec9a18a8560e59aa463d96069f2bd33896..5cbd79e369c4545c28569e400f3120ad417e6f21 100644 (file)
@@ -781,11 +781,8 @@ int i915_reset(struct drm_device *dev)
                 * reset and the re-install of drm irq. Skip for ironlake per
                 * previous concerns that it doesn't respond well to some forms
                 * of re-init after reset. */
-               if (INTEL_INFO(dev)->gen > 5) {
-                       mutex_lock(&dev->struct_mutex);
+               if (INTEL_INFO(dev)->gen > 5)
                        intel_enable_gt_powersave(dev);
-                       mutex_unlock(&dev->struct_mutex);
-               }
 
                intel_hpd_init(dev);
        } else {
index 4ebb93c4e1107ac4330bf3653e6521373305d9f2..e5b9f08d109830e68409f1894b995539e56f01d4 100644 (file)
@@ -4556,9 +4556,11 @@ void intel_enable_gt_powersave(struct drm_device *dev)
        struct drm_i915_private *dev_priv = dev->dev_private;
 
        if (IS_IRONLAKE_M(dev)) {
+               mutex_lock(&dev->struct_mutex);
                ironlake_enable_drps(dev);
                ironlake_enable_rc6(dev);
                intel_init_emon(dev);
+               mutex_unlock(&dev->struct_mutex);
        } else if (IS_GEN6(dev) || IS_GEN7(dev)) {
                /*
                 * PCU communication is slow and this doesn't need to be