commit 64161b4d52c8d313ade6427cd290b90ff1e26581
author David Hendricks <dhendrix@chromium.org> Wed Dec 17 13:11:22 2014 -0800
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Thu Dec 18 02:04:37 2014 +0000
tree c3c502cdaba00d0a59e60c1f947d26e11c59cd83
parent d08fcc6cb47a1be4a90a8ec4f4e3eb8cd81695b3

saft: Don't invoke mosys with -f

The -f flag forces mosys to ignore the global IPC lock and sanity
checking which can be really bad if stuff like flashrom is running
at the same time.

If -f is needed due to flashrom (or something) holding the lock, then
that may indicate another issue and should be addressed.

BUG=chrome-os-partner:34430
BRANCH=none
TEST=none

Change-Id: I860add89c884e238c960f706ee2ae05fb0282805
Reviewed-on: https://chromium-review.googlesource.com/236366
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: David Hendricks <dhendrix@chromium.org>
Tested-by: David Hendricks <dhendrix@chromium.org>

client/cros/faft/utils/saft_flashrom_util.py

diff --git a/client/cros/faft/utils/saft_flashrom_util.py b/client/cros/faft/utils/saft_flashrom_util.py
index 19bc3a1..6c6ea82 100644
--- a/client/cros/faft/utils/saft_flashrom_util.py
+++ b/client/cros/faft/utils/saft_flashrom_util.py
@@ -79,7 +79,7 @@
         "<name>"="value" pairs, all this output is passed to the caller.
         """
 
-        mosys_cmd = 'mosys -f -k eeprom map %s' % file_name
+        mosys_cmd = 'mosys -k eeprom map %s' % file_name
         return self.os_if.run_shell_command_get_output(mosys_cmd)
 
     def _line_to_dictionary(self, line):