Blame SOURCES/0114-arm-arm64-loader-Better-memory-allocation-and-error-.patch

8e15ce
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
8e15ce
From: Peter Jones <pjones@redhat.com>
8e15ce
Date: Thu, 11 Jul 2019 14:38:57 +0200
8e15ce
Subject: [PATCH] arm/arm64 loader: Better memory allocation and error
8e15ce
 messages.
8e15ce
8e15ce
On mustang, our memory map looks like:
8e15ce
8e15ce
Type      Physical start  - end             #Pages        Size Attributes
8e15ce
reserved  0000004000000000-00000040001fffff 00000200      2MiB UC WC WT WB
8e15ce
conv-mem  0000004000200000-0000004393ffffff 00393e00  14654MiB UC WC WT WB
8e15ce
ldr-code  0000004394000000-00000043f7ffffff 00064000   1600MiB UC WC WT WB
8e15ce
BS-data   00000043f8000000-00000043f801ffff 00000020    128KiB UC WC WT WB
8e15ce
conv-mem  00000043f8020000-00000043fa15bfff 0000213c  34032KiB UC WC WT WB
8e15ce
ldr-code  00000043fa15c000-00000043fa2a1fff 00000146   1304KiB UC WC WT WB
8e15ce
ldr-data  00000043fa2a2000-00000043fa3e8fff 00000147   1308KiB UC WC WT WB
8e15ce
conv-mem  00000043fa3e9000-00000043fa3e9fff 00000001      4KiB UC WC WT WB
8e15ce
ldr-data  00000043fa3ea000-00000043fa3eafff 00000001      4KiB UC WC WT WB
8e15ce
ldr-code  00000043fa3eb000-00000043fa4affff 000000c5    788KiB UC WC WT WB
8e15ce
BS-code   00000043fa4b0000-00000043fa59ffff 000000f0    960KiB UC WC WT WB
8e15ce
RT-code   00000043fa5a0000-00000043fa5affff 00000010     64KiB RT UC WC WT WB
8e15ce
RT-data   00000043fa5b0000-00000043fa5bffff 00000010     64KiB RT UC WC WT WB
8e15ce
RT-code   00000043fa5c0000-00000043fa5cffff 00000010     64KiB RT UC WC WT WB
8e15ce
ldr-data  00000043fa5d0000-00000043fa5d0fff 00000001      4KiB UC WC WT WB
8e15ce
BS-code   00000043fa5d1000-00000043fa5ddfff 0000000d     52KiB UC WC WT WB
8e15ce
reserved  00000043fa5de000-00000043fa60ffff 00000032    200KiB UC WC WT WB
8e15ce
ACPI-rec  00000043fa610000-00000043fa6affff 000000a0    640KiB UC WC WT WB
8e15ce
ACPI-nvs  00000043fa6b0000-00000043fa6bffff 00000010     64KiB UC WC WT WB
8e15ce
ACPI-rec  00000043fa6c0000-00000043fa70ffff 00000050    320KiB UC WC WT WB
8e15ce
RT-code   00000043fa710000-00000043fa72ffff 00000020    128KiB RT UC WC WT WB
8e15ce
RT-data   00000043fa730000-00000043fa78ffff 00000060    384KiB RT UC WC WT WB
8e15ce
RT-code   00000043fa790000-00000043fa79ffff 00000010     64KiB RT UC WC WT WB
8e15ce
RT-data   00000043fa7a0000-00000043fa99ffff 00000200      2MiB RT UC WC WT WB
8e15ce
RT-code   00000043fa9a0000-00000043fa9affff 00000010     64KiB RT UC WC WT WB
8e15ce
RT-data   00000043fa9b0000-00000043fa9cffff 00000020    128KiB RT UC WC WT WB
8e15ce
BS-code   00000043fa9d0000-00000043fa9d9fff 0000000a     40KiB UC WC WT WB
8e15ce
reserved  00000043fa9da000-00000043fa9dbfff 00000002      8KiB UC WC WT WB
8e15ce
conv-mem  00000043fa9dc000-00000043fc29dfff 000018c2  25352KiB UC WC WT WB
8e15ce
BS-data   00000043fc29e000-00000043fc78afff 000004ed   5044KiB UC WC WT WB
8e15ce
conv-mem  00000043fc78b000-00000043fca01fff 00000277   2524KiB UC WC WT WB
8e15ce
BS-data   00000043fca02000-00000043fcea3fff 000004a2   4744KiB UC WC WT WB
8e15ce
conv-mem  00000043fcea4000-00000043fcea4fff 00000001      4KiB UC WC WT WB
8e15ce
BS-data   00000043fcea5000-00000043fd192fff 000002ee   3000KiB UC WC WT WB
8e15ce
conv-mem  00000043fd193000-00000043fd2b0fff 0000011e   1144KiB UC WC WT WB
8e15ce
BS-data   00000043fd2b1000-00000043ff80ffff 0000255f  38268KiB UC WC WT WB
8e15ce
BS-code   00000043ff810000-00000043ff99ffff 00000190   1600KiB UC WC WT WB
8e15ce
RT-code   00000043ff9a0000-00000043ff9affff 00000010     64KiB RT UC WC WT WB
8e15ce
conv-mem  00000043ff9b0000-00000043ff9bffff 00000010     64KiB UC WC WT WB
8e15ce
RT-data   00000043ff9c0000-00000043ff9effff 00000030    192KiB RT UC WC WT WB
8e15ce
conv-mem  00000043ff9f0000-00000043ffa05fff 00000016     88KiB UC WC WT WB
8e15ce
BS-data   00000043ffa06000-00000043ffffffff 000005fa   6120KiB UC WC WT WB
8e15ce
MMIO      0000000010510000-0000000010510fff 00000001      4KiB RT
8e15ce
MMIO      0000000010548000-0000000010549fff 00000002      8KiB RT
8e15ce
MMIO      0000000017000000-0000000017001fff 00000002      8KiB RT
8e15ce
MMIO      000000001c025000-000000001c025fff 00000001      4KiB RT
8e15ce
8e15ce
This patch adds a requirement when we're trying to find the base of ram, that
8e15ce
the memory we choose is actually /allocatable/ conventional memory, not merely
8e15ce
write-combining.  On this machine that means we wind up with an allocation
8e15ce
around 0x4392XXXXXX, which is a reasonable address.
8e15ce
8e15ce
This also changes grub_efi_allocate_pages_real() so that if 0 is allocated, it
8e15ce
tries to allocate again starting with the same max address it did the first
8e15ce
time, rather than interposing GRUB_EFI_MAX_USABLE_ADDRESS there, so that any
8e15ce
per-platform constraints on its given address are maintained.
8e15ce
8e15ce
Signed-off-by: Peter Jones <pjones@redhat.com>
8e15ce
---
8e15ce
 grub-core/kern/efi/mm.c        | 33 +++++++++++++++-----
8e15ce
 grub-core/loader/arm64/linux.c | 68 +++++++++++++++++++++++++++++++-----------
8e15ce
 2 files changed, 76 insertions(+), 25 deletions(-)
8e15ce
8e15ce
diff --git a/grub-core/kern/efi/mm.c b/grub-core/kern/efi/mm.c
8e15ce
index f6aef0ef649..85ad4b4494c 100644
8e15ce
--- a/grub-core/kern/efi/mm.c
8e15ce
+++ b/grub-core/kern/efi/mm.c
8e15ce
@@ -154,6 +154,7 @@ grub_efi_allocate_pages_real (grub_efi_physical_address_t address,
8e15ce
 {
8e15ce
   grub_efi_status_t status;
8e15ce
   grub_efi_boot_services_t *b;
8e15ce
+  grub_efi_physical_address_t ret = address;
8e15ce
 
8e15ce
   /* Limit the memory access to less than 4GB for 32-bit platforms.  */
8e15ce
   if (address > GRUB_EFI_MAX_USABLE_ADDRESS)
8e15ce
@@ -170,19 +171,22 @@ grub_efi_allocate_pages_real (grub_efi_physical_address_t address,
8e15ce
     }
8e15ce
 
8e15ce
   b = grub_efi_system_table->boot_services;
8e15ce
-  status = efi_call_4 (b->allocate_pages, alloctype, memtype, pages, &address);
8e15ce
+  status = efi_call_4 (b->allocate_pages, alloctype, memtype, pages, &ret;;
8e15ce
   if (status != GRUB_EFI_SUCCESS)
8e15ce
     {
8e15ce
+      grub_dprintf ("efi",
8e15ce
+		    "allocate_pages(%d, %d, 0x%0lx, 0x%016lx) = 0x%016lx\n",
8e15ce
+		    alloctype, memtype, pages, address, status);
8e15ce
       grub_error (GRUB_ERR_OUT_OF_MEMORY, N_("out of memory"));
8e15ce
       return NULL;
8e15ce
     }
8e15ce
 
8e15ce
-  if (address == 0)
8e15ce
+  if (ret == 0)
8e15ce
     {
8e15ce
       /* Uggh, the address 0 was allocated... This is too annoying,
8e15ce
 	 so reallocate another one.  */
8e15ce
-      address = GRUB_EFI_MAX_USABLE_ADDRESS;
8e15ce
-      status = efi_call_4 (b->allocate_pages, alloctype, memtype, pages, &address);
8e15ce
+      ret = address;
8e15ce
+      status = efi_call_4 (b->allocate_pages, alloctype, memtype, pages, &ret;;
8e15ce
       grub_efi_free_pages (0, pages);
8e15ce
       if (status != GRUB_EFI_SUCCESS)
8e15ce
 	{
8e15ce
@@ -191,9 +195,9 @@ grub_efi_allocate_pages_real (grub_efi_physical_address_t address,
8e15ce
 	}
8e15ce
     }
8e15ce
 
8e15ce
-  grub_efi_store_alloc (address, pages);
8e15ce
+  grub_efi_store_alloc (ret, pages);
8e15ce
 
8e15ce
-  return (void *) ((grub_addr_t) address);
8e15ce
+  return (void *) ((grub_addr_t) ret);
8e15ce
 }
8e15ce
 
8e15ce
 void *
8e15ce
@@ -713,8 +717,21 @@ grub_efi_get_ram_base(grub_addr_t *base_addr)
8e15ce
   for (desc = memory_map, *base_addr = GRUB_EFI_MAX_USABLE_ADDRESS;
8e15ce
        (grub_addr_t) desc < ((grub_addr_t) memory_map + memory_map_size);
8e15ce
        desc = NEXT_MEMORY_DESCRIPTOR (desc, desc_size))
8e15ce
-    if (desc->attribute & GRUB_EFI_MEMORY_WB)
8e15ce
-      *base_addr = grub_min (*base_addr, desc->physical_start);
8e15ce
+    {
8e15ce
+      if (desc->type == GRUB_EFI_CONVENTIONAL_MEMORY &&
8e15ce
+          (desc->attribute & GRUB_EFI_MEMORY_WB))
8e15ce
+        {
8e15ce
+          *base_addr = grub_min (*base_addr, desc->physical_start);
8e15ce
+          grub_dprintf ("efi", "setting base_addr=0x%016lx\n", *base_addr);
8e15ce
+        }
8e15ce
+      else
8e15ce
+        {
8e15ce
+          grub_dprintf ("efi", "ignoring address 0x%016lx\n", desc->physical_start);
8e15ce
+        }
8e15ce
+    }
8e15ce
+
8e15ce
+  if (*base_addr == GRUB_EFI_MAX_USABLE_ADDRESS)
8e15ce
+    grub_dprintf ("efi", "base_addr 0x%016lx is probably wrong.\n", *base_addr);
8e15ce
 
8e15ce
   grub_free(memory_map);
8e15ce
 
8e15ce
diff --git a/grub-core/loader/arm64/linux.c b/grub-core/loader/arm64/linux.c
8e15ce
index 04994d5c67d..70a0075ec5e 100644
8e15ce
--- a/grub-core/loader/arm64/linux.c
8e15ce
+++ b/grub-core/loader/arm64/linux.c
8e15ce
@@ -71,20 +71,25 @@ finalize_params_linux (void)
8e15ce
 {
8e15ce
   grub_efi_loaded_image_t *loaded_image = NULL;
8e15ce
   int node, retval, len;
8e15ce
-
8e15ce
+  grub_err_t err = GRUB_ERR_NONE;
8e15ce
   void *fdt;
8e15ce
 
8e15ce
   fdt = grub_fdt_load (GRUB_EFI_LINUX_FDT_EXTRA_SPACE);
8e15ce
-
8e15ce
   if (!fdt)
8e15ce
-    goto failure;
8e15ce
+    {
8e15ce
+      err = grub_error(GRUB_ERR_BAD_OS, "failed to load FDT");
8e15ce
+      goto failure;
8e15ce
+    }
8e15ce
 
8e15ce
   node = grub_fdt_find_subnode (fdt, 0, "chosen");
8e15ce
   if (node < 0)
8e15ce
     node = grub_fdt_add_subnode (fdt, 0, "chosen");
8e15ce
 
8e15ce
   if (node < 1)
8e15ce
-    goto failure;
8e15ce
+    {
8e15ce
+      err = grub_error(grub_errno, "failed to load chosen fdt node.");
8e15ce
+      goto failure;
8e15ce
+    }
8e15ce
 
8e15ce
   /* Set initrd info */
8e15ce
   if (initrd_start && initrd_end > initrd_start)
8e15ce
@@ -95,15 +100,26 @@ finalize_params_linux (void)
8e15ce
       retval = grub_fdt_set_prop64 (fdt, node, "linux,initrd-start",
8e15ce
 				    initrd_start);
8e15ce
       if (retval)
8e15ce
-	goto failure;
8e15ce
+	{
8e15ce
+	  err = grub_error(retval, "Failed to set linux,initrd-start property");
8e15ce
+	  goto failure;
8e15ce
+	}
8e15ce
+
8e15ce
       retval = grub_fdt_set_prop64 (fdt, node, "linux,initrd-end",
8e15ce
 				    initrd_end);
8e15ce
       if (retval)
8e15ce
-	goto failure;
8e15ce
+	{
8e15ce
+	  err = grub_error(retval, "Failed to set linux,initrd-end property");
8e15ce
+	  goto failure;
8e15ce
+	}
8e15ce
     }
8e15ce
 
8e15ce
-  if (grub_fdt_install() != GRUB_ERR_NONE)
8e15ce
-    goto failure;
8e15ce
+  retval = grub_fdt_install();
8e15ce
+  if (retval != GRUB_ERR_NONE)
8e15ce
+    {
8e15ce
+      err = grub_error(retval, "Failed to install fdt");
8e15ce
+      goto failure;
8e15ce
+    }
8e15ce
 
8e15ce
   grub_dprintf ("linux", "Installed/updated FDT configuration table @ %p\n",
8e15ce
                 fdt);
8e15ce
@@ -111,14 +127,20 @@ finalize_params_linux (void)
8e15ce
   /* Convert command line to UCS-2 */
8e15ce
   loaded_image = grub_efi_get_loaded_image (grub_efi_image_handle);
8e15ce
   if (!loaded_image)
8e15ce
-    goto failure;
8e15ce
+    {
8e15ce
+      err = grub_error(grub_errno, "Failed to install fdt");
8e15ce
+      goto failure;
8e15ce
+    }
8e15ce
 
8e15ce
   loaded_image->load_options_size = len =
8e15ce
     (grub_strlen (linux_args) + 1) * sizeof (grub_efi_char16_t);
8e15ce
   loaded_image->load_options =
8e15ce
     grub_efi_allocate_any_pages (GRUB_EFI_BYTES_TO_PAGES (loaded_image->load_options_size));
8e15ce
   if (!loaded_image->load_options)
8e15ce
-    return grub_error(GRUB_ERR_BAD_OS, "failed to create kernel parameters");
8e15ce
+    {
8e15ce
+      err = grub_error(GRUB_ERR_BAD_OS, "failed to create kernel parameters");
8e15ce
+      goto failure;
8e15ce
+    }
8e15ce
 
8e15ce
   loaded_image->load_options_size =
8e15ce
     2 * grub_utf8_to_utf16 (loaded_image->load_options, len,
8e15ce
@@ -128,7 +150,7 @@ finalize_params_linux (void)
8e15ce
 
8e15ce
 failure:
8e15ce
   grub_fdt_unload();
8e15ce
-  return grub_error(GRUB_ERR_BAD_OS, "failed to install/update FDT");
8e15ce
+  return err;
8e15ce
 }
8e15ce
 
8e15ce
 static void
8e15ce
@@ -212,16 +234,28 @@ grub_linux_unload (void)
8e15ce
 static void *
8e15ce
 allocate_initrd_mem (int initrd_pages)
8e15ce
 {
8e15ce
-  grub_addr_t max_addr;
8e15ce
+  grub_addr_t max_addr = 0;
8e15ce
+  grub_err_t err;
8e15ce
+  void *ret;
8e15ce
 
8e15ce
-  if (grub_efi_get_ram_base (&max_addr) != GRUB_ERR_NONE)
8e15ce
-    return NULL;
8e15ce
+  err = grub_efi_get_ram_base (&max_addr);
8e15ce
+  if (err != GRUB_ERR_NONE)
8e15ce
+    {
8e15ce
+      grub_error (err, "grub_efi_get_ram_base() failed");
8e15ce
+      return NULL;
8e15ce
+    }
8e15ce
+
8e15ce
+  grub_dprintf ("linux", "max_addr: 0x%016lx, INITRD_MAX_ADDRESS_OFFSET: 0x%016llx\n",
8e15ce
+		max_addr, INITRD_MAX_ADDRESS_OFFSET);
8e15ce
 
8e15ce
   max_addr += INITRD_MAX_ADDRESS_OFFSET - 1;
8e15ce
+  grub_dprintf ("linux", "calling grub_efi_allocate_pages_real (0x%016lx, 0x%08x, EFI_ALLOCATE_MAX_ADDRESS, EFI_LOADER_DATA)", max_addr, initrd_pages);
8e15ce
 
8e15ce
-  return grub_efi_allocate_pages_real (max_addr, initrd_pages,
8e15ce
-				       GRUB_EFI_ALLOCATE_MAX_ADDRESS,
8e15ce
-				       GRUB_EFI_LOADER_DATA);
8e15ce
+  ret = grub_efi_allocate_pages_real (max_addr, initrd_pages,
8e15ce
+				      GRUB_EFI_ALLOCATE_MAX_ADDRESS,
8e15ce
+				      GRUB_EFI_LOADER_DATA);
8e15ce
+  grub_dprintf ("linux", "got 0x%016llx\n", (unsigned long long)ret);
8e15ce
+  return ret;
8e15ce
 }
8e15ce
 
8e15ce
 static grub_err_t