Attention is currently required from: Yi Chou.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79800?usp=email )
Change subject: libpayload: Move back the ttb_buffer section ......................................................................
Patch Set 3: Code-Review+2
(2 comments)
File payloads/libpayload/arch/arm64/libpayload.ldscript:
https://review.coreboot.org/c/coreboot/+/79800/comment/1610f893_312fec5b : PS3, Line 65: *(.bss.ttb_buffer) I think you might as well remove this completely. We only need this for depthcharge since the whole BSS-erasing thing only happens in depthcharge. With this new approach, it's enough to do this only in the depthcharge linker script and let the default libpayload script subsume it under `.bss.*`.
File payloads/libpayload/arch/arm64/mmu.c:
https://review.coreboot.org/c/coreboot/+/79800/comment/62a77f56_db402cb4 : PS3, Line 45: /* Don't chage the name of this variable without also changing the name in the (Would need to change this text a bit then to explain that we refer to this in the linker script for ChormeOS's depthcharge payload and to please not change the name without discussing with us. Maybe put your email address as a contact.)