Martin Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/52965 )
Change subject: amd/cezanne: verify transfer buffer in bootblock ......................................................................
amd/cezanne: verify transfer buffer in bootblock
Verify if transfer buffer is valid before progressing further to catch invalid transfer buffer early.
Signed-off-by: Kangheui Won khwon@chromium.org Change-Id: I4c470b156944b50e581dcdee47b196f46b0993f3 Reviewed-on: https://review.coreboot.org/c/coreboot/+/52965 Reviewed-by: Felix Held felix-coreboot@felixheld.de Reviewed-by: Raul Rangel rrangel@chromium.org Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/amd/cezanne/bootblock.c 1 file changed, 7 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved Raul Rangel: Looks good to me, approved
diff --git a/src/soc/amd/cezanne/bootblock.c b/src/soc/amd/cezanne/bootblock.c index 2447de6..21635c9 100644 --- a/src/soc/amd/cezanne/bootblock.c +++ b/src/soc/amd/cezanne/bootblock.c @@ -11,6 +11,7 @@ #include <cpu/x86/tsc.h> #include <soc/iomap.h> #include <soc/southbridge.h> +#include <soc/psp_transfer.h> #include <stdint.h>
/* @@ -110,5 +111,11 @@ { u32 val = cpuid_eax(1); printk(BIOS_DEBUG, "Family_Model: %08x\n", val); + + if (CONFIG(VBOOT_STARTS_BEFORE_BOOTBLOCK)) { + verify_psp_transfer_buf(); + show_psp_transfer_info(); + } + fch_early_init(); }