Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Only fillLowPowerStats when needed

Browse Source 
This fixes a boottime regression.
Prior to fix, FinishBooting took ~900ms to complete
After fix, FinishBooting took ~120ms to complete

Fixes: 193074223
Test: atest google/perf/boottime/boottime-test
Change-Id: If85a93f1f92c8af8251c870bf576bb125778b011
This commit is contained in:
Michael Wachenschwanz
2021-07-12 20:16:27 -07:00
parent 224f71520f
commit 44fc5760dc
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
services/core/java/com/android/server/am/BatteryExternalStatsWorker.java
View File

@@ -579,8 +579,10 @@ class BatteryExternalStatsWorker implements BatteryStatsImpl.ExternalStatsSync {
}
}
// Collect the latest low power stats without holding the mStats lock.
mStats.fillLowPowerStats();
if ((updateFlags & BatteryStatsImpl.ExternalStatsSync.UPDATE_RPM) != 0) {
// Collect the latest low power stats without holding the mStats lock.
mStats.fillLowPowerStats();
}
final WifiActivityEnergyInfo wifiInfo = awaitControllerInfo(wifiReceiver);
final BluetoothActivityEnergyInfo bluetoothInfo = awaitControllerInfo(bluetoothReceiver);