From 574d16559210ef2b17f24fa40ac4be17101a062b Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Thu, 17 Dec 2015 12:38:45 -0700 Subject: [PATCH] board_status.sh: Clean up output, show what the script is doing - Print what the script is doing so when it asks for the ssh password several times in a row, it's obvious that it's actually doing different things, not that the password failed. - Don't print the output from cbfstool - it's not useful. Change-Id: I785283475e14f242117682800c26db6b4f9f1e2c Signed-off-by: Martin Roth Reviewed-on: https://review.coreboot.org/12759 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- util/board_status/board_status.sh | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh index fce45fbbd6..578e40a673 100755 --- a/util/board_status/board_status.sh +++ b/util/board_status/board_status.sh @@ -225,7 +225,8 @@ if test ! -x build/cbfstool; then make -C util/cbfstool/ && cp util/cbfstool/cbfstool build/cbfstool fi test_cmd $LOCAL "$cbfstool_cmd" -$cbfstool_cmd build/coreboot.rom extract -n config -f ${tmpdir}/config.txt +echo "Extracting config.txt from build/coreboot.rom" +$cbfstool_cmd build/coreboot.rom extract -n config -f "${tmpdir}/config.txt" >/dev/null 2>&1 mv ${tmpdir}/config.txt ${tmpdir}/config.short.txt cp ${tmpdir}/config.short.txt ${tmpcfg} yes "" | make DOTCONFIG=${tmpcfg} oldconfig 2>/dev/null >/dev/null @@ -272,13 +273,17 @@ printf "Upstream URL: %s\n" $($getrevision -U)>> ${tmpdir}/${results}/revision.t printf "Timestamp: %s\n" "$timestamp" >> ${tmpdir}/${results}/revision.txt if [ -z "$SERIAL_DEVICE" ]; then + echo "Verifying that CBMEM is available on remote device" test_cmd $REMOTE "cbmem" + echo "Getting coreboot boot log" cmd $REMOTE "cbmem -c" "${tmpdir}/${results}/coreboot_console.txt" + echo "Getting timestamp data" cmd_nonfatal $REMOTE "cbmem -t" "${tmpdir}/${results}/coreboot_timestamps.txt" else get_serial_bootlog "$SERIAL_DEVICE" "$SERIAL_PORT_SPEED" "${tmpdir}/${results}/coreboot_console.txt" fi +echo "Getting remote dmesg" cmd $REMOTE dmesg "${tmpdir}/${results}/kernel_log.txt" #