[PATCH 2/8] drm: remove redundant drm_file->uid

2016-08-04 Thread David Herrmann
Hey On Wed, Aug 3, 2016 at 9:01 PM, Chris Wilson wrote: > On Wed, Aug 03, 2016 at 08:04:26PM +0200, David Herrmann wrote: >> @@ -98,13 +99,14 @@ int drm_clients_info(struct seq_file *m, void *data) >> >> rcu_read_lock(); /* locks pid_task()->comm */ >> task = pid_task

[PATCH 2/8] drm: remove redundant drm_file->uid

2016-08-03 Thread David Herrmann
Each DRM file-context caches the EUID of the process that opened the file. It is used exclusively for debugging purposes in /proc/dri/ and friends. Note, however, that "struct file" already caches the credentials of a process at open-time. So lets just use drm_file->filp->f_cred->euid if available

[PATCH 2/8] drm: remove redundant drm_file->uid

2016-08-03 Thread Chris Wilson
On Wed, Aug 03, 2016 at 08:04:26PM +0200, David Herrmann wrote: > @@ -98,13 +99,14 @@ int drm_clients_info(struct seq_file *m, void *data) > > rcu_read_lock(); /* locks pid_task()->comm */ > task = pid_task(priv->pid, PIDTYPE_PID); > + uid = priv->filp ? pr

[PATCH 2/8] drm: remove redundant drm_file->uid

2016-08-03 Thread Chris Wilson
On Wed, Aug 03, 2016 at 08:04:26PM +0200, David Herrmann wrote: > diff --git a/drivers/gpu/drm/drm_fops.c b/drivers/gpu/drm/drm_fops.c > index 323c238..e9d66f5 100644 > --- a/drivers/gpu/drm/drm_fops.c > +++ b/drivers/gpu/drm/drm_fops.c > @@ -199,7 +199,6 @@ static int drm_open_helper(struct file *