[RFC PATCH v2 38/48] lib: Kconfig: add a config symbol for getting lmb memory map updates

[Ilias Apalodimas]

On Fri, 5 Jul 2024 at 22:51, Tom Rini <trini at konsulko.com> wrote:
>
> On Thu, Jul 04, 2024 at 01:05:34PM +0530, Sughosh Ganu wrote:
>
> > Add a Kconfig symbol to enable getting updates on any memory map
> > changes that might be done by the LMB module. This notification
> > mechanism can then be used to have a synchronous view of allocated and
> > free memory.
> >
> > Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> > ---
> > Changes since V1:
> > * Change the description to highlight only LMB notifications.
> > * Add a separate line for dependencies.
> >
> >  lib/Kconfig | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> >
> > diff --git a/lib/Kconfig b/lib/Kconfig
> > index 7eea517b3b..b422183a0f 100644
> > --- a/lib/Kconfig
> > +++ b/lib/Kconfig
> > @@ -74,6 +74,16 @@ config HAVE_PRIVATE_LIBGCC
> >  config LIB_UUID
> >       bool
> >
> > +config MEM_MAP_UPDATE_NOTIFY
> > +     bool "Get notified of any changes to the LMB memory map"
> > +     depends on EVENT && LMB && EFI_LOADER
> > +     default y
> > +     help
> > +       Enable this option to get notification on any changes to the
> > +       memory that is allocated or freed by the LMB module. This will
> > +       allow different modules that allocate memory or maintain a memory
> > +       map to have a synchronous view of available and allocated memory.
>
> This needs to be select'd when it's going to be used, opting out of
> making sure memory reservations are obeyed isn't a good idea.

+1 which begs the question, do we need the config option at all ?

Cheers
/Ilias
>
> --
> Tom