Attention is currently required from: Arthur Heymans. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55069 )
Change subject: lib/bootmem.c: Use only e820 compatible types ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1: Also see I98e43917c104cc732e8557d1413aab7133aee387 [1] for FILO, it’s actually a bug in Linux:
This seems to be a coreboot or Linux kernel problem though as everything that is “reserved for future” in ACPI 6.3 Table 15-374 should be mapped to 2.
[1]: https://review.coreboot.org/c/filo/+/51120
SeaBIOS does the remapping already.
For GRUB:
``` Date: Mon, 17 May 2021 10:24:36 +0200 Subject: [PATCH] kern/coreboot/mmap: Map to reserved
https://git.savannah.gnu.org/cgit/grub.git/commit/?id=6de9ee86bf9ae50967413e...
Signed-off-by: Paul Menzel pmenzel@molgen.mpg.de --- grub-core/kern/coreboot/mmap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/grub-core/kern/coreboot/mmap.c b/grub-core/kern/coreboot/mmap.c index caf8f7cef..2fc316e8d 100644 --- a/grub-core/kern/coreboot/mmap.c +++ b/grub-core/kern/coreboot/mmap.c @@ -59,7 +59,7 @@ iterate_linuxbios_table (grub_linuxbios_table_item_t table_item, void *data) /* Multiboot mmaps match with the coreboot mmap definition. Therefore, we can just pass type through. */ - mem_region->type, + (mem_region->type >= 13) ? 2 : mem_region->type, ctx->hook_data)) return 1; if (start < 0xa0000) @@ -81,7 +81,7 @@ iterate_linuxbios_table (grub_linuxbios_table_item_t table_item, void *data) /* Multiboot mmaps match with the coreboot mmap definition. Therefore, we can just pass type through. */ - mem_region->type, + (mem_region->type >= 13) ? 2 : mem_region->type, ctx->hook_data)) return 1; }