drm/i915: Hold rpm wakeref for modifying the global seqno
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 2 Jan 2018 15:12:31 +0000 (15:12 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Wed, 3 Jan 2018 11:42:33 +0000 (11:42 +0000)
To modify the global seqno may require rewriting a few registers, which
requires us to hold the rpm wakeref. We must therefore take it around
the call to i915_gem_set_global_seqno() in debugfs, on behalf of the
user.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: MichaƂ Winiarski <michal.winiarski@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180102151235.3949-15-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_debugfs.c

index d81cb2513069348f9858222b031673f1f0c2aea6..2bb63073d73fd25ccd41f48122dc8a2cdf22bd42 100644 (file)
@@ -988,7 +988,10 @@ i915_next_seqno_set(void *data, u64 val)
        if (ret)
                return ret;
 
+       intel_runtime_pm_get(dev_priv);
        ret = i915_gem_set_global_seqno(dev, val);
+       intel_runtime_pm_put(dev_priv);
+
        mutex_unlock(&dev->struct_mutex);
 
        return ret;