[PATCH] drm/virtio: use new debugfs device-centered functions
Dmitry Osipenko
dmitry.osipenko at collabora.com
Mon Aug 11 10:56:02 UTC 2025
On 8/9/25 15:54, Brahmajit Das wrote:
> Replace the use of drm_debugfs_create_files() with the new
> drm_debugfs_add_files() function, which centers the debugfs files
> management on the drm_device instead of drm_minor.
>
> Signed-off-by: Brahmajit Das <listout at listout.xyz>
> ---
> drivers/gpu/drm/virtio/virtgpu_debugfs.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/virtio/virtgpu_debugfs.c b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
> index 853dd9aa397e..b7c895fe5ddc 100644
> --- a/drivers/gpu/drm/virtio/virtgpu_debugfs.c
> +++ b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
> @@ -94,7 +94,7 @@ virtio_gpu_debugfs_host_visible_mm(struct seq_file *m, void *data)
> return 0;
> }
>
> -static struct drm_info_list virtio_gpu_debugfs_list[] = {
> +static struct drm_debugfs_info virtio_gpu_debugfs_list[] = {
> { "virtio-gpu-features", virtio_gpu_features },
> { "virtio-gpu-irq-fence", virtio_gpu_debugfs_irq_info, 0, NULL },
> { "virtio-gpu-host-visible-mm", virtio_gpu_debugfs_host_visible_mm },
> @@ -105,7 +105,6 @@ static struct drm_info_list virtio_gpu_debugfs_list[] = {
> void
> virtio_gpu_debugfs_init(struct drm_minor *minor)
> {
> - drm_debugfs_create_files(virtio_gpu_debugfs_list,
> - VIRTIO_GPU_DEBUGFS_ENTRIES,
> - minor->debugfs_root, minor);
> + drm_debugfs_add_files(minor->dev, virtio_gpu_debugfs_list,
> + VIRTIO_GPU_DEBUGFS_ENTRIES);
> }
drm_debugfs_create_files() uses drm_info_node, while
drm_debugfs_add_files() uses drm_debugfs_entry. This patch looks wrong
and, AFAICT, doesn't improve anything. Hence, let's keep this code as-is
if there is no real benefit from the change.
--
Best regards,
Dmitry
More information about the dri-devel
mailing list