commit fa137fc83dd3ae0bd772ca7b23197e87383ba82a
author Fabio Utzig <utzig@apache.org> Thu Nov 23 20:01:02 2017 -0200
committer David Brown <davidb@davidb.org> Thu Dec 21 13:36:34 2017 -0700
tree c4fadebb31bf990e940d42a1c42975dcaf1e088e
parent f5c895e3fc875f963a699088537e179e1a7ace5d

Add option to disable verifying contents before writes

At the moment writes always checking that they are being
performed on an erased section of memory. This patch enables
a test to disable that, to enable looking for other error
causes.

Signed-off-by: Fabio Utzig <utzig@apache.org>

sim/simflash/src/lib.rs

diff --git a/sim/simflash/src/lib.rs b/sim/simflash/src/lib.rs
index d4817c1..c38baac 100644
--- a/sim/simflash/src/lib.rs
+++ b/sim/simflash/src/lib.rs
@@ -41,6 +41,8 @@
     fn add_bad_region(&mut self, offset: usize, len: usize, rate: f32) -> Result<()>;
     fn reset_bad_regions(&mut self);
 
+    fn set_verify_writes(&mut self, enable: bool);
+
     fn sector_iter(&self) -> SectorIter;
     fn device_size(&self) -> usize;
 }
@@ -69,6 +71,7 @@
     bad_region: Vec<(usize, usize, f32)>,
     // Alignment required for writes.
     align: usize,
+    verify_writes: bool,
 }
 
 impl SimFlash {
@@ -85,6 +88,7 @@
             sectors: sectors,
             bad_region: Vec::new(),
             align: align,
+            verify_writes: true,
         }
     }
 
@@ -176,7 +180,7 @@
         }
 
         for (i, x) in &mut self.write_safe[offset .. offset + payload.len()].iter_mut().enumerate() {
-            if !(*x) {
+            if self.verify_writes && !(*x) {
                 panic!("Write to unerased location at 0x{:x}", offset + i);
             }
             *x = false;
@@ -215,6 +219,10 @@
         self.bad_region.clear();
     }
 
+    fn set_verify_writes(&mut self, enable: bool) {
+        self.verify_writes = enable;
+    }
+
     /// An iterator over each sector in the device.
     fn sector_iter(&self) -> SectorIter {
         SectorIter {