<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
looks good to me as well.<br>
<br>
Reviewed-by:
Arunpravin Paneer Selvam <<a href="mailto:Arunpravin.PaneerSelvam@amd.com" class="moz-txt-link-freetext">Arunpravin.PaneerSelvam@amd.com</a>><br>
<br>
<div class="moz-cite-prefix">On 8/7/2023 12:17 PM, Christian König
wrote:<br>
</div>
<blockquote type="cite" cite="mid:ed0b3a5c-9e81-6fee-aba1-5d1c9ca89130@amd.com">Am
04.08.23 um 08:56 schrieb Ma Jun:
<br>
<blockquote type="cite">[1] Change pages to blocks to avoid
confusion.
<br>
[2] Fix output format to align the output info.
<br>
<br>
Signed-off-by: Ma Jun <a class="moz-txt-link-rfc2396E" href="mailto:Jun.Ma2@amd.com"><Jun.Ma2@amd.com></a>
<br>
</blockquote>
<br>
Of hand looks good to me, but Arun should probably judge.
<br>
<br>
Christian.
<br>
<br>
<blockquote type="cite">---
<br>
drivers/gpu/drm/drm_buddy.c | 8 ++++----
<br>
1 file changed, 4 insertions(+), 4 deletions(-)
<br>
<br>
diff --git a/drivers/gpu/drm/drm_buddy.c
b/drivers/gpu/drm/drm_buddy.c
<br>
index 3d1f50f481cf..ef3dd15c334a 100644
<br>
--- a/drivers/gpu/drm/drm_buddy.c
<br>
+++ b/drivers/gpu/drm/drm_buddy.c
<br>
@@ -781,15 +781,15 @@ void drm_buddy_print(struct drm_buddy *mm,
struct drm_printer *p)
<br>
count++;
<br>
}
<br>
- drm_printf(p, "order-%d ", order);
<br>
+ drm_printf(p, "order-%2d ", order);
<br>
free = count * (mm->chunk_size << order);
<br>
if (free < SZ_1M)
<br>
- drm_printf(p, "free: %lluKiB", free >> 10);
<br>
+ drm_printf(p, "free: %8llu KiB", free >> 10);
<br>
else
<br>
- drm_printf(p, "free: %lluMiB", free >> 20);
<br>
+ drm_printf(p, "free: %8llu MiB", free >> 20);
<br>
- drm_printf(p, ", pages: %llu\n", count);
<br>
+ drm_printf(p, ", blocks: %llu\n", count);
<br>
}
<br>
}
<br>
EXPORT_SYMBOL(drm_buddy_print);
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>