GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

commonlib/timestamp_serialized: Add comment explaining "ignore for x86" 

BUG=b:240624460
TEST=None

Signed-off-by: Reka Norman <rekanorman@chromium.org>
Change-Id: I8542c9bb624a366bc1bb01f6eae66ba97520d19c
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66381
Reviewed-by: Julius Werner <jwerner@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Reka Norman 2022-08-03 10:22:32 +10:00 committed by Martin L Roth
parent d454f86ed8
commit abbfa555b0
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/commonlib/include/commonlib/timestamp_serialized.h
View File

@ -199,6 +199,14 @@ static const struct timestamp_id_to_name {
TS_NAME_DEF(TS_BOOTBLOCK_END, 0, "end of bootblock"),
TS_NAME_DEF(TS_COPYROM_START, TS_COPYROM_END, "starting to load romstage"),
TS_NAME_DEF(TS_COPYROM_END, 0, "finished loading romstage"),
/*
* "ignore for x86": On platforms with memory-mapped flash, it's
* impossible to separate loading times from decompression times because
* the flash accesses happen in the background as the decompression is
* running. So it makes more sense to consider the total loading +
* decompression time by looking at the timestamps before and after the
* decompression timestamps.
*/
TS_NAME_DEF(TS_ULZMA_START, TS_ULZMA_END, "starting LZMA decompress (ignore for x86)"),
TS_NAME_DEF(TS_ULZMA_END, 0, "finished LZMA decompress (ignore for x86)"),
TS_NAME_DEF(TS_ULZ4F_START, TS_ULZ4F_END, "starting LZ4 decompress (ignore for x86)"),