Browse Source

arm64/efi: mark UEFI reserved regions as MEMBLOCK_NOMAP

Change the EFI memory reservation logic to use memblock_mark_nomap()
rather than memblock_reserve() to mark UEFI reserved regions as
occupied. In addition to reserving them against allocations done by
memblock, this will also prevent them from being covered by the linear
mapping.

Reviewed-by: Matt Fleming <matt@codeblueprint.co.uk>
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Ard Biesheuvel 9 years ago
parent
commit
4dffbfc48d
1 changed files with 4 additions and 3 deletions
  1. 4 3
      arch/arm64/kernel/efi.c

+ 4 - 3
arch/arm64/kernel/efi.c

@@ -187,7 +187,7 @@ static __init void reserve_regions(void)
 			early_init_dt_add_memory_arch(paddr, size);
 			early_init_dt_add_memory_arch(paddr, size);
 
 
 		if (is_reserve_region(md)) {
 		if (is_reserve_region(md)) {
-			memblock_reserve(paddr, size);
+			memblock_mark_nomap(paddr, size);
 			if (efi_enabled(EFI_DBG))
 			if (efi_enabled(EFI_DBG))
 				pr_cont("*");
 				pr_cont("*");
 		}
 		}
@@ -209,8 +209,6 @@ void __init efi_init(void)
 
 
 	efi_system_table = params.system_table;
 	efi_system_table = params.system_table;
 
 
-	memblock_reserve(params.mmap & PAGE_MASK,
-			 PAGE_ALIGN(params.mmap_size + (params.mmap & ~PAGE_MASK)));
 	memmap.phys_map = params.mmap;
 	memmap.phys_map = params.mmap;
 	memmap.map = early_memremap(params.mmap, params.mmap_size);
 	memmap.map = early_memremap(params.mmap, params.mmap_size);
 	if (memmap.map == NULL) {
 	if (memmap.map == NULL) {
@@ -230,6 +228,9 @@ void __init efi_init(void)
 
 
 	reserve_regions();
 	reserve_regions();
 	early_memunmap(memmap.map, params.mmap_size);
 	early_memunmap(memmap.map, params.mmap_size);
+	memblock_mark_nomap(params.mmap & PAGE_MASK,
+			    PAGE_ALIGN(params.mmap_size +
+				       (params.mmap & ~PAGE_MASK)));
 }
 }
 
 
 static bool __init efi_virtmap_init(void)
 static bool __init efi_virtmap_init(void)