hi Simon, On Mon, 15 Jul 2024 at 17:09, Simon Glass <s...@chromium.org> wrote: > > Hi Sughosh, > > On Mon, 15 Jul 2024 at 10:42, Sughosh Ganu <sughosh.g...@linaro.org> wrote: > > > > hi Simon, > > > > On Sat, 13 Jul 2024 at 20:46, Simon Glass <s...@chromium.org> wrote: > > > > > > Hi Sughosh, > > > > > > On Thu, 4 Jul 2024 at 08:38, Sughosh Ganu <sughosh.g...@linaro.org> wrote: > > > > > > > > Mark the EFI runtime memory region as reserved memory during board > > > > init so that it does not get allocated by the LMB module on subsequent > > > > memory requests. > > > > > > > > Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org> > > > > --- > > > > Changes since V1: New patch > > > > > > > > lib/lmb.c | 41 ++++++++++++++++++++--------------------- > > > > 1 file changed, 20 insertions(+), 21 deletions(-) > > > > > > I see again that this is getting circular. Can you look at what is > > > actually allocated by EFI on init (i.e. before anything is booted)? > > > > This code is simply ensuring that the EFI runtime regions are marked > > as reserved by the LMB module to prevent that region from being > > allocated. Do you have any other way by which this can be communicated > > to the LMB module ? > > But this region is within the U-Boot code area, so it should be > covered by the normal exclusion of that area?
Yes that is the case for all arch's today. Let me check if I can simply drop this function then. -sughosh > > Regards, > Simon
