Reduce log noise in KeyguardUpdateMonitorLog

Time tick log and battery update logs result in useful log messages being evicted over a period of time.
Removing these two logs will help us have more debug logs for any issues in the wild.

Bug: 275097662
Test: Everything builds, verified time tick logs don't show up in log buffer.
Change-Id: I1de4f5c200f4fb12a4578ee4edc393549c439068
This commit is contained in:
Chandru S
2023-06-01 09:01:57 -07:00
parent cf2e76555e
commit e964e40467
2 changed files with 24 additions and 5 deletions

View File

@@ -3565,7 +3565,6 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab
*/
private void handleTimeUpdate() {
Assert.isMainThread();
mLogger.d("handleTimeUpdate");
for (int i = 0; i < mCallbacks.size(); i++) {
KeyguardUpdateMonitorCallback cb = mCallbacks.get(i).get();
if (cb != null) {
@@ -3630,9 +3629,9 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab
private void handleBatteryUpdate(BatteryStatus status) {
Assert.isMainThread();
final boolean batteryUpdateInteresting = isBatteryUpdateInteresting(mBatteryStatus, status);
mLogger.logHandleBatteryUpdate(batteryUpdateInteresting);
mBatteryStatus = status;
if (batteryUpdateInteresting) {
mLogger.logHandleBatteryUpdate(mBatteryStatus);
for (int i = 0; i < mCallbacks.size(); i++) {
KeyguardUpdateMonitorCallback cb = mCallbacks.get(i).get();
if (cb != null) {

View File

@@ -30,7 +30,7 @@ import com.android.keyguard.FaceAuthUiEvent
import com.android.keyguard.KeyguardListenModel
import com.android.keyguard.KeyguardUpdateMonitorCallback
import com.android.keyguard.TrustGrantFlags
import com.android.systemui.log.dagger.KeyguardUpdateMonitorLog
import com.android.settingslib.fuelgauge.BatteryStatus
import com.android.systemui.log.LogBuffer
import com.android.systemui.log.LogLevel
import com.android.systemui.log.LogLevel.DEBUG
@@ -38,6 +38,7 @@ import com.android.systemui.log.LogLevel.ERROR
import com.android.systemui.log.LogLevel.INFO
import com.android.systemui.log.LogLevel.VERBOSE
import com.android.systemui.log.LogLevel.WARNING
import com.android.systemui.log.dagger.KeyguardUpdateMonitorLog
import com.google.errorprone.annotations.CompileTimeConstant
import javax.inject.Inject
@@ -683,8 +684,27 @@ constructor(@KeyguardUpdateMonitorLog private val logBuffer: LogBuffer) {
)
}
fun logHandleBatteryUpdate(isInteresting: Boolean) {
logBuffer.log(TAG, DEBUG, { bool1 = isInteresting }, { "handleBatteryUpdate: $bool1" })
fun logHandleBatteryUpdate(batteryStatus: BatteryStatus?) {
logBuffer.log(
TAG,
DEBUG,
{
bool1 = batteryStatus != null
int1 = batteryStatus?.status ?: -1
int2 = batteryStatus?.chargingStatus ?: -1
long1 = (batteryStatus?.level ?: -1).toLong()
long2 = (batteryStatus?.maxChargingWattage ?: -1).toLong()
str1 = "${batteryStatus?.plugged ?: -1}"
},
{
"handleBatteryUpdate: isNotNull: $bool1 " +
"BatteryStatus{status= $int1, " +
"level=$long1, " +
"plugged=$str1, " +
"chargingStatus=$int2, " +
"maxChargingWattage= $long2}"
}
)
}
fun scheduleWatchdog(@CompileTimeConstant watchdogType: String) {