drm/i915: cancel RPS work before disabling RPS
authorJesse Barnes <jbarnes@virtuousgeek.org>
Tue, 23 Apr 2013 17:09:27 +0000 (10:09 -0700)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 24 Apr 2013 09:09:13 +0000 (11:09 +0200)
Ville noticed this while doing another review; we may as well cancel
this work just to make sure we don't try anything fancy after disabling
the RPS interfaces.

Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_pm.c

index 93b01e197f42940d60af9407b23a303c116546b0..72ad817b6bf4401e27e5e8c10c1ab653f2d9a310 100644 (file)
@@ -3657,6 +3657,7 @@ void intel_disable_gt_powersave(struct drm_device *dev)
                ironlake_disable_rc6(dev);
        } else if (INTEL_INFO(dev)->gen >= 6) {
                cancel_delayed_work_sync(&dev_priv->rps.delayed_resume_work);
+               cancel_work_sync(&dev_priv->rps.work);
                if (IS_VALLEYVIEW(dev))
                        cancel_delayed_work_sync(&dev_priv->rps.vlv_work);
                mutex_lock(&dev_priv->rps.hw_lock);