From c01ce0f88dc1f52d84ecfa98f8b9cc83f09ba724 Mon Sep 17 00:00:00 2001 From: Yu-Ping Wu Date: Wed, 17 Jan 2024 09:59:40 +0800 Subject: libpayload/arch/arm64/mmu: Specify ttb_buffer section name explicitly Although a section ".bss.ttb_buffer" is created automatically for 'ttb_buffer' with the GCC option '-fdata-sections', specify the section name explicitly to make the name stand out to code readers, and to reduce the chance of accidentally changing the section name by renaming the variable. Change-Id: I2930f238f63b555c4caa65709768afa314d9cf87 Signed-off-by: Yu-Ping Wu Reviewed-on: https://review.coreboot.org/c/coreboot/+/80014 Tested-by: build bot (Jenkins) Reviewed-by: Julius Werner --- payloads/libpayload/arch/arm64/mmu.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'payloads/libpayload') diff --git a/payloads/libpayload/arch/arm64/mmu.c b/payloads/libpayload/arch/arm64/mmu.c index 9fc227dc8b..3822a891cf 100644 --- a/payloads/libpayload/arch/arm64/mmu.c +++ b/payloads/libpayload/arch/arm64/mmu.c @@ -42,10 +42,13 @@ static uint64_t *xlat_addr; static int free_idx; -/* We refer to this in the linker script for ChormeOS's depthcharge payload - * and to please not change the name without discussing with us. - * Please contact: jwerner@chromium.org or yich@chromium.org */ -static uint8_t ttb_buffer[TTB_DEFAULT_SIZE] __aligned(GRANULE_SIZE); +/* + * We refer to the section ".bss.ttb_buffer" in the linker script for ChromeOS's depthcharge + * payload. Please DO NOT change the section name without discussing with us. + * Please contact: jwerner@chromium.org or yich@chromium.org + */ +static uint8_t ttb_buffer[TTB_DEFAULT_SIZE] __aligned(GRANULE_SIZE) + __section(".bss.ttb_buffer"); static const char * const tag_to_string[] = { [TYPE_NORMAL_MEM] = "normal", -- cgit v1.2.3