drm/i915: Fix retrieval of hangcheck stats
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 16 Jun 2017 13:28:49 +0000 (14:28 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Fri, 16 Jun 2017 15:22:43 +0000 (16:22 +0100)
The default context is always supported (as it contains the global
hangcheck stats) and the contexts for hangcheck are not limited
to any ring.

This was dropped in 2013 because it was supposed to have been included
with Ben's full-ppgtt patch set. It never landed and the bug remains.

References: https://bugs.freedesktop.org/show_bug.cgi?id=65845
Link: http://patchwork.freedesktop.org/patch/msgid/1372175222-27622-1-git-send-email-mika.kuoppala@intel.com
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170616132849.29597-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_gem_context.c

index c5d1666d7071271335b6c614b05ed75b5412651c..81c73dee095f503f05a25f5778ee9e5fc8d5f2bc 100644 (file)
@@ -1034,9 +1034,6 @@ int i915_gem_context_reset_stats_ioctl(struct drm_device *dev,
        if (args->flags || args->pad)
                return -EINVAL;
 
-       if (args->ctx_id == DEFAULT_CONTEXT_HANDLE && !capable(CAP_SYS_ADMIN))
-               return -EPERM;
-
        ret = i915_mutex_lock_interruptible(dev);
        if (ret)
                return ret;