Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/82230?usp=email )
Change subject: commonlib/timestamp_serialized: Define VB_AUXFW_SYNC_DONE timestamp ......................................................................
commonlib/timestamp_serialized: Define VB_AUXFW_SYNC_DONE timestamp
Define a new timestamp to identify the completion of Auxiliary Firmware Sync. Without that, it gets accounted into a different timestamp ID in a misleading way.
BUG=None TEST=Build Brox BIOS image and boot to OS. Confirm the timestamp is recorded in cbmem.
Change-Id: Icd01c68a5848e2aed7bbdcc794987bc780e78dab Signed-off-by: Karthikeyan Ramasubramanian kramasub@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/82230 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Julius Werner jwerner@chromium.org Reviewed-by: Shelley Chen shchen@google.com --- M src/commonlib/include/commonlib/timestamp_serialized.h 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: Julius Werner: Looks good to me, approved Shelley Chen: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/commonlib/include/commonlib/timestamp_serialized.h b/src/commonlib/include/commonlib/timestamp_serialized.h index fc7dba8..d34c8ae 100644 --- a/src/commonlib/include/commonlib/timestamp_serialized.h +++ b/src/commonlib/include/commonlib/timestamp_serialized.h @@ -165,6 +165,7 @@ TS_VB_EC_VBOOT_DONE = 1030, TS_VB_STORAGE_INIT_DONE = 1040, TS_VB_READ_KERNEL_DONE = 1050, + TS_VB_AUXFW_SYNC_DONE = 1060, TS_VB_VBOOT_DONE = 1100,
TS_KERNEL_START = 1101, @@ -358,6 +359,7 @@ TS_NAME_DEF(TS_VB_EC_VBOOT_DONE, 0, "finished EC verification"), TS_NAME_DEF(TS_VB_STORAGE_INIT_DONE, 0, "finished storage device initialization"), TS_NAME_DEF(TS_VB_READ_KERNEL_DONE, 0, "finished reading kernel from disk"), + TS_NAME_DEF(TS_VB_AUXFW_SYNC_DONE, 0, "finished AuxFW Sync"), TS_NAME_DEF(TS_VB_VBOOT_DONE, 0, "finished vboot kernel verification"),
TS_NAME_DEF(TS_KERNEL_START, 0, "jumping to kernel"),