]> xenbits.xensource.com Git - people/aperard/xen-unstable.git/commitdiff
Arm: drop bogus ALIGN() from linker script
authorJan Beulich <jbeulich@suse.com>
Thu, 15 Jun 2023 09:00:22 +0000 (11:00 +0200)
committerJan Beulich <jbeulich@suse.com>
Thu, 15 Jun 2023 09:00:22 +0000 (11:00 +0200)
Having ALIGN() inside a section definition usually makes sense only with
a label definition following (an exception case is a few lines out of
context, where cache line sharing is intended to be avoided).
Constituents of .bss.page_aligned need to specify their own alignment
correctly anyway, or else they're susceptible to link order changing.
This requirement is already met: Arm-specific code has no such object,
while common (EFI) code has another one. That one has suitable alignment
specified.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>
xen/arch/arm/xen.lds.S

index 6ca3caefe6076e77bc67857702924c06ddea7d62..be58c2c395147992868a332cf3ffda21aa92387f 100644 (file)
@@ -199,7 +199,6 @@ SECTIONS
   .bss : {                     /* BSS */
        __bss_start = .;
        *(.bss.stack_aligned)
-       . = ALIGN(PAGE_SIZE);
        *(.bss.page_aligned)
        . = ALIGN(PAGE_SIZE);
        __per_cpu_start = .;