From 1fbc1123d798137324cc8876db6386584c475da1 Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Mon, 23 Jan 2023 18:38:45 -0600 Subject: soc/amd/common/block/gfx: Use TPM-stored hash for vbios cache validation Write the SHA256 hash of the cached VBIOS data when saving to FMAP, and use it to validate the data read from FMAP on subsequent boots. Add TPM2 as a dependency to the selection of VBIOS_CACHE_IN_FMAP. BUG=b:255812886 TEST=tested with rest of patch train Change-Id: I9c8f23b000b90a1072aeb7a57d3b7b2b2bc626dc Signed-off-by: Matt DeVillier Reviewed-on: https://review.coreboot.org/c/coreboot/+/72402 Reviewed-by: Martin L Roth Tested-by: build bot (Jenkins) --- src/soc/amd/common/block/graphics/Kconfig | 2 +- src/soc/amd/common/block/graphics/graphics.c | 11 ++++++----- 2 files changed, 7 insertions(+), 6 deletions(-) (limited to 'src/soc/amd') diff --git a/src/soc/amd/common/block/graphics/Kconfig b/src/soc/amd/common/block/graphics/Kconfig index 511f304373..75b4013138 100644 --- a/src/soc/amd/common/block/graphics/Kconfig +++ b/src/soc/amd/common/block/graphics/Kconfig @@ -13,7 +13,7 @@ config SOC_AMD_COMMON_BLOCK_GRAPHICS_ATIF config SOC_AMD_GFX_CACHE_VBIOS_IN_FMAP bool "Support for caching modified VBIOS tables in flash" - depends on SOC_AMD_COMMON_BLOCK_GRAPHICS && CHROMEOS && RUN_FSP_GOP + depends on SOC_AMD_COMMON_BLOCK_GRAPHICS && CHROMEOS && RUN_FSP_GOP && TPM2 default n help Enable support for flash based VBIOS cache. diff --git a/src/soc/amd/common/block/graphics/graphics.c b/src/soc/amd/common/block/graphics/graphics.c index 843bf787ee..70e3758b08 100644 --- a/src/soc/amd/common/block/graphics/graphics.c +++ b/src/soc/amd/common/block/graphics/graphics.c @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -234,6 +235,9 @@ static void write_vbios_cache_to_fmap(void *unused) VBIOS_CACHE_FMAP_SIZE) != VBIOS_CACHE_FMAP_SIZE) printk(BIOS_ERR, "Failed to save vbios data to flash; rdev_writeat() failed.\n"); + /* save data hash to TPM NVRAM for validation on subsequent boots */ + vbios_cache_update_hash(vbios_data, VBIOS_CACHE_FMAP_SIZE); + printk(BIOS_SPEW, "VBIOS cache successfully written to FMAP.\n"); } @@ -254,14 +258,11 @@ void vbios_load_from_cache(void) /* * Return true if VBIOS cache data is valid * - * For now, just compare first 2 bytes of data - * TODO: replace with TPM hash verification once implemented + * Compare hash of data with hash stored in TPM NVRAM */ bool vbios_cache_is_valid(void) { - bool is_valid = vbios_data[0] == 0x55 && vbios_data[1] == 0xaa; - printk(BIOS_SPEW, "VBIOS cache is %s\n", is_valid ? "valid" : "invalid"); - return is_valid; + return vbios_cache_verify_hash(vbios_data, VBIOS_CACHE_FMAP_SIZE) == CB_SUCCESS; } BOOT_STATE_INIT_ENTRY(BS_PRE_DEVICE, BS_ON_EXIT, read_vbios_cache_from_fmap, NULL); -- cgit v1.2.3