Paul Menzel has uploaded this change for review.

View Change

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>
---
M util/board_status/board_status.sh
1 file changed, 5 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/41857/1
diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh
index 2e93fe2..e772238 100755
--- a/util/board_status/board_status.sh
+++ b/util/board_status/board_status.sh
@@ -431,6 +431,11 @@

echo "Getting timestamp data"
cmd_nonfatal $LOCAL "$cbmem_cmd -t" "${tmpdir}/${results}/coreboot_timestamps.txt"
+ if [ $(grep -- unknown "${tmpdir}/${results}/coreboot_timestamps.txt") ]; then
+ echo "Unknown timestamps found in 'coreboot_timestamps.txt'." \
+ "Please rebuild the utility 'cbmem'."
+ exit $EXIT_FAILURE
+ fi

if [ "$cmos_enabled" -eq 1 ]; then
echo "Verifying that nvramtool is available"

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: 1
Gerrit-Owner: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newchange