[AMD Public Use]

-----Original Message-----
From: Christian König <ckoenig.leichtzumer...@gmail.com> 
Sent: Tuesday, September 22, 2020 12:54 AM
To: Chauhan, Madhav <madhav.chau...@amd.com>; amd-gfx@lists.freedesktop.org
Cc: Surampalli, Kishore <kishore.surampa...@amd.com>; Patel, Mihir 
<mihir.pa...@amd.com>; Sharma, Shashank <shashank.sha...@amd.com>; Deucher, 
Alexander <alexander.deuc...@amd.com>; Saleem, Athar <athar.sal...@amd.com>
Subject: Re: [PATCH] drm/amdgpu: Add uid info to process BO list

Am 21.09.20 um 21:18 schrieb Madhav Chauhan:
> UID is helpful while doing analysis of BO allocated by a process.

Looks like a bit overkill to me, why not get the uid from the process info?

Not sure if I got your point , but used the similar method implemented at drm 
level inside drm_debugfs.c. Thanks

Regards,
Madhav

Christian.

>
> Signed-off-by: Madhav Chauhan <madhav.chau...@amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> index f4c2e2e75b8f..c1982349ec7b 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> @@ -892,6 +892,7 @@ static int amdgpu_debugfs_gem_info(struct seq_file *m, 
> void *data)
>       struct drm_info_node *node = (struct drm_info_node *)m->private;
>       struct drm_device *dev = node->minor->dev;
>       struct drm_file *file;
> +     kuid_t uid;
>       int r;
>   
>       r = mutex_lock_interruptible(&dev->filelist_mutex);
> @@ -909,7 +910,10 @@ static int amdgpu_debugfs_gem_info(struct seq_file *m, 
> void *data)
>                */
>               rcu_read_lock();
>               task = pid_task(file->pid, PIDTYPE_PID);
> -             seq_printf(m, "pid %8d command %s:\n", pid_nr(file->pid),
> +             uid = task ? __task_cred(task)->euid : GLOBAL_ROOT_UID;
> +             seq_printf(m, "pid %8d uid %5d command %s:\n",
> +                        pid_nr(file->pid),
> +                        from_kuid_munged(seq_user_ns(m), uid),
>                          task ? task->comm : "<unknown>");
>               rcu_read_unlock();
>   
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to