[PATCH v5 0/6] DRM_SET_CLIENT_NAME ioctl

[Deucher, Alexander]

[Public]

> -----Original Message-----
> From: Koenig, Christian <Christian.Koenig at amd.com>
> Sent: Tuesday, October 8, 2024 4:27 AM
> To: Deucher, Alexander <Alexander.Deucher at amd.com>
> Cc: robdclark at gmail.com; amd-gfx at lists.freedesktop.org; Pelloux-Prayer, Pierre-
> Eric <Pierre-eric.Pelloux-prayer at amd.com>; dri-devel at lists.freedesktop.org;
> dmitry.osipenko at collabora.com; jani.nikula at linux.intel.com; tursulin at igalia.com;
> simona.vetter at ffwll.ch
> Subject: Re: [PATCH v5 0/6] DRM_SET_CLIENT_NAME ioctl
>
> Hi guys,
>
> I've pushed the first two patches to drm-misc-next.
>
> @Alex any objections to merge the amdgpu changes through drm-misc-next as
> well?

That's fine with me.

Alex

>
> Thanks,
> Christian.
>
> Am 03.10.24 um 14:43 schrieb Pierre-Eric Pelloux-Prayer:
> > v5 of this series which is adding a new ioctl to let userspace
> > associate a free-form name to a drm_file.
> >
> > Having this name is useful when debugging or tracking resource usage;
> > in particular when using virtio native-context driver, where a single
> > process (say QEMU), uses 1 drm_file per-guest application.
> > With this change, fdinfo and other tools can map each drm_file
> > activity to the guest application.
> >
> > virglrenderer MR using this new ioctl to associate the context "debug_name"
> > to the fd:
> > https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/14
> > 28
> >
> > Changes since v4:
> > * patches 1, 4 and 6 were slightly reworked based on Tvrtko' suggestions.
> >
> > v4:
> > https://lists.freedesktop.org/archives/dri-devel/2024-September/471695
> > .html
> >
> >
> > Pierre-Eric Pelloux-Prayer (6):
> >    drm: add DRM_SET_CLIENT_NAME ioctl
> >    drm: use drm_file client_name in fdinfo
> >    drm/amdgpu: delay the use of amdgpu_vm_set_task_info
> >    drm/amdgpu: alloc and init vm::task_info from first submit
> >    drm/amdgpu: make process_name a flexible array
> >    drm/amdgpu: use drm_file::name in task_info::process_desc
> >
> >   Documentation/gpu/drm-usage-stats.rst         |  5 ++
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h    |  1 +
> >   .../gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c  |  3 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c        |  6 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c   |  2 +-
> >   .../gpu/drm/amd/amdgpu/amdgpu_dev_coredump.c  |  4 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_job.c       |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c        | 67 +++++++++++++------
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h        |  4 +-
> >   drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c        |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/gmc_v11_0.c        |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c        |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c         |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c         |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c        |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/sdma_v4_4_2.c      |  2 +-
> >   drivers/gpu/drm/amd/amdkfd/kfd_events.c       |  2 +-
> >   drivers/gpu/drm/amd/amdkfd/kfd_process.c      |  3 +
> >   drivers/gpu/drm/drm_debugfs.c                 | 14 ++--
> >   drivers/gpu/drm/drm_file.c                    | 10 +++
> >   drivers/gpu/drm/drm_ioctl.c                   | 51 ++++++++++++++
> >   include/drm/drm_file.h                        |  9 +++
> >   include/uapi/drm/drm.h                        | 17 +++++
> >   23 files changed, 172 insertions(+), 42 deletions(-)
> >