Jan Dabros has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43512 )
Change subject: lib/bootmem.c: Improve bootmem_allocate_buffer algorithm ......................................................................
lib/bootmem.c: Improve bootmem_allocate_buffer algorithm
Since regions in bootmem are sorted by increasing base address, we may boil out of the search loop as soon as the region_base is bigger than the max address allowed.
Signed-off-by: Jan Dabros jsd@semihalf.com Change-Id: I44b44bf9618fd0615103cbf74271235d61d49473 --- M src/lib/bootmem.c 1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/43512/1
diff --git a/src/lib/bootmem.c b/src/lib/bootmem.c index fa1f8bc..1fe23c2 100644 --- a/src/lib/bootmem.c +++ b/src/lib/bootmem.c @@ -231,15 +231,15 @@ size = ALIGN(size, 4096); region = NULL; memranges_each_entry(r, &bootmem) { + if (range_entry_base(r) >= max_addr) + break; + if (range_entry_size(r) < size) continue;
if (range_entry_tag(r) != BM_MEM_RAM) continue;
- if (range_entry_base(r) >= max_addr) - continue; - end = range_entry_end(r); if (end > max_addr) end = max_addr;