summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSami Tolvanen <samitolvanen@google.com>2015-06-10 17:58:12 +0200
committerSami Tolvanen <samitolvanen@google.com>2015-06-10 17:58:12 +0200
commite82fa18d3db21cb770f21b1a4c4b00950120758d (patch)
treea49e149d6efdf165c92650ebd6d7f410ee63b4f6
parentZero blocks before BLKDISCARD (diff)
downloadandroid_bootable_recovery-e82fa18d3db21cb770f21b1a4c4b00950120758d.tar
android_bootable_recovery-e82fa18d3db21cb770f21b1a4c4b00950120758d.tar.gz
android_bootable_recovery-e82fa18d3db21cb770f21b1a4c4b00950120758d.tar.bz2
android_bootable_recovery-e82fa18d3db21cb770f21b1a4c4b00950120758d.tar.lz
android_bootable_recovery-e82fa18d3db21cb770f21b1a4c4b00950120758d.tar.xz
android_bootable_recovery-e82fa18d3db21cb770f21b1a4c4b00950120758d.tar.zst
android_bootable_recovery-e82fa18d3db21cb770f21b1a4c4b00950120758d.zip
-rw-r--r--updater/blockimg.c14
1 files changed, 9 insertions, 5 deletions
diff --git a/updater/blockimg.c b/updater/blockimg.c
index dfba7e420..5d0f1560c 100644
--- a/updater/blockimg.c
+++ b/updater/blockimg.c
@@ -39,6 +39,11 @@
#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
@@ -1278,7 +1283,8 @@ static int PerformCommandZero(CommandParameters* params) {
}
if (params->cmdname[0] == 'z') {
- // Update only for the zero command, as the erase command will call this
+ // Update only for the zero command, as the erase command will call
+ // this if DEBUG_ERASE is defined.
params->written += tgt->size;
}
@@ -1464,10 +1470,8 @@ static int PerformCommandErase(CommandParameters* params) {
struct stat st;
uint64_t blocks[2];
- // Always zero the blocks first to work around possibly flaky BLKDISCARD
- // Bug: 20881595
- if (PerformCommandZero(params) != 0) {
- goto pceout;
+ if (DEBUG_ERASE) {
+ return PerformCommandZero(params);
}
if (!params) {