Yi Chou has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/79800?usp=email )
Change subject: Move back the ttb_buffer section ......................................................................
Move back the ttb_buffer section
If the move it into the .ttb_buffer section, the LOAD flag would not be set. We should move it back to .bss.ttb_buffer section.
BUG=b:248610274 TEST=Boot the device. BRANCH=none
Cq-Depend: chromium:5173448 Change-Id: I9bb08878dd4be01d9ed3f96933f774dd6296f76e Signed-off-by: Yi Chou yich@google.com --- M payloads/libpayload/arch/arm64/libpayload.ldscript M payloads/libpayload/arch/arm64/mmu.c 2 files changed, 5 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/79800/1
diff --git a/payloads/libpayload/arch/arm64/libpayload.ldscript b/payloads/libpayload/arch/arm64/libpayload.ldscript index c9881f7..e3e6faa 100644 --- a/payloads/libpayload/arch/arm64/libpayload.ldscript +++ b/payloads/libpayload/arch/arm64/libpayload.ldscript @@ -62,7 +62,7 @@ }
.bss : { - *(.ttb_buffer) + *(.bss.ttb_buffer)
_bss = .; *(.sbss) diff --git a/payloads/libpayload/arch/arm64/mmu.c b/payloads/libpayload/arch/arm64/mmu.c index 5865b1b..ae7ac3a 100644 --- a/payloads/libpayload/arch/arm64/mmu.c +++ b/payloads/libpayload/arch/arm64/mmu.c @@ -41,8 +41,10 @@ static uint64_t *xlat_addr;
static int free_idx; -static uint8_t ttb_buffer[TTB_DEFAULT_SIZE] __aligned(GRANULE_SIZE) - __attribute__((__section__(".ttb_buffer"))); + +/* Don't chage the name of this variable without also changing the name in the + * linker script. */ +static uint8_t ttb_buffer[TTB_DEFAULT_SIZE] __aligned(GRANULE_SIZE);
static const char * const tag_to_string[] = { [TYPE_NORMAL_MEM] = "normal",