On Thu, 11 Apr 2019 at 11:39, Ilias Apalodimas
<ilias.apalodi...@linaro.org> wrote:
>
> Following Ard's suggestion:
> Runtime data sections are intended for data that is used by the runtime
> services implementations.
> Let's change they type to EFI_ACPI_RECLAIM_MEMORY
>
> Suggested-by: Ard Biesheuvel <ard.biesheu...@linaro.org>
> Signed-off-by: Ilias Apalodimas <ilias.apalodi...@linaro.org>

It makes sense to mention that this actually fixes the boot issue you
were seeing with the ARM Linux kernel when using short descriptors.


> ---
>  cmd/bootefi.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cmd/bootefi.c b/cmd/bootefi.c
> index 3619a20e6433..b54181909aff 100644
> --- a/cmd/bootefi.c
> +++ b/cmd/bootefi.c
> @@ -111,13 +111,13 @@ static efi_status_t copy_fdt(void **fdtp)
>         new_fdt_addr = (uintptr_t)map_sysmem(fdt_ram_start + 0x7f00000 +
>                                              fdt_size, 0);
>         ret = efi_allocate_pages(EFI_ALLOCATE_MAX_ADDRESS,
> -                                EFI_RUNTIME_SERVICES_DATA, fdt_pages,
> +                                EFI_ACPI_RECLAIM_MEMORY, fdt_pages,
>                                  &new_fdt_addr);
>         if (ret != EFI_SUCCESS) {
>                 /* If we can't put it there, put it somewhere */
>                 new_fdt_addr = (ulong)memalign(EFI_PAGE_SIZE, fdt_size);
>                 ret = efi_allocate_pages(EFI_ALLOCATE_MAX_ADDRESS,
> -                                        EFI_RUNTIME_SERVICES_DATA, fdt_pages,
> +                                        EFI_ACPI_RECLAIM_MEMORY, fdt_pages,
>                                          &new_fdt_addr);
>                 if (ret != EFI_SUCCESS) {
>                         printf("ERROR: Failed to reserve space for FDT\n");
> --
> 2.7.4
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to