Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/85218?usp=email )
Change subject: util/cbfstool: Add eventLog support for ELOG_TYPE_FW_CSE_SYNC ......................................................................
util/cbfstool: Add eventLog support for ELOG_TYPE_FW_CSE_SYNC
This patch introduces logging for the ELOG_TYPE_FW_CSE_SYNC event. This event logs data related to CSE synchronization, along with the relevant boot stage information.
BUG=b:305898363 TEST=boot verified on google/rex0 and google/rex64
Change-Id: I4d6d3c4e07ab8677feb6a8acf8d4c6604ab704b8 Signed-off-by: Dinesh Gehlot digehlot@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/85218 Reviewed-by: Karthik Ramasubramanian kramasub@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M util/cbfstool/eventlog.c 1 file changed, 15 insertions(+), 1 deletion(-)
Approvals: Karthik Ramasubramanian: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/util/cbfstool/eventlog.c b/util/cbfstool/eventlog.c index 5e76df2..e3fccde 100644 --- a/util/cbfstool/eventlog.c +++ b/util/cbfstool/eventlog.c @@ -170,6 +170,7 @@ {ELOG_TYPE_PSR_DATA_LOST, "PSR data lost"}, {ELOG_TYPE_FW_SPLASH_SCREEN, "Firmware Splash Screen"}, {ELOG_TYPE_FW_LATE_SOL, "Late Sign of Life "}, + {ELOG_TYPE_FW_CSE_SYNC, "Firmware CSE sync"}, {ELOG_TYPE_EOL, "End of log"}, };
@@ -489,6 +490,13 @@ {0, NULL}, };
+ static const struct valstr cse_sync_path_types[] = { + {ELOG_FW_EARLY_CSE_SYNC, "Early CSE Sync"}, + {ELOG_FW_LATE_CSE_SYNC, "Late CSE Sync"}, + {ELOG_FW_CSE_SYNC_AT_PAYLOAD, "CSE Sync at Payload"}, + {0, NULL}, + }; + size_t elog_type_to_min_size[] = { [ELOG_TYPE_LOG_CLEAR] = sizeof(uint16_t), [ELOG_TYPE_BOOT] = sizeof(uint32_t), @@ -510,7 +518,8 @@ [ELOG_TYPE_FW_EARLY_SOL] = sizeof(uint8_t), [ELOG_TYPE_PSR_DATA_BACKUP] = sizeof(uint8_t), [ELOG_TYPE_FW_SPLASH_SCREEN] = sizeof(uint8_t), - [ELOG_TYPE_FW_LATE_SOL] = sizeof(uint8_t), + [ELOG_TYPE_FW_LATE_SOL] = sizeof(uint8_t), + [ELOG_TYPE_FW_CSE_SYNC] = sizeof(uint8_t), [0xff] = 0, };
@@ -682,6 +691,11 @@ eventlog_printf("%s", val2str(*sol_event, late_sol_path_types)); break; } + case ELOG_TYPE_FW_CSE_SYNC: { + const uint8_t *cse_event = event_get_data(event); + eventlog_printf("%s", val2str(*cse_event, cse_sync_path_types)); + break; + } default: break; }