From: Zack Rusin <za...@vmware.com>

Switch to using the TTM resource manager debugfs helpers. It's
exactly the same functionality but the debugfs code is shared with
other drivers.

The TTM resource managers need to stay valid for as long as the
drm debugfs_root is valid.

Signed-off-by: Zack Rusin <za...@vmware.com>
Cc: Alex Deucher <alexander.deuc...@amd.com>
Cc: "Christian König" <christian.koe...@amd.com>
Cc: "Pan, Xinhui" <xinhui....@amd.com>
Cc: David Airlie <airl...@linux.ie>
Cc: Daniel Vetter <dan...@ffwll.ch>
Cc: Felix Kuehling <felix.kuehl...@amd.com>
Cc: Nirmoy Das <nirmoy....@amd.com>
Cc: Thomas Zimmermann <tzimmerm...@suse.de>
Cc: amd-...@lists.freedesktop.org
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 86 +++++--------------------
 1 file changed, 16 insertions(+), 70 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index 57ac118fc266..37fe9fb3f8e5 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -2079,17 +2079,6 @@ int amdgpu_ttm_evict_resources(struct amdgpu_device 
*adev, int mem_type)
 
 #if defined(CONFIG_DEBUG_FS)
 
-static int amdgpu_mm_vram_table_show(struct seq_file *m, void *unused)
-{
-       struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
-       struct ttm_resource_manager *man = ttm_manager_type(&adev->mman.bdev,
-                                                           TTM_PL_VRAM);
-       struct drm_printer p = drm_seq_file_printer(m);
-
-       ttm_resource_manager_debug(man, &p);
-       return 0;
-}
-
 static int amdgpu_ttm_page_pool_show(struct seq_file *m, void *unused)
 {
        struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
@@ -2097,55 +2086,6 @@ static int amdgpu_ttm_page_pool_show(struct seq_file *m, 
void *unused)
        return ttm_pool_debugfs(&adev->mman.bdev.pool, m);
 }
 
-static int amdgpu_mm_tt_table_show(struct seq_file *m, void *unused)
-{
-       struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
-       struct ttm_resource_manager *man = ttm_manager_type(&adev->mman.bdev,
-                                                           TTM_PL_TT);
-       struct drm_printer p = drm_seq_file_printer(m);
-
-       ttm_resource_manager_debug(man, &p);
-       return 0;
-}
-
-static int amdgpu_mm_gds_table_show(struct seq_file *m, void *unused)
-{
-       struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
-       struct ttm_resource_manager *man = ttm_manager_type(&adev->mman.bdev,
-                                                           AMDGPU_PL_GDS);
-       struct drm_printer p = drm_seq_file_printer(m);
-
-       ttm_resource_manager_debug(man, &p);
-       return 0;
-}
-
-static int amdgpu_mm_gws_table_show(struct seq_file *m, void *unused)
-{
-       struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
-       struct ttm_resource_manager *man = ttm_manager_type(&adev->mman.bdev,
-                                                           AMDGPU_PL_GWS);
-       struct drm_printer p = drm_seq_file_printer(m);
-
-       ttm_resource_manager_debug(man, &p);
-       return 0;
-}
-
-static int amdgpu_mm_oa_table_show(struct seq_file *m, void *unused)
-{
-       struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
-       struct ttm_resource_manager *man = ttm_manager_type(&adev->mman.bdev,
-                                                           AMDGPU_PL_OA);
-       struct drm_printer p = drm_seq_file_printer(m);
-
-       ttm_resource_manager_debug(man, &p);
-       return 0;
-}
-
-DEFINE_SHOW_ATTRIBUTE(amdgpu_mm_vram_table);
-DEFINE_SHOW_ATTRIBUTE(amdgpu_mm_tt_table);
-DEFINE_SHOW_ATTRIBUTE(amdgpu_mm_gds_table);
-DEFINE_SHOW_ATTRIBUTE(amdgpu_mm_gws_table);
-DEFINE_SHOW_ATTRIBUTE(amdgpu_mm_oa_table);
 DEFINE_SHOW_ATTRIBUTE(amdgpu_ttm_page_pool);
 
 /*
@@ -2355,17 +2295,23 @@ void amdgpu_ttm_debugfs_init(struct amdgpu_device *adev)
                                 &amdgpu_ttm_vram_fops, adev->gmc.mc_vram_size);
        debugfs_create_file("amdgpu_iomem", 0444, root, adev,
                            &amdgpu_ttm_iomem_fops);
-       debugfs_create_file("amdgpu_vram_mm", 0444, root, adev,
-                           &amdgpu_mm_vram_table_fops);
-       debugfs_create_file("amdgpu_gtt_mm", 0444, root, adev,
-                           &amdgpu_mm_tt_table_fops);
-       debugfs_create_file("amdgpu_gds_mm", 0444, root, adev,
-                           &amdgpu_mm_gds_table_fops);
-       debugfs_create_file("amdgpu_gws_mm", 0444, root, adev,
-                           &amdgpu_mm_gws_table_fops);
-       debugfs_create_file("amdgpu_oa_mm", 0444, root, adev,
-                           &amdgpu_mm_oa_table_fops);
        debugfs_create_file("ttm_page_pool", 0444, root, adev,
                            &amdgpu_ttm_page_pool_fops);
+       ttm_resource_manager_create_debugfs(ttm_manager_type(&adev->mman.bdev,
+                                                            TTM_PL_VRAM),
+                                           root, "amdgpu_vram_mm");
+       ttm_resource_manager_create_debugfs(ttm_manager_type(&adev->mman.bdev,
+                                                            TTM_PL_TT),
+                                           root, "amdgpu_gtt_mm");
+       ttm_resource_manager_create_debugfs(ttm_manager_type(&adev->mman.bdev,
+                                                            AMDGPU_PL_GDS),
+                                           root, "amdgpu_gds_mm");
+       ttm_resource_manager_create_debugfs(ttm_manager_type(&adev->mman.bdev,
+                                                            AMDGPU_PL_GWS),
+                                           root, "amdgpu_gws_mm");
+       ttm_resource_manager_create_debugfs(ttm_manager_type(&adev->mman.bdev,
+                                                            AMDGPU_PL_OA),
+                                           root, "amdgpu_oa_mm");
+
 #endif
 }
-- 
2.32.0

Reply via email to