From 84532dae1424e97909400c1e8da88c6909cfbaac Mon Sep 17 00:00:00 2001 From: MAULIK V VAGHELA Date: Wed, 25 Aug 2021 16:41:23 +0530 Subject: [PATCH] soc/intel/alderlake: Change VBOOT_HASH_BLOCK_SIZE to 4 KiB Default VBOOT_HASH_BLOCK_SIZE is 1 KiB and increasing it to 4 KiB helps in improving overall boot time since it reduces hashing and body loading time. Increasing it over 4 KiB doesn't result in significant improvement, thus keeping the value at 4 KiB as of now. Timing data: Note that before Data is with 1 KiB block size. |------------------------------------------------------| | Stage | Block Size | Before | After | |finished loading body| 4 KiB | 205,187 | 189,947 | |finished loading body| 8 KiB | 205,187 | 188,708 | |finished loading body| 16 KiB | 205,187 | 188,085 | |finished loading body| 32 KiB | 205,187 | 187,793 | |------------------------------------------------------| BUG=b:188577893 BRANCH=None TEST=Boot time for Brya improves by 20 - 25 msec Change-Id: I9222761c7d58e4a370d3a41c651b6c169599d792 Signed-off-by: MAULIK V VAGHELA Reviewed-on: https://review.coreboot.org/c/coreboot/+/57141 Tested-by: build bot (Jenkins) Reviewed-by: Tim Wawrzynczak --- src/soc/intel/alderlake/Kconfig | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/soc/intel/alderlake/Kconfig b/src/soc/intel/alderlake/Kconfig index e2116c787e..cdf127f36d 100644 --- a/src/soc/intel/alderlake/Kconfig +++ b/src/soc/intel/alderlake/Kconfig @@ -260,6 +260,13 @@ config VBOOT select VBOOT_VBNV_CMOS_BACKUP_TO_FLASH select VBOOT_X86_SHA256_ACCELERATION +# Default hash block size is 1KiB. Increasing it to 4KiB to improve +# hashing time as well as read time. This helps in improving +# boot time for Alder Lake. +config VBOOT_HASH_BLOCK_SIZE + hex + default 0x1000 + config CBFS_SIZE default 0x200000