Yi Chou has uploaded this change for review.
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",
To view, visit change 79800. To unsubscribe, or for help writing mail filters, visit settings.