[PATCH v2 2/3] drm: use drm_file name in fdinfo
Tvrtko Ursulin
tvrtko.ursulin at igalia.com
Mon Sep 16 14:36:31 UTC 2024
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
More information about the dri-devel
mailing list