diff options
author | Tianjie Xu <xunchang@google.com> | 2019-07-10 22:59:32 +0200 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2019-07-10 22:59:32 +0200 |
commit | f6e8941ee978f16e161fe1d8443ab72828d1433a (patch) | |
tree | 15e028fddab0693f79f8ac7c789a5fc642a2c3b9 /recovery.cpp | |
parent | Merge "Change the symlink to system_ext" am: c992962503 (diff) | |
parent | Merge "Create a fallback to install from fuse if mmap fails" (diff) | |
download | android_bootable_recovery-f6e8941ee978f16e161fe1d8443ab72828d1433a.tar android_bootable_recovery-f6e8941ee978f16e161fe1d8443ab72828d1433a.tar.gz android_bootable_recovery-f6e8941ee978f16e161fe1d8443ab72828d1433a.tar.bz2 android_bootable_recovery-f6e8941ee978f16e161fe1d8443ab72828d1433a.tar.lz android_bootable_recovery-f6e8941ee978f16e161fe1d8443ab72828d1433a.tar.xz android_bootable_recovery-f6e8941ee978f16e161fe1d8443ab72828d1433a.tar.zst android_bootable_recovery-f6e8941ee978f16e161fe1d8443ab72828d1433a.zip |
Diffstat (limited to 'recovery.cpp')
-rw-r--r-- | recovery.cpp | 28 |
1 files changed, 24 insertions, 4 deletions
diff --git a/recovery.cpp b/recovery.cpp index b18a8e74a..97ca0a504 100644 --- a/recovery.cpp +++ b/recovery.cpp @@ -84,6 +84,8 @@ const char* reason = nullptr; * * The arguments which may be supplied in the recovery.command file: * --update_package=path - verify install an OTA package file + * --install_with_fuse - install the update package with FUSE. This allows installation of large + * packages on LP32 builds. Since the mmap will otherwise fail due to out of memory. * --wipe_data - erase user data (and cache), then reboot * --prompt_and_wipe_data - prompt the user that data is corrupt, with their consent erase user * data (and cache), then reboot @@ -577,6 +579,7 @@ Device::BuiltinAction start_recovery(Device* device, const std::vector<std::stri static constexpr struct option OPTIONS[] = { { "fastboot", no_argument, nullptr, 0 }, { "fsck_unshare_blocks", no_argument, nullptr, 0 }, + { "install_with_fuse", no_argument, nullptr, 0 }, { "just_exit", no_argument, nullptr, 'x' }, { "locale", required_argument, nullptr, 0 }, { "prompt_and_wipe_data", no_argument, nullptr, 0 }, @@ -597,6 +600,7 @@ Device::BuiltinAction start_recovery(Device* device, const std::vector<std::stri }; const char* update_package = nullptr; + bool install_with_fuse = false; // memory map the update package by default. bool should_wipe_data = false; bool should_prompt_and_wipe_data = false; bool should_wipe_cache = false; @@ -632,6 +636,8 @@ Device::BuiltinAction start_recovery(Device* device, const std::vector<std::stri std::string option = OPTIONS[option_index].name; if (option == "fsck_unshare_blocks") { fsck_unshare_blocks = true; + } else if (option == "install_with_fuse") { + install_with_fuse = true; } else if (option == "locale" || option == "fastboot") { // Handled in recovery_main.cpp } else if (option == "prompt_and_wipe_data") { @@ -737,10 +743,24 @@ Device::BuiltinAction start_recovery(Device* device, const std::vector<std::stri } else { ensure_path_mounted(update_package); } - // TODO(xunchang) install package from fuse for large packages on ILP32 builds. - auto package = Package::CreateMemoryPackage( - update_package, std::bind(&RecoveryUI::SetProgress, ui, std::placeholders::_1)); - status = InstallPackage(package.get(), update_package, should_wipe_cache, retry_count, ui); + + if (install_with_fuse) { + LOG(INFO) << "Installing package " << update_package << " with fuse"; + status = InstallWithFuseFromPath(update_package, ui); + } else if (auto memory_package = Package::CreateMemoryPackage( + update_package, + std::bind(&RecoveryUI::SetProgress, ui, std::placeholders::_1)); + memory_package != nullptr) { + status = InstallPackage(memory_package.get(), update_package, should_wipe_cache, + retry_count, ui); + } else { + // We may fail to memory map the package on 32 bit builds for packages with 2GiB+ size. + // In such cases, we will try to install the package with fuse. This is not the default + // installation method because it introduces a layer of indirection from the kernel space. + LOG(WARNING) << "Failed to memory map package " << update_package + << "; falling back to install with fuse"; + status = InstallWithFuseFromPath(update_package, ui); + } if (status != INSTALL_SUCCESS) { ui->Print("Installation aborted.\n"); |