GNUBoot/coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

libpayload/arch/arm64/mmu: Specify ttb_buffer section name explicitly

Browse source 
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>
This commit is contained in:
Yu-Ping Wu 2024-01-17 09:59:40 +08:00 committed by Felix Held
parent 462a0c0cd2
commit c01ce0f88d
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
payloads/libpayload/arch/arm64/mmu.c
View file

@ -42,10 +42,13 @@ static uint64_t *xlat_addr;
static int free_idx; 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. * We refer to the section ".bss.ttb_buffer" in the linker script for ChromeOS's depthcharge
* Please contact: jwerner@chromium.org or yich@chromium.org */ * payload. Please DO NOT change the section name without discussing with us.
static uint8_t ttb_buffer[TTB_DEFAULT_SIZE] __aligned(GRANULE_SIZE); * 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[] = { static const char * const tag_to_string[] = {
[TYPE_NORMAL_MEM] = "normal", [TYPE_NORMAL_MEM] = "normal",