diff options
| author | Yu-Ping Wu <yupingso@chromium.org> | 2024-01-17 09:59:40 +0800 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2024-01-22 12:20:58 +0000 |
| commit | c01ce0f88dc1f52d84ecfa98f8b9cc83f09ba724 (patch) | |
| tree | e6ec6995781694e9d2fcafb72442a19e73fcdf73 | |
| parent | 462a0c0cd2e2bde287a5d56ecafe146115b64088 (diff) | |
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 <yupingso@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80014
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
| -rw-r--r-- | payloads/libpayload/arch/arm64/mmu.c | 11 |
1 files changed, 7 insertions, 4 deletions
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", |