[U-Boot] [PATCH 29/45] malloc_simple: Add logging of allocations

[Simon Glass]

It is sometimes useful to see what memory is being allocated early during
boot. Add logging to support this, using a new LOGC_ALLOC category.

Signed-off-by: Simon Glass <s...@chromium.org>
---

 common/malloc_simple.c | 58 +++++++++++++++++++++++++++---------------
 include/malloc.h       |  1 +
 2 files changed, 38 insertions(+), 21 deletions(-)

diff --git a/common/malloc_simple.c b/common/malloc_simple.c
index 871b5444bd7..aae0b66680a 100644
--- a/common/malloc_simple.c
+++ b/common/malloc_simple.c
@@ -5,6 +5,8 @@
  * Copyright (c) 2014 Google, Inc
  */
 
+#define LOG_CATEGORY LOGC_ALLOC
+
 #include <common.h>
 #include <malloc.h>
 #include <mapmem.h>
@@ -12,40 +14,47 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-void *malloc_simple(size_t bytes)
+static void *alloc_simple(size_t bytes, int align)
 {
-       ulong new_ptr;
+       ulong addr, new_ptr;
        void *ptr;
 
-       new_ptr = gd->malloc_ptr + bytes;
-       debug("%s: size=%zx, ptr=%lx, limit=%lx: ", __func__, bytes, new_ptr,
-             gd->malloc_limit);
+       addr = ALIGN(gd->malloc_base + gd->malloc_ptr, align);
+       new_ptr = addr + bytes - gd->malloc_base;
+       log_debug("size=%zx, ptr=%lx, limit=%lx: ", bytes, new_ptr,
+       gd->malloc_limit);
        if (new_ptr > gd->malloc_limit) {
-               debug("space exhausted\n");
+               log(LOGC_ALLOC, LOGL_ERR, "alloc space exhausted\n");
                return NULL;
        }
-       ptr = map_sysmem(gd->malloc_base + gd->malloc_ptr, bytes);
+
+       ptr = map_sysmem(addr, bytes);
        gd->malloc_ptr = ALIGN(new_ptr, sizeof(new_ptr));
-       debug("%lx\n", (ulong)ptr);
 
        return ptr;
 }
 
-void *memalign_simple(size_t align, size_t bytes)
+void *malloc_simple(size_t bytes)
 {
-       ulong addr, new_ptr;
        void *ptr;
 
-       addr = ALIGN(gd->malloc_base + gd->malloc_ptr, align);
-       new_ptr = addr + bytes - gd->malloc_base;
-       if (new_ptr > gd->malloc_limit) {
-               debug("space exhausted\n");
-               return NULL;
-       }
+       ptr = alloc_simple(bytes, 1);
+       if (!ptr)
+               return ptr;
 
-       ptr = map_sysmem(addr, bytes);
-       gd->malloc_ptr = ALIGN(new_ptr, sizeof(new_ptr));
-       debug("%lx\n", (ulong)ptr);
+       log_debug("%lx\n", (ulong)ptr);
+
+       return ptr;
+}
+
+void *memalign_simple(size_t align, size_t bytes)
+{
+       void *ptr;
+
+       ptr = alloc_simple(bytes, align);
+       if (!ptr)
+               return ptr;
+       log_debug("aligned to %lx\n", (ulong)ptr);
 
        return ptr;
 }
@@ -57,9 +66,16 @@ void *calloc(size_t nmemb, size_t elem_size)
        void *ptr;
 
        ptr = malloc(size);
-       if (ptr)
-               memset(ptr, '\0', size);
+       if (!ptr)
+               return ptr;
+       memset(ptr, '\0', size);
 
        return ptr;
 }
 #endif
+
+void malloc_simple_info(void)
+{
+       log_info("malloc_simple: %lx bytes used, %lx remain\n", gd->malloc_ptr,
+                CONFIG_VAL(SYS_MALLOC_F_LEN) - gd->malloc_ptr);
+}
diff --git a/include/malloc.h b/include/malloc.h
index 8175c75920c..b714fedf457 100644
--- a/include/malloc.h
+++ b/include/malloc.h
@@ -880,6 +880,7 @@ static inline void free(void *ptr) {}
 void *calloc(size_t nmemb, size_t size);
 void *memalign_simple(size_t alignment, size_t bytes);
 void *realloc_simple(void *ptr, size_t size);
+void malloc_simple_info(void);
 #else
 
 # ifdef USE_DL_PREFIX
-- 
2.19.0.605.g01d371f741-goog

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot