From 0db62f43de504e87f82f10df50f8085388759753 Mon Sep 17 00:00:00 2001 From: Bowgo Tsai Date: Wed, 7 Sep 2022 18:45:36 +0800 Subject: [PATCH] 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 --- fs_mgr/fs_mgr.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/fs_mgr/fs_mgr.cpp b/fs_mgr/fs_mgr.cpp index 1f54f5b98..9fe8e188a 100644 --- a/fs_mgr/fs_mgr.cpp +++ b/fs_mgr/fs_mgr.cpp @@ -903,9 +903,9 @@ 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 - << " rec[" << i << "].fs_type=" << fstab[i].fs_type << " already mounted as " - << fstab[*attempted_idx].fs_type; + 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,9 +933,9 @@ 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 " - << fstab[i].mount_point << " with fs_type=" << fstab[i].fs_type - << " instead of " << fstab[start_idx].fs_type; + 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; } fs_stat &= ~FS_STAT_FULL_MOUNT_FAILED; mount_errno = 0;