efi_loader: create reservations after ft_board_setup
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Sat, 14 Mar 2020 09:59:34 +0000 (10:59 +0100)
committerHeinrich Schuchardt <xypron.glpk@gmx.de>
Tue, 17 Mar 2020 07:29:52 +0000 (08:29 +0100)
Some memory reservations are made in ft_board_setup(). Ensure that we
create reserved memory map entries after ft_board_setup().

The downside of this patch is that if bootefi is called multiple times with
an devicetree argument superfluous reservations for the old copies of the
device tree will exist. But that is still better than missing a reservation.

Deleting the superfluous reservations is not possible because reservations
in the memory map are rounded to page size and may be coallesced.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
cmd/bootefi.c

index 24fc42ae898ed3d3ed2173ff7fccf04dbfe5ddc7..522ed28d6ac573ec2ec279eb5a886e0daceb5a48 100644 (file)
@@ -263,9 +263,6 @@ efi_status_t efi_install_fdt(void *fdt)
                return EFI_LOAD_ERROR;
        }
 
-       /* Create memory reservations as indicated by the device tree */
-       efi_carve_out_dt_rsv(fdt);
-
        /* Prepare device tree for payload */
        ret = copy_fdt(&fdt);
        if (ret) {
@@ -278,6 +275,9 @@ efi_status_t efi_install_fdt(void *fdt)
                return EFI_LOAD_ERROR;
        }
 
+       /* Create memory reservations as indicated by the device tree */
+       efi_carve_out_dt_rsv(fdt);
+
        /* Install device tree as UEFI table */
        ret = efi_install_configuration_table(&efi_guid_fdt, fdt);
        if (ret != EFI_SUCCESS) {