On Sun, 13 Oct 2024 at 04:56, Sughosh Ganu <[email protected]> wrote: > > The variable overlap_only_ram is used to specify that the new memory > region that is being created needs to come from the free memory pool > -- this is done by carving out the memory region from the free > memory. The name is a bit confusing though, as other allocated memory > regions, like boot-services code and data are also part of the RAM > memory. Rename the variable to overlap_conventional to highlight the > fact that it is the free/conventional memory that is being referred to > in this context. > > Signed-off-by: Sughosh Ganu <[email protected]> > --- > Changes since V2: > * Rename overlap_no_freemem to overlap_conventional as suggested by > Ilias. > > include/efi_loader.h | 16 +++++++----- > lib/efi_loader/efi_memory.c | 51 ++++++++++++++++++++----------------- > 2 files changed, 37 insertions(+), 30 deletions(-) >
Reviewed-by: Simon Glass <[email protected]>

