x86/boot: Convert remaining uses of the legacy ALIGN

There are only two remaining standalone uses the legacy ALIGN macro.

Drop these by switching the .incbin's over to using FUNC()/END() which has
alignment handled internally.  While the incbin's aren't technically one
single function, they behave as if they are.

Finally, expand ALIGN inside the legacy ENTRY() macro in order to remove ALIGN
itself.

No functional change.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
This commit is contained in:
Andrew Cooper committed 2024-10-04 12:13:32 +01:00
1 parent d82e0e094e
commit 7fd80df2bf
1 file changed
+5 -6
+5 -6
View File
@@ -10,9 +10,8 @@
#include <asm/cpufeature.h>
#include <public/elfnote.h>
#define ALIGN .align CONFIG_FUNCTION_ALIGNMENT, CODE_FILL
#define ENTRY(name) \
ALIGN; \
.align CONFIG_FUNCTION_ALIGNMENT, CODE_FILL; \
GLOBAL(name)
#define GLOBAL(name) \
.globl name; \
@@ -764,13 +763,13 @@ trampoline_setup:
* cmdline and reloc are written in C, and linked to be 32bit PIC with
* entrypoints at 0 and using the fastcall convention.
*/
ALIGN
cmdline_parse_early:
FUNC_LOCAL(cmdline_parse_early)
.incbin "cmdline.bin"
END(cmdline_parse_early)
ALIGN
reloc:
FUNC_LOCAL(reloc)
.incbin "reloc.bin"
END(reloc)
ENTRY(trampoline_start)
#include "trampoline.S"
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.