[RFC PATCH v2 33/48] test: bdinfo: dump the global LMB memory map

[Sughosh Ganu]

The LMB code has been changed to make the memory reservations
persistent and global. Make corresponding change the the
lmb_test_dump_all() function to print the global LMB available and
used memory.

Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org>
---
Changes since V1:
* Corresponding changes needed to work with alist based lmb lists.

 test/cmd/bdinfo.c | 28 ++++++++++++++++------------
 1 file changed, 16 insertions(+), 12 deletions(-)

diff --git a/test/cmd/bdinfo.c b/test/cmd/bdinfo.c
index 1cd81a195b..3184aaf629 100644
--- a/test/cmd/bdinfo.c
+++ b/test/cmd/bdinfo.c
@@ -5,6 +5,7 @@
  * Copyright 2023 Marek Vasut <marek.vasut+rene...@mailbox.org>
  */
 
+#include <alist.h>
 #include <console.h>
 #include <mapmem.h>
 #include <asm/global_data.h>
@@ -21,6 +22,7 @@
 #include <asm/cache.h>
 #include <asm/global_data.h>
 #include <display_options.h>
+#include <linux/kernel.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -99,19 +101,20 @@ static int test_video_info(struct unit_test_state *uts)
 }
 
 static int lmb_test_dump_region(struct unit_test_state *uts,
-                               struct lmb_region *rgn, char *name)
+                               struct alist *lmb_rgn_lst, char *name)
 {
+       struct lmb_region *rgn = lmb_rgn_lst->data;
        unsigned long long base, size, end;
        enum lmb_flags flags;
        int i;
 
-       ut_assert_nextline(" %s.cnt = 0x%lx / max = 0x%lx", name, rgn->cnt, 
rgn->max);
+       ut_assert_nextline(" %s.count = 0x%hx", name, lmb_rgn_lst->count);
 
-       for (i = 0; i < rgn->cnt; i++) {
-               base = rgn->region[i].base;
-               size = rgn->region[i].size;
+       for (i = 0; i < lmb_rgn_lst->count; i++) {
+               base = rgn[i].base;
+               size = rgn[i].size;
                end = base + size - 1;
-               flags = rgn->region[i].flags;
+               flags = rgn[i].flags;
 
                if (!IS_ENABLED(CONFIG_SANDBOX) && i == 3) {
                        ut_assert_nextlinen(" %s[%d]\t[", name, i);
@@ -124,11 +127,14 @@ static int lmb_test_dump_region(struct unit_test_state 
*uts,
        return 0;
 }
 
-static int lmb_test_dump_all(struct unit_test_state *uts, struct lmb *lmb)
+static int lmb_test_dump_all(struct unit_test_state *uts)
 {
+       extern struct alist lmb_free_mem;
+       extern struct alist lmb_used_mem;
+
        ut_assert_nextline("lmb_dump_all:");
-       ut_assertok(lmb_test_dump_region(uts, &lmb->memory, "memory"));
-       ut_assertok(lmb_test_dump_region(uts, &lmb->reserved, "reserved"));
+       ut_assertok(lmb_test_dump_region(uts, &lmb_free_mem, "memory"));
+       ut_assertok(lmb_test_dump_region(uts, &lmb_used_mem, "reserved"));
 
        return 0;
 }
@@ -190,9 +196,7 @@ static int bdinfo_test_all(struct unit_test_state *uts)
 #endif
 
        if (IS_ENABLED(CONFIG_LMB) && gd->fdt_blob) {
-               struct lmb lmb;
-
-               ut_assertok(lmb_test_dump_all(uts, &lmb));
+               ut_assertok(lmb_test_dump_all(uts));
                if (IS_ENABLED(CONFIG_OF_REAL))
                        ut_assert_nextline("devicetree  = %s", 
fdtdec_get_srcname());
        }
-- 
2.34.1