Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/29670 )
Change subject: util/board_status: run dmesg with sudo ......................................................................
util/board_status: run dmesg with sudo
Newer kernels only allow root to access the kernel log buffer. In another case (cbmem) we use sudo to get past that, so we can expect sudo to be available here, too.
Change-Id: I654422992e5ba1e98a786f65d50289efbcd46602 Signed-off-by: Patrick Georgi patrick@georgi.software Reviewed-on: https://review.coreboot.org/29670 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com --- M util/board_status/board_status.sh 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh index 4acbb6c..bba3d1f 100755 --- a/util/board_status/board_status.sh +++ b/util/board_status/board_status.sh @@ -381,7 +381,7 @@ cmd_nonfatal $REMOTE "$cbmem_cmd -t" "${tmpdir}/${results}/coreboot_timestamps.txt"
echo "Getting remote dmesg" - cmd $REMOTE dmesg "${tmpdir}/${results}/kernel_log.txt" + cmd $REMOTE sudo dmesg "${tmpdir}/${results}/kernel_log.txt" else echo "Verifying that CBMEM is available" if [ $(id -u) -ne 0 ]; then