diff mbox series

x86: drop map-low-16Mb leftovers

Message ID 71348420-c60a-4d5c-9548-8e5f24f82c76@suse.com (mailing list archive)
State New
Headers show
Series x86: drop map-low-16Mb leftovers | expand

Commit Message

Jan Beulich Aug. 29, 2024, 11:47 a.m. UTC 
Prior work has fully eliminated that hardcoded boundary. Drop both the
linker script assertion (the upper bound is now the stubs area) and the
artificial extending of xen.efi's image size.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

Comments

Andrew Cooper Aug. 29, 2024, 12:01 p.m. UTC | #1 
On 29/08/2024 12:47 pm, Jan Beulich wrote:
> Prior work has fully eliminated that hardcoded boundary.

I'd cite e.g. commit cbabbc9f5659 ("x86/boot: Size the boot/directmap
mappings dynamically") here.

>  Drop both the
> linker script assertion (the upper bound is now the stubs area) and the
> artificial extending of xen.efi's image size.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>

Sorry I didn't drop the assertion at the time.
diff mbox series

Patch

--- a/xen/arch/x86/xen.lds.S
+++ b/xen/arch/x86/xen.lds.S
@@ -359,14 +359,6 @@  SECTIONS
 
   DWARF2_DEBUG_SECTIONS
 
-#ifdef EFI
-  /* Trick the linker into setting the image size to no less than 16Mb. */
-  __image_end__ = .;
-  .pad ALIGN(__section_alignment__) : {
-    . = __image_end__ < __image_base__ + MB(16) ? ALIGN(MB(16)) : .;
-  }
-#endif
-
 #ifdef CONFIG_HYPERV_GUEST
   hv_hcall_page = ABSOLUTE(HV_HCALL_PAGE - XEN_VIRT_START + __XEN_VIRT_START);
 #endif
@@ -429,6 +421,3 @@  ASSERT((trampoline_end - trampoline_star
     "not enough room for trampoline and mbi data")
 ASSERT((wakeup_stack - wakeup_stack_start) >= WAKEUP_STACK_MIN,
     "wakeup stack too small")
-
-/* Plenty of boot code assumes that Xen isn't larger than 16M. */
-ASSERT(_end - _start <= MB(16), "Xen too large for early-boot assumptions")