[PATCH v2 2/3] drm: use drm_file name in fdinfo

[Tvrtko Ursulin]

On 16/09/2024 14:32, Pierre-Eric Pelloux-Prayer wrote:
> Add an optional drm-client-name field to drm fdinfo's output.
> 
> Signed-off-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer at amd.com>
> ---
>   Documentation/gpu/drm-usage-stats.rst | 5 +++++
>   drivers/gpu/drm/drm_file.c            | 5 +++++
>   2 files changed, 10 insertions(+)
> 
> diff --git a/Documentation/gpu/drm-usage-stats.rst b/Documentation/gpu/drm-usage-stats.rst
> index a80f95ca1b2f..ed1d7edbbc5f 100644
> --- a/Documentation/gpu/drm-usage-stats.rst
> +++ b/Documentation/gpu/drm-usage-stats.rst
> @@ -73,6 +73,11 @@ scope of each device, in which case `drm-pdev` shall be present as well.
>   Userspace should make sure to not double account any usage statistics by using
>   the above described criteria in order to associate data to individual clients.
>   
> +- drm-client-name: <valstr>
> +
> +String optionally set by userspace using DRM_IOCTL_SET_NAME.
> +
> +
>   Utilization
>   ^^^^^^^^^^^
>   
> diff --git a/drivers/gpu/drm/drm_file.c b/drivers/gpu/drm/drm_file.c
> index e9dd0e90a1f9..6a3621f50784 100644
> --- a/drivers/gpu/drm/drm_file.c
> +++ b/drivers/gpu/drm/drm_file.c
> @@ -955,6 +955,11 @@ void drm_show_fdinfo(struct seq_file *m, struct file *f)
>   			   PCI_SLOT(pdev->devfn), PCI_FUNC(pdev->devfn));
>   	}
>   
> +	mutex_lock(&file->name_lock);
> +	if (file->name)
> +		drm_printf(&p, "drm-client-name:\t%s\n", file->name);
> +	mutex_unlock(&file->name_lock);
> +
>   	if (dev->driver->show_fdinfo)
>   		dev->driver->show_fdinfo(&p, file);
>   }

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at igalia.com>

Regards,

Tvrtko