[PATCH 05/40] spl: alloc: call full alloc functions if malloc pool is available

Simon Glass sjg at chromium.org
Mon Jul 29 17:26:11 CEST 2024


Hi Sughosh,

On Mon, 29 Jul 2024 at 01:46, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> hi Simon,
>
> On Fri, 26 Jul 2024 at 05:02, Simon Glass <sjg at chromium.org> wrote:
> >
> > Hi Sughosh,
> >
> > On Wed, 24 Jul 2024 at 00:03, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
> > >
> > > If the malloc simple functionality is enabled in SPL, it is not
> > > possible to call the full-implementation alloc functions even after
> > > the heap is set-up in ram memory. Check for this condition and
> > > call the functions when enabled.
> >
> > Is this because you want to use lmb in SPL. Is that needed?
>
> Yes, there was a discussion on this, and Tom [1], and you also [2] had
> mentioned on the earlier series that we do need lmb in SPL. You had
> mentioned that this would subsequently be needed for VPL too.

OK, well at least this way we can make sure it is disabled if not
needed. We can worry about how to pass information through the phases,
later.


>
> -sughosh
>
> [1] - https://lists.denx.de/pipermail/u-boot/2024-July/558250.html
> [2] - https://lists.denx.de/pipermail/u-boot/2024-July/558644.html
>
> >
> > BTW I'll send a patch to allow alist to run without realloc().
> >
> > >
> > > Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> > > ---
> > > Changes since rfc: New patch
> > >
> > >  common/malloc_simple.c | 13 +++++++++++++
> > >  1 file changed, 13 insertions(+)

Regards,
Simon


More information about the U-Boot mailing list