Moving recovery resources from /system to /vendor

This change is part of a topic that moves the recovery resources from the
system partition to the vendor partition, if it exists, or the vendor directory
on the system partition otherwise. The recovery resources are moving from the
system image to the vendor partition so that a single system image may be used
with either an A/B or a non-A/B vendor image. The topic removes a delta in the
system image that prevented such reuse in the past.

The recovery resources that are moving are involved with updating the recovery
partition after an update. In a non-A/B configuration, the system boots from
the recovery partition, updates the other partitions (system, vendor, etc.)
Then, the next time the system boots normally, a script updates the recovery
partition (if necessary). This script, the executables it invokes, and the data
files that it uses were previously on the system partition. The resources that
are moving include the following.

* install-recovery.sh
* applypatch
* recovery-resource.dat (if present)
* recovery-from-boot.p (if present)

This makes the applypatch executable a vendor module.

This change supports making dependencies of the applypatch executable available
to applypatch, which is now on vendor.

Since install-recovery.sh is now a vendor service, we add the
applypatch/vendor_flash_recovery.rc file to /vendor/etc/init to start the
service.

Bug: 68319577
Test: Ensure that recovery partition is updated correctly.
Change-Id: I01c0800ee6078aa6c9d716d5f154ad2d63c7af84
This commit is contained in:
Bill Peckham 2019-09-17 17:11:50 -07:00
parent 4782e79324
commit 341644d657
6 changed files with 21 additions and 1 deletions

View file

@ -61,6 +61,14 @@ $(call add-clean-step, rm -rf $(PRODUCT_OUT)/system/priv-app/SystemUpdaterSample
$(call add-clean-step, rm -rf $(PRODUCT_OUT)/system/lib*/libbrotli.so)
$(call add-clean-step, rm -rf $(PRODUCT_OUT)/system/lib*/libbz.so)
# Move recovery resources from /system to /vendor.
$(call add-clean-step, rm -f $(PRODUCT_OUT)/system/bin/applypatch)
$(call add-clean-step, rm -r $(PRODUCT_OUT)/symbols/system/bin/applypatch)
$(call add-clean-step, rm -rf $(PRODUCT_OUT)/obj/PACKAGING/target_files_intermediates/*-target_files-*/SYSTEM/bin/applypatch)
$(call add-clean-step, rm -rf $(PRODUCT_OUT)/obj/PACKAGING/target_files_intermediates/*-target_files-*/SYSTEM/bin/install-recovery.sh)
$(call add-clean-step, rm -rf $(PRODUCT_OUT)/obj/PACKAGING/target_files_intermediates/*-target_files-*/SYSTEM/etc/recovery-resource.dat)
$(call add-clean-step, rm -rf $(PRODUCT_OUT)/obj/PACKAGING/target_files_intermediates/*-target_files-*/SYSTEM/recovery-from-boot.p)
# ************************************************
# NEWER CLEAN STEPS MUST BE AT THE END OF THE LIST
# ************************************************

View file

@ -31,6 +31,7 @@ cc_library_static {
name: "libapplypatch",
host_supported: true,
vendor_available: true,
defaults: [
"applypatch_defaults",
@ -69,6 +70,7 @@ cc_library_static {
cc_library_static {
name: "libapplypatch_modes",
vendor_available: true,
defaults: [
"applypatch_defaults",
@ -92,6 +94,7 @@ cc_library_static {
cc_binary {
name: "applypatch",
vendor: true,
defaults: [
"applypatch_defaults",
@ -120,6 +123,10 @@ cc_binary {
"libz",
"libziparchive",
],
init_rc: [
"vendor_flash_recovery.rc",
],
}
cc_library_host_static {

View file

@ -279,7 +279,7 @@ bool PatchPartition(const Partition& target, const Partition& source, const Valu
}
FileContents source_file;
if (ReadPartitionToBuffer(source, &source_file, true)) {
if (ReadPartitionToBuffer(source, &source_file, backup_source)) {
return GenerateTarget(target, source_file, patch, bonus, backup_source);
}

View file

@ -0,0 +1,3 @@
service vendor_flash_recovery /vendor/bin/install-recovery.sh
class main
oneshot

View file

@ -16,6 +16,7 @@ cc_library_static {
name: "libedify",
host_supported: true,
vendor_available: true,
srcs: [
"expr.cpp",

View file

@ -16,6 +16,7 @@ cc_library_static {
name: "libotautil",
host_supported: true,
vendor_available: true,
recovery_available: true,
defaults: [