| Message ID | 20240916133223.1023773-2-pierre-eric.pelloux-prayer@amd.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/3] drm: add DRM_SET_NAME ioctl | expand |
Am 16.09.24 um 15:32 schrieb Pierre-Eric Pelloux-Prayer: > Add an optional drm-client-name field to drm fdinfo's output. > > Signed-off-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Reviewed-by: Christian König <christian.koenig@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); > }
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@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@igalia.com> Regards, Tvrtko
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); }
Add an optional drm-client-name field to drm fdinfo's output. Signed-off-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> --- Documentation/gpu/drm-usage-stats.rst | 5 +++++ drivers/gpu/drm/drm_file.c | 5 +++++ 2 files changed, 10 insertions(+)