Chris Wilson <[email protected]> writes: > Since we only count hangs towards the owner of the fd issuing the > command, we can allow that fd to inspect its own default context without > leaking global information. We introduced per-fd accounting with > > commit 0eea67eb26000657079b7fc41079097942339452 > Author: Ben Widawsky <[email protected]> > Date: Fri Dec 6 14:11:19 2013 -0800 > > drm/i915: Create a per file_priv default context > > Signed-off-by: Chris Wilson <[email protected]> > Cc: Ben Widawsky <[email protected]> > Cc: Mika Kuoppala <[email protected]> > Cc: Kenneth Graunke <[email protected]>
I think we need ack from Kenneth too. I will fix testcase to not expect EPERM for nonroot. Reviewed-by: Mika Kuoppala <[email protected]> > --- > drivers/gpu/drm/i915/intel_uncore.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_uncore.c > b/drivers/gpu/drm/i915/intel_uncore.c > index 2adf6aa..f8f13ae 100644 > --- a/drivers/gpu/drm/i915/intel_uncore.c > +++ b/drivers/gpu/drm/i915/intel_uncore.c > @@ -930,9 +930,6 @@ int i915_get_reset_stats_ioctl(struct drm_device *dev, > if (args->flags || args->pad) > return -EINVAL; > > - if (args->ctx_id == DEFAULT_CONTEXT_ID && !capable(CAP_SYS_ADMIN)) > - return -EPERM; > - > ret = mutex_lock_interruptible(&dev->struct_mutex); > if (ret) > return ret; > -- > 1.7.9.5 _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
