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

Merge "Update Rail Data Collection Frequency" into qt-dev

Browse Source
This commit is contained in:
Blake Kragten
2019-04-23 14:26:48 +00:00
committed by Android (Google) Code Review
parent c2f1cc8b8e f68fa9e364
commit 25b06f04d4
2 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
core/java/com/android/internal/os/BatteryStatsImpl.java
View File

@@ -609,9 +609,7 @@ public class BatteryStatsImpl extends BatteryStats {
int UPDATE_RADIO = 0x04; int UPDATE_RADIO = 0x04;
int UPDATE_BT = 0x08; int UPDATE_BT = 0x08;
int UPDATE_RPM = 0x10; // 16 int UPDATE_RPM = 0x10; // 16
int UPDATE_RAIL = 0x20; // 32 int UPDATE_ALL = UPDATE_CPU | UPDATE_WIFI | UPDATE_RADIO | UPDATE_BT | UPDATE_RPM;
int UPDATE_ALL = UPDATE_CPU | UPDATE_WIFI | UPDATE_RADIO | UPDATE_BT | UPDATE_RPM
| UPDATE_RAIL;
Future<?> scheduleSync(String reason, int flags); Future<?> scheduleSync(String reason, int flags);
Future<?> scheduleCpuSyncDueToRemovedUid(int uid); Future<?> scheduleCpuSyncDueToRemovedUid(int uid);

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

@@ -407,6 +407,7 @@ class BatteryExternalStatsWorker implements BatteryStatsImpl.ExternalStatsSync {
SynchronousResultReceiver wifiReceiver = null; SynchronousResultReceiver wifiReceiver = null;
SynchronousResultReceiver bluetoothReceiver = null; SynchronousResultReceiver bluetoothReceiver = null;
SynchronousResultReceiver modemReceiver = null; SynchronousResultReceiver modemReceiver = null;
boolean railUpdated = false;
if ((updateFlags & BatteryStatsImpl.ExternalStatsSync.UPDATE_WIFI) != 0) { if ((updateFlags & BatteryStatsImpl.ExternalStatsSync.UPDATE_WIFI) != 0) {
// We were asked to fetch WiFi data. // We were asked to fetch WiFi data.
@@ -426,6 +427,10 @@ class BatteryExternalStatsWorker implements BatteryStatsImpl.ExternalStatsSync {
// Oh well. // Oh well.
} }
} }
synchronized (mStats) {
mStats.updateRailStatsLocked();
}
railUpdated = true;
} }
if ((updateFlags & BatteryStatsImpl.ExternalStatsSync.UPDATE_BT) != 0) { if ((updateFlags & BatteryStatsImpl.ExternalStatsSync.UPDATE_BT) != 0) {
@@ -447,6 +452,11 @@ class BatteryExternalStatsWorker implements BatteryStatsImpl.ExternalStatsSync {
modemReceiver = new SynchronousResultReceiver("telephony"); modemReceiver = new SynchronousResultReceiver("telephony");
mTelephony.requestModemActivityInfo(modemReceiver); mTelephony.requestModemActivityInfo(modemReceiver);
} }
if (!railUpdated) {
synchronized (mStats) {
mStats.updateRailStatsLocked();
}
}
} }
final WifiActivityEnergyInfo wifiInfo = awaitControllerInfo(wifiReceiver); final WifiActivityEnergyInfo wifiInfo = awaitControllerInfo(wifiReceiver);
@@ -477,10 +487,6 @@ class BatteryExternalStatsWorker implements BatteryStatsImpl.ExternalStatsSync {
mStats.updateRpmStatsLocked(); mStats.updateRpmStatsLocked();
} }
if ((updateFlags & UPDATE_RAIL) != 0) {
mStats.updateRailStatsLocked();
}
if (bluetoothInfo != null) { if (bluetoothInfo != null) {
if (bluetoothInfo.isValid()) { if (bluetoothInfo.isValid()) {
mStats.updateBluetoothStateLocked(bluetoothInfo); mStats.updateBluetoothStateLocked(bluetoothInfo);