On 11/7/24 10:56, Ilias Apalodimas wrote:
On Sat, 2 Nov 2024 at 06:41, Heinrich Schuchardt
<[email protected]> wrote:
In the message string " %s[%d]\t[0x%llx-0x%llx], 0x%08llx bytes flags: "
a comma is missing before flags.
To avoid increasing the code size replace '0x%' by '%#'.
Signed-off-by: Heinrich Schuchardt <[email protected]>
---
lib/lmb.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/lmb.c b/lib/lmb.c
index 00c8888936a..95adf2e93a4 100644
--- a/lib/lmb.c
+++ b/lib/lmb.c
@@ -97,7 +97,7 @@ static void lmb_dump_region(struct alist *lmb_rgn_lst, char
*name)
enum lmb_flags flags;
int i;
- printf(" %s.count = 0x%x\n", name, lmb_rgn_lst->count);
+ printf(" %s.count = %#x\n", name, lmb_rgn_lst->count);
for (i = 0; i < lmb_rgn_lst->count; i++) {
base = rgn[i].base;
@@ -105,7 +105,7 @@ static void lmb_dump_region(struct alist *lmb_rgn_lst, char
*name)
end = base + size - 1;
flags = rgn[i].flags;
- printf(" %s[%d]\t[0x%llx-0x%llx], 0x%08llx bytes flags: ",
+ printf(" %s[%d]\t[%#llx-%#llx], %#08llx bytes, flags: ",
If we use #, we have to increase the size (8) by two to get the same
output. But why have leading zeros for the size if we have none for the
addresses?
Let's drop the 08. Allocations of more than 4 GiB anyway would not fit
into 8 digits.
Best regards
Heinrich
name, i, base, end, size);
lmb_print_region_flags(flags);
}
--
2.45.2
Reviewed-by: Ilias Apalodimas <[email protected]>
