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

Commit 61c07a09 authored by Kelvin Zhang's avatar Kelvin Zhang Committed by Gerrit Code Review
Browse files

Revert^2 "Set block device as RO/RW before mount" 

Original CL reverted due to missing permission on vold.
The missing permission is added in aosp/3202313 , hence re-submit.

This reverts commit 1f00f794.


Reason for revert: b/349507086

Change-Id: Ia1086f3c8c65c476bdc119fe59ed4fd3a21f8591
parent 1f00f794

fs_mgr/fs_mgr.cpp

+3 −3
Original line number Diff line number Diff line
@@ -826,6 +826,9 @@ static int __mount(const std::string& source, const std::string& target, const F 
    if (read_only) {

        mountflags |= MS_RDONLY;

    }

    if (!fs_mgr_set_blk_ro(source, read_only)) {

        PLOG(ERROR) << "Failed to set " << source << " as " << (read_only ? "RO" : "RW");

    }

    int ret = 0;

    int save_errno = 0;

    int gc_allowance = 0;
@@ -880,9 +883,6 @@ static int __mount(const std::string& source, const std::string& target, const F 
    }

    PINFO << __FUNCTION__ << "(source=" << source << source_missing << ",target=" << target

          << target_missing << ",type=" << entry.fs_type << ")=" << ret;

    if ((ret == 0) && (mountflags & MS_RDONLY) != 0) {

        fs_mgr_set_blk_ro(source);

    }

    if (ret == 0) {

        android::base::SetProperty("ro.boottime.init.mount." + Basename(target),

                                   std::to_string(t.duration().count()));