From 78c63860a61c4562e78a35c5d5457085acf27670 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Sun, 4 Sep 2016 16:37:04 -0600 Subject: selfboot: Move the usable-RAM check into a function In preparation for making this check optional, move it into its own function. load_self_segments() is already long and we don't want to make it longer. BUG=chrome-os-partner:56314 BRANCH=none TEST=boot on gru and see that BL31 loads and runs correctly Change-Id: If48d2bf485a23f21c5599670e77a7b8b098f1a88 Signed-off-by: Martin Roth Original-Commit-Id: 2381e02efa2033857ac06acbc4f0c0dd08de1080 Original-Change-Id: I005e5e4d9b2136605bdd95e9060655df7a8238cb Original-Signed-off-by: Simon Glass Original-Reviewed-on: https://chromium-review.googlesource.com/381092 Original-Reviewed-by: Aaron Durbin Reviewed-on: https://review.coreboot.org/16585 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber Reviewed-by: Paul Menzel --- src/lib/selfboot.c | 26 ++++++++++++++++++-------- 1 file changed, 18 insertions(+), 8 deletions(-) (limited to 'src') diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c index 2fdf8ce332..1ce7f94f5a 100644 --- a/src/lib/selfboot.c +++ b/src/lib/selfboot.c @@ -328,23 +328,20 @@ static int build_self_segment_list( return 1; } -static int load_self_segments( - struct segment *head, - struct prog *payload) +static int payload_targets_usable_ram(struct segment *head) { - struct segment *ptr; const unsigned long one_meg = (1UL << 20); - unsigned long bounce_high = lb_end; + struct segment *ptr; - for(ptr = head->next; ptr != head; ptr = ptr->next) { + for (ptr = head->next; ptr != head; ptr = ptr->next) { if (bootmem_region_targets_usable_ram(ptr->s_dstaddr, - ptr->s_memsz)) + ptr->s_memsz)) continue; if (ptr->s_dstaddr < one_meg && (ptr->s_dstaddr + ptr->s_memsz) <= one_meg) { printk(BIOS_DEBUG, - "Payload being loaded below 1MiB " + "Payload being loaded at below 1MiB " "without region being marked as RAM usable.\n"); continue; } @@ -357,6 +354,19 @@ static int load_self_segments( return 0; } + return 1; +} + +static int load_self_segments( + struct segment *head, + struct prog *payload) +{ + struct segment *ptr; + unsigned long bounce_high = lb_end; + + if (!payload_targets_usable_ram(head)) + return 0; + for(ptr = head->next; ptr != head; ptr = ptr->next) { /* * Add segments to bootmem memory map before a bounce buffer is -- cgit v1.2.3