drm/amdgpu: move debug print into the MM managers
authorChristian König <christian.koenig@amd.com>
Mon, 7 Aug 2017 13:47:51 +0000 (15:47 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 17 Aug 2017 19:46:01 +0000 (15:46 -0400)
Instead of the separate switch/case in the calling function.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.c

index 26818b0a0cc86a2612ad122bf265f43856ee113a..97c63ee1e20a23e61017bc304c5254ddc559e9f6 100644 (file)
@@ -153,15 +153,6 @@ int amdgpu_gtt_mgr_alloc(struct ttm_mem_type_manager *man,
        return r;
 }
 
-void amdgpu_gtt_mgr_print(struct seq_file *m, struct ttm_mem_type_manager *man)
-{
-       struct amdgpu_device *adev = amdgpu_ttm_adev(man->bdev);
-       struct amdgpu_gtt_mgr *mgr = man->priv;
-
-       seq_printf(m, "man size:%llu pages, gtt available:%llu pages, usage:%lluMB\n",
-                  man->size, mgr->available, (u64)atomic64_read(&adev->gtt_usage) >> 20);
-
-}
 /**
  * amdgpu_gtt_mgr_new - allocate a new node
  *
@@ -260,11 +251,16 @@ static void amdgpu_gtt_mgr_del(struct ttm_mem_type_manager *man,
 static void amdgpu_gtt_mgr_debug(struct ttm_mem_type_manager *man,
                                 struct drm_printer *printer)
 {
+       struct amdgpu_device *adev = amdgpu_ttm_adev(man->bdev);
        struct amdgpu_gtt_mgr *mgr = man->priv;
 
        spin_lock(&mgr->lock);
        drm_mm_print(&mgr->mm, printer);
        spin_unlock(&mgr->lock);
+
+       drm_printf(printer, "man size:%llu pages, gtt available:%llu pages, usage:%lluMB\n",
+                  man->size, mgr->available,
+                  (u64)atomic64_read(&adev->gtt_usage) >> 20);
 }
 
 const struct ttm_mem_type_manager_func amdgpu_gtt_mgr_func = {
index e0520974e8636be562c2299b072c88a370988098..8b2c294f6f7999371a045aa985dd409ef12dd2d5 100644 (file)
@@ -1641,8 +1641,6 @@ error_free:
 
 #if defined(CONFIG_DEBUG_FS)
 
-extern void amdgpu_gtt_mgr_print(struct seq_file *m, struct ttm_mem_type_manager
-                                *man);
 static int amdgpu_mm_dump_table(struct seq_file *m, void *data)
 {
        struct drm_info_node *node = (struct drm_info_node *)m->private;
@@ -1653,17 +1651,6 @@ static int amdgpu_mm_dump_table(struct seq_file *m, void *data)
        struct drm_printer p = drm_seq_file_printer(m);
 
        man->func->debug(man, &p);
-       switch (ttm_pl) {
-       case TTM_PL_VRAM:
-               seq_printf(m, "man size:%llu pages, ram usage:%lluMB, vis usage:%lluMB\n",
-                          adev->mman.bdev.man[ttm_pl].size,
-                          (u64)atomic64_read(&adev->vram_usage) >> 20,
-                          (u64)atomic64_read(&adev->vram_vis_usage) >> 20);
-               break;
-       case TTM_PL_TT:
-               amdgpu_gtt_mgr_print(m, &adev->mman.bdev.man[TTM_PL_TT]);
-               break;
-       }
        return 0;
 }
 
index 3f86b47984a088eccd9f2b9f53cc24788d53ade6..1eb8d5d3acf26b4af1dc574aaf75069e45b8582d 100644 (file)
@@ -211,11 +211,17 @@ static void amdgpu_vram_mgr_del(struct ttm_mem_type_manager *man,
 static void amdgpu_vram_mgr_debug(struct ttm_mem_type_manager *man,
                                  struct drm_printer *printer)
 {
+       struct amdgpu_device *adev = amdgpu_ttm_adev(man->bdev);
        struct amdgpu_vram_mgr *mgr = man->priv;
 
        spin_lock(&mgr->lock);
        drm_mm_print(&mgr->mm, printer);
        spin_unlock(&mgr->lock);
+
+       drm_printf(printer, "man size:%llu pages, ram usage:%lluMB, vis usage:%lluMB\n",
+                  adev->mman.bdev.man[TTM_PL_VRAM].size,
+                  (u64)atomic64_read(&adev->vram_usage) >> 20,
+                  (u64)atomic64_read(&adev->vram_vis_usage) >> 20);
 }
 
 const struct ttm_mem_type_manager_func amdgpu_vram_mgr_func = {