Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 54abc85d authored by Akilesh Kailash's avatar Akilesh Kailash Committed by Automerger Merge Worker
Browse files

Merge "init: umount dynamic partitions during shutdown" into main am: f17936dc 

Original change: https://android-review.googlesource.com/c/platform/system/core/+/3495262



Change-Id: I50aa69f23f4a479b2d7c28c491ffaee320071215
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 52ce54d8 f17936dc

init/reboot.cpp

+15 −2
Original line number Diff line number Diff line
@@ -394,7 +394,8 @@ void RebootMonitorThread(unsigned int cmd, const std::string& reboot_target, 
    }

}



static void UmountDynamicPartitions(const std::vector<std::string>& dynamic_partitions) {

static bool UmountDynamicPartitions(const std::vector<std::string>& dynamic_partitions) {

    bool ret = true;

    for (auto device : dynamic_partitions) {

        // Cannot unmount /system

        if (device == "/system") {
@@ -405,8 +406,10 @@ static void UmountDynamicPartitions(const std::vector<std::string>& dynamic_part 
            LOG(INFO) << "Umounted success: " << device;

        } else {

            PLOG(WARNING) << "Cannot umount: " << device;

            ret = false;

        }

    }

    return ret;

}



/* Try umounting all emulated file systems R/W block device cfile systems.
@@ -454,7 +457,17 @@ static UmountStat TryUmountAndFsck(unsigned int cmd, bool run_fsck, 
        // still not doing fsck when all processes are killed.

        //

        if (ota_update_in_progress) {

            bool umount_dynamic_partitions = UmountDynamicPartitions(dynamic_partitions);

            LOG(INFO) << "Sending SIGTERM to all process";

            // Send SIGTERM to all processes except init

            WriteStringToFile("e", PROC_SYSRQ);

            // Wait for processes to terminate

            std::this_thread::sleep_for(1s);

            // Try one more attempt to umount other partitions which failed

            // earlier

            if (!umount_dynamic_partitions) {

                UmountDynamicPartitions(dynamic_partitions);

            }

            return stat;

        }

        KillAllProcesses();