commit 2fd12979eac8987d4aaa89b56c508a710450c33d
author Sami Tolvanen <samitolvanen@google.com> Tue Jun 09 21:28:46 2015 +0000
committer Android Git Automerger <android-git-automerger@android.com> Tue Jun 09 21:28:46 2015 +0000
tree 46d64e674bd137408ae0cc04eb6ea74075af2050
parent 7baa13f6d6ffa77e35e3596018efa5ca9a1febf7
parent 69608fe7548727ee765c89f69a6861479f391487

am 69608fe7: Merge "Zero blocks before BLKDISCARD"

* commit '69608fe7548727ee765c89f69a6861479f391487':
  Zero blocks before BLKDISCARD

updater/blockimg.c

diff --git a/updater/blockimg.c b/updater/blockimg.c
index 5d0f156..dfba7e4 100644
--- a/updater/blockimg.c
+++ b/updater/blockimg.c
@@ -39,11 +39,6 @@
 
 #define BLOCKSIZE 4096
 
-// Set this to 0 to interpret 'erase' transfers to mean do a
-// BLKDISCARD ioctl (the normal behavior).  Set to 1 to interpret
-// erase to mean fill the region with zeroes.
-#define DEBUG_ERASE  0
-
 #ifndef BLKDISCARD
 #define BLKDISCARD _IO(0x12,119)
 #endif
@@ -1283,8 +1278,7 @@
     }
 
     if (params->cmdname[0] == 'z') {
-        // Update only for the zero command, as the erase command will call
-        // this if DEBUG_ERASE is defined.
+        // Update only for the zero command, as the erase command will call this
         params->written += tgt->size;
     }
 
@@ -1470,8 +1464,10 @@
     struct stat st;
     uint64_t blocks[2];
 
-    if (DEBUG_ERASE) {
-        return PerformCommandZero(params);
+    // Always zero the blocks first to work around possibly flaky BLKDISCARD
+    // Bug: 20881595
+    if (PerformCommandZero(params) != 0) {
+        goto pceout;
     }
 
     if (!params) {