Martin Roth (martinroth@google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16585
-gerrit
commit 826b9279eb99d0260cf295622603dfb1312f3606 Author: Simon Glass sjg@chromium.org Date: Sun Sep 4 16:37:04 2016 -0600
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 martinroth@chromium.org Original-Commit-Id: 2381e02efa2033857ac06acbc4f0c0dd08de1080 Original-Change-Id: I005e5e4d9b2136605bdd95e9060655df7a8238cb Original-Signed-off-by: Simon Glass sjg@chromium.org Original-Reviewed-on: https://chromium-review.googlesource.com/381092 Original-Reviewed-by: Aaron Durbin adurbin@chromium.org --- src/lib/selfboot.c | 26 ++++++++++++++++++-------- 1 file changed, 18 insertions(+), 8 deletions(-)
diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c index 2fdf8ce..1ce7f94 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