diff --git a/services/core/java/com/android/server/power/batterysaver/BatterySaverStateMachine.java b/services/core/java/com/android/server/power/batterysaver/BatterySaverStateMachine.java index 4c88bf44d5a83..20ceed43d27da 100644 --- a/services/core/java/com/android/server/power/batterysaver/BatterySaverStateMachine.java +++ b/services/core/java/com/android/server/power/batterysaver/BatterySaverStateMachine.java @@ -196,6 +196,7 @@ public class BatterySaverStateMachine { h.postDelayed(r, delayMillis); } + @GuardedBy("mLock") void refreshSettingsLocked() { final boolean lowPowerModeEnabled = getGlobalSetting( Settings.Global.LOW_POWER_MODE, 0) != 0; @@ -214,6 +215,7 @@ public class BatterySaverStateMachine { * * Note this will be called before {@link #onBootCompleted} too. */ + @GuardedBy("mLock") @VisibleForTesting void setSettingsLocked(boolean batterySaverEnabled, boolean batterySaverEnabledSticky, int batterySaverTriggerThreshold) { @@ -288,6 +290,7 @@ public class BatterySaverStateMachine { /** * Decide whether to auto-start / stop battery saver. */ + @GuardedBy("mLock") private void doAutoBatterySaverLocked() { if (DEBUG) { Slog.d(TAG, "doAutoBatterySaverLocked: mBootCompleted=" + mBootCompleted @@ -351,6 +354,7 @@ public class BatterySaverStateMachine { * Actually enable / disable battery saver. Write the new state to the global settings * and propagate it to {@link #mBatterySaverController}. */ + @GuardedBy("mLock") private void enableBatterySaverLocked(boolean enable, boolean manual, int intReason, String strReason) { if (DEBUG) { @@ -402,6 +406,7 @@ public class BatterySaverStateMachine { } } + @GuardedBy("mLock") private void updateSnoozingLocked(boolean snoozing, String reason) { if (mBatterySaverSnoozing == snoozing) { return; diff --git a/services/core/java/com/android/server/power/batterysaver/BatterySavingStats.java b/services/core/java/com/android/server/power/batterysaver/BatterySavingStats.java index f53a5dc6ac16a..821320559017c 100644 --- a/services/core/java/com/android/server/power/batterysaver/BatterySavingStats.java +++ b/services/core/java/com/android/server/power/batterysaver/BatterySavingStats.java @@ -484,6 +484,7 @@ public class BatterySavingStats { (BatterySaverState.MASK << BatterySaverState.SHIFT) | (InteractiveState.MASK << InteractiveState.SHIFT); + @GuardedBy("BatterySavingStats.this.mLock") public void transitionStateLocked( int newState, long now, int batteryLevel, int batteryPercent) { final boolean stateChanging = @@ -503,6 +504,7 @@ public class BatterySavingStats { mLastState = newState; } + @GuardedBy("BatterySavingStats.this.mLock") void reportLocked(int state, long deltaTimeMs, int startBatteryLevelUa, int startBatteryLevelPercent, int endBatteryLevelUa, int endBatteryLevelPercent) {