[coreboot-gerrit] Change in coreboot[master]: util/board_status: run dmesg with sudo

Patrick Georgi (Code Review) gerrit at coreboot.org
Sat Nov 17 16:53:53 CET 2018


Patrick Georgi has uploaded this change for review. ( 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 at georgi.software>
---
M util/board_status/board_status.sh
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/29670/1

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

-- 
To view, visit https://review.coreboot.org/29670
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I654422992e5ba1e98a786f65d50289efbcd46602
Gerrit-Change-Number: 29670
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181117/538a4859/attachment.html>


More information about the coreboot-gerrit mailing list