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

Commit 0db62f43 authored by Bowgo Tsai's avatar Bowgo Tsai
Browse files

Change log level in mount_with_alternatives() 

mount_with_alternatives() supports mounting any of the
consecutive fstab entries. Some log messages shouldn't
be treated as error so changing log level to INFO instead.

Bug: 245468764
Test: TreeHugger
Change-Id: I94a18d4cf91ee5bb58cf5ba5f853a0e6599071d1
parent 98e474ab

fs_mgr/fs_mgr.cpp

+6 −6
Original line number Diff line number Diff line
@@ -903,7 +903,7 @@ static bool mount_with_alternatives(Fstab& fstab, int start_idx, int* end_idx, 
        // Deal with alternate entries for the same point which are required to be all following

        // each other.

        if (mounted) {

            LERROR << __FUNCTION__ << "(): skipping fstab dup mountpoint=" << fstab[i].mount_point

            LINFO << __FUNCTION__ << "(): skipping fstab dup mountpoint=" << fstab[i].mount_point

                  << " rec[" << i << "].fs_type=" << fstab[i].fs_type << " already mounted as "

                  << fstab[*attempted_idx].fs_type;

            continue;
@@ -933,7 +933,7 @@ static bool mount_with_alternatives(Fstab& fstab, int start_idx, int* end_idx, 
                *attempted_idx = i;

                mounted = true;

                if (i != start_idx) {

                    LERROR << __FUNCTION__ << "(): Mounted " << fstab[i].blk_device << " on "

                    LINFO << __FUNCTION__ << "(): Mounted " << fstab[i].blk_device << " on "

                          << fstab[i].mount_point << " with fs_type=" << fstab[i].fs_type

                          << " instead of " << fstab[start_idx].fs_type;

                }