Hi Rasmus, On Mon, 25 Sept 2023 at 02:54, Rasmus Villemoes <rasmus.villem...@prevas.dk> wrote: > > On 21/09/2023 03.02, Simon Glass wrote: > > Hi Rasmus, > > > > On Fri, 15 Sept 2023 at 11:51, Rasmus Villemoes > > <rasmus.villem...@prevas.dk> wrote: > >> > >> Currently, setting both CONFIG_SPL_SYS_MALLOC_SIMPLE and > >> CONFIG_SYS_SPL_MALLOC (but not CONFIG_SPL_STACK_R) doesn't work as > >> expected: The SIMPLE option means that all malloc etc. calls are > >> directed at build-time to the implementation in malloc_simple.c, but > > > > Sort-of. It is control by both a CONFIG and GD_FLG_FULL_MALLOC_INIT. > > > > Eh, no? The CONFIG option completely preempts any run-time gd flag checking > > #if CONFIG_IS_ENABLED(SYS_MALLOC_SIMPLE) > #define malloc malloc_simple > #define realloc realloc_simple > #define memalign memalign_simple > > Yes, _without_ that CONFIG option, the "real" malloc() functions do > check that gd flag and if not set fall back to the _simple variants. But > what I wrote is not merely "sort-of" correct.
OK I see what you mean, yes. Regards, Simon
