Patrick Georgi submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved
util/board-status: Reject logs with unknown timestamps

Check the output of `cbmem -t` for unknown timestamps. If present, ask
the user to rebuild `cbmem`.

Change-Id: Ief7aa1a698f10d9721964ad1bee057fcd9f4aa40
Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41857
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
---
M util/board_status/board_status.sh
1 file changed, 6 insertions(+), 0 deletions(-)

diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh
index dd3d8f2..31c3e7c 100755
--- a/util/board_status/board_status.sh
+++ b/util/board_status/board_status.sh
@@ -459,6 +459,12 @@
exit $EXIT_FAILURE
fi

+if [ $(grep -- unknown "${tmpdir}/${results}/coreboot_timestamps.txt") ]; then
+ echo "Unknown timestamps found in 'coreboot_timestamps.txt'." \
+ "Please rebuild the 'cbmem' utility and try again."
+ exit $EXIT_FAILURE
+fi
+
#
# Finish up.
#

To view, visit change 41857. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ief7aa1a698f10d9721964ad1bee057fcd9f4aa40
Gerrit-Change-Number: 41857
Gerrit-PatchSet: 5
Gerrit-Owner: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged