The "gems" debugfs file is getting a little too wide for comfort. While
a lot of this is unavoidable due to the theoretical upper limits of
numbers here (e.g. size needs to be 16 chars because 2**48-1 in decimal
is 15 digits, plus one space for separation), the refcount column has a
decent 5 characters to be saved, as it can only ever contain a 10-digit
decimal number.

Reduce the refcount column's width to 11, which fulfils this requirement
with an additional space for separation.

Reviewed-by: Steven Price <[email protected]>
Signed-off-by: Nicolas Frattaroli <[email protected]>
---
 drivers/gpu/drm/panthor/panthor_gem.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/panthor/panthor_gem.c 
b/drivers/gpu/drm/panthor/panthor_gem.c
index 068aa935c8fc..dfdcda3d0189 100644
--- a/drivers/gpu/drm/panthor/panthor_gem.c
+++ b/drivers/gpu/drm/panthor/panthor_gem.c
@@ -1644,7 +1644,7 @@ static void panthor_gem_debugfs_bo_print(struct 
panthor_gem_object *bo,
 
        snprintf(creator_info, sizeof(creator_info),
                 "%s/%d", bo->debugfs.creator.process_name, 
bo->debugfs.creator.tgid);
-       seq_printf(m, "%-32s%-16d%-16d%-11d%-16zd%-16zd0x%-16lx",
+       seq_printf(m, "%-32s%-16d%-11d%-11d%-16zd%-16zd0x%-16lx",
                   creator_info,
                   bo->base.name,
                   refcount,
@@ -1681,8 +1681,8 @@ static void panthor_gem_debugfs_print_bos(struct 
panthor_device *ptdev,
 
        panthor_gem_debugfs_print_flag_names(m);
 
-       seq_puts(m, "created-by                      global-name     refcount   
     evictions  size            resident-size   file-offset       state      
usage       label\n");
-       seq_puts(m, 
"---------------------------------------------------------------------------------------------------------------------------------------------------------\n");
+       seq_puts(m, "created-by                      global-name     refcount   
evictions  size            resident-size   file-offset       state      usage   
    label\n");
+       seq_puts(m, 
"----------------------------------------------------------------------------------------------------------------------------------------------------\n");
 
        scoped_guard(mutex, &ptdev->gems.lock) {
                list_for_each_entry(bo, &ptdev->gems.node, debugfs.node) {
@@ -1690,7 +1690,7 @@ static void panthor_gem_debugfs_print_bos(struct 
panthor_device *ptdev,
                }
        }
 
-       seq_puts(m, 
"=========================================================================================================================================================\n");
+       seq_puts(m, 
"====================================================================================================================================================\n");
        seq_printf(m, "Total size: %zd, Total resident: %zd, Total reclaimable: 
%zd\n",
                   totals.size, totals.resident, totals.reclaimable);
 }

-- 
2.54.0

Reply via email to