summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJack Rosenthal <jrosenth@chromium.org>2021-10-21 13:11:56 -0600
committerAngel Pons <th3fanbus@gmail.com>2021-10-27 09:12:31 +0000
commitb3df1390dc8949cf909d2a037acfc56793bb3380 (patch)
treeb047bd9fc765062aed12580c924a13831d7855f9
parent8d8281c3d627da489499dc127de2f728912064de (diff)
downloadflashrom-b3df1390dc8949cf909d2a037acfc56793bb3380.tar.gz
flashrom-b3df1390dc8949cf909d2a037acfc56793bb3380.tar.bz2
flashrom-b3df1390dc8949cf909d2a037acfc56793bb3380.zip
flashrom_tester: Use elogtool to list firmware eventlog
Mosys is dropping the eventlog command, in favor of the elogtool command provided in coreboot. The output is compatible with what mosys used to output. Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I8c4be82fed28b6a19746e6b93fafce23bd8ede5d Reviewed-on: https://review.coreboot.org/c/flashrom/+/58527 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Ricardo Quesada <ricardoq@google.com> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
-rw-r--r--util/flashrom_tester/src/cros_sysinfo.rs8
-rw-r--r--util/flashrom_tester/src/tests.rs6
2 files changed, 7 insertions, 7 deletions
diff --git a/util/flashrom_tester/src/cros_sysinfo.rs b/util/flashrom_tester/src/cros_sysinfo.rs
index ddb080265..4d1b347fd 100644
--- a/util/flashrom_tester/src/cros_sysinfo.rs
+++ b/util/flashrom_tester/src/cros_sysinfo.rs
@@ -61,13 +61,13 @@ pub fn bios_info() -> IoResult<String> {
}
pub fn eventlog_list() -> Result<String, std::io::Error> {
- mosys_dispatch(&["eventlog", "list"])
+ elogtool_dispatch(&["list"])
}
-fn mosys_dispatch<S: AsRef<OsStr> + Debug>(args: &[S]) -> IoResult<String> {
- info!("mosys_dispatch() running: /usr/sbin/mosys {:?}", args);
+fn elogtool_dispatch<S: AsRef<OsStr> + Debug>(args: &[S]) -> IoResult<String> {
+ info!("elogtool_dispatch() running: /usr/bin/elogtool {:?}", args);
- let output = Command::new("/usr/sbin/mosys")
+ let output = Command::new("/usr/bin/elogtool")
.args(args)
.stdin(Stdio::null())
.output()?;
diff --git a/util/flashrom_tester/src/tests.rs b/util/flashrom_tester/src/tests.rs
index 9ef98e5c7..86c0da5bd 100644
--- a/util/flashrom_tester/src/tests.rs
+++ b/util/flashrom_tester/src/tests.rs
@@ -234,13 +234,13 @@ fn lock_test(env: &mut TestEnv) -> TestResult {
fn elog_sanity_test(env: &mut TestEnv) -> TestResult {
// Check that the elog contains *something*, as an indication that Coreboot
- // is actually able to write to the Flash. Because this invokes mosys on the
- // host, it doesn't make sense to run for other chips.
+ // is actually able to write to the Flash. Because this invokes elogtool on
+ // the host, it doesn't make sense to run for other chips.
if env.chip_type() != FlashChip::HOST {
info!("Skipping ELOG sanity check for non-host chip");
return Ok(());
}
- // mosys reads the flash, it should be back in the golden state
+ // elogtool reads the flash, it should be back in the golden state
env.ensure_golden()?;
// Output is one event per line, drop empty lines in the interest of being defensive.
let event_count = cros_sysinfo::eventlog_list()?