[coreboot-gerrit] Change in coreboot[master]: lib/bootmem.c: Improve bootmem_allocate_buffer algorithm

[Angel Pons (Code Review)]

Angel Pons has submitted this change. ( 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
bail out of the search loop as soon as the region_base is bigger than
the max address allowed.

Signed-off-by: Jan Dabros 
Change-Id: I44b44bf9618fd0615103cbf74271235d61d49473
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43512
Reviewed-by: Angel Pons 
Reviewed-by: Paul Fagerburg 
Reviewed-by: Julius Werner 
Tested-by: build bot (Jenkins) 
---
M src/lib/bootmem.c
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Julius Werner: Looks good to me, approved
  Angel Pons: Looks good to me, approved
  Paul Fagerburg: Looks good to me, approved

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;