summaryrefslogtreecommitdiffstats
path: root/updater/blockimg.cpp
diff options
context:
space:
mode:
authorandroid-build-team Robot <android-build-team-robot@google.com>2018-05-18 11:38:29 +0200
committerandroid-build-team Robot <android-build-team-robot@google.com>2018-05-18 11:38:29 +0200
commit10107b09c0a9abb17252bdbb6794120cdf96427a (patch)
tree28c8a0c6fda7a46acf1e39b25f58253843f71ee1 /updater/blockimg.cpp
parentSnap for 4778411 from baa4f0d2af970e014bb7b48f7dc8a5b92eb29bee to qt-release (diff)
parentMerge "recovery: Reset optind to 1 after getopt" am: 38d90316b2 am: 25d31c9f9c (diff)
downloadandroid_bootable_recovery-10107b09c0a9abb17252bdbb6794120cdf96427a.tar
android_bootable_recovery-10107b09c0a9abb17252bdbb6794120cdf96427a.tar.gz
android_bootable_recovery-10107b09c0a9abb17252bdbb6794120cdf96427a.tar.bz2
android_bootable_recovery-10107b09c0a9abb17252bdbb6794120cdf96427a.tar.lz
android_bootable_recovery-10107b09c0a9abb17252bdbb6794120cdf96427a.tar.xz
android_bootable_recovery-10107b09c0a9abb17252bdbb6794120cdf96427a.tar.zst
android_bootable_recovery-10107b09c0a9abb17252bdbb6794120cdf96427a.zip
Diffstat (limited to 'updater/blockimg.cpp')
-rw-r--r--updater/blockimg.cpp18
1 files changed, 7 insertions, 11 deletions
diff --git a/updater/blockimg.cpp b/updater/blockimg.cpp
index 156a82939..236644e7f 100644
--- a/updater/blockimg.cpp
+++ b/updater/blockimg.cpp
@@ -132,8 +132,7 @@ static bool FsyncDir(const std::string& dirname) {
return true;
}
-// Update the last command index in the last_command_file if the current command writes to the
-// stash either explicitly or implicitly.
+// Update the last executed command index in the last_command_file.
static bool UpdateLastCommandIndex(int command_index, const std::string& command_string) {
const std::string& last_command_file = Paths::Get().last_command_file();
std::string last_command_tmp = last_command_file + ".tmp";
@@ -1161,10 +1160,6 @@ static int LoadSrcTgtVersion3(CommandParameters& params, RangeSet& tgt, size_t*
return -1;
}
- if (!UpdateLastCommandIndex(params.cmdindex, params.cmdline)) {
- LOG(WARNING) << "Failed to update the last command file.";
- }
-
params.stashed += *src_blocks;
// Can be deleted when the write has completed.
if (!stash_exists) {
@@ -1275,10 +1270,6 @@ static int PerformCommandStash(CommandParameters& params) {
LOG(INFO) << "stashing " << blocks << " blocks to " << id;
int result = WriteStash(params.stashbase, id, blocks, params.buffer, false, nullptr);
if (result == 0) {
- if (!UpdateLastCommandIndex(params.cmdindex, params.cmdline)) {
- LOG(WARNING) << "Failed to update the last command file.";
- }
-
params.stashed += blocks;
}
return result;
@@ -1701,7 +1692,7 @@ static Value* PerformBlockImageUpdate(const char* name, State* state,
params.createdstash = res;
// When performing an update, save the index and cmdline of the current command into
- // the last_command_file if this command writes to the stash either explicitly of implicitly.
+ // the last_command_file.
// Upon resuming an update, read the saved index first; then
// 1. In verification mode, check if the 'move' or 'diff' commands before the saved index has
// the expected target blocks already. If not, these commands cannot be skipped and we need
@@ -1797,6 +1788,11 @@ static Value* PerformBlockImageUpdate(const char* name, State* state,
PLOG(ERROR) << "fsync failed";
goto pbiudone;
}
+
+ if (!UpdateLastCommandIndex(params.cmdindex, params.cmdline)) {
+ LOG(WARNING) << "Failed to update the last command file.";
+ }
+
fprintf(cmd_pipe, "set_progress %.4f\n", static_cast<double>(params.written) / total_blocks);
fflush(cmd_pipe);
}