Hi Tom,
On Thu, 29 Aug 2024 at 19:26, Tom Rini <tr...@konsulko.com> wrote:
>
> On Thu, Aug 29, 2024 at 08:57:49AM -0600, Simon Glass wrote:
> > Show a bit more information when malloc() space is exhausted and
> > debugging is enabled.
> >
> > Signed-off-by: Simon Glass <s...@chromium.org>
> > ---
> >
> > common/malloc_simple.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/common/malloc_simple.c b/common/malloc_simple.c
> > index 5a8ec538f8f..f2b3dc53689 100644
> > --- a/common/malloc_simple.c
> > +++ b/common/malloc_simple.c
> > @@ -26,7 +26,8 @@ static void *alloc_simple(size_t bytes, int align)
> > log_debug("size=%lx, ptr=%lx, limit=%x: ", (ulong)bytes, new_ptr,
> > gd->malloc_limit);
> > if (new_ptr > gd->malloc_limit) {
> > - log_err("alloc space exhausted\n");
> > + log_err("alloc space exhausted %lx %x\n", new_ptr,
> > + gd->malloc_limit);
> > return NULL;
> > }
>
> Since debugging is enabled you should update the error message to say
> what the values mean...
OK, will do. This message is shown without debugging enabled, so I do
want to keep it short.
Regards,
Simon
