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

Merge "CPU time min threshold for CpuTimePerUidFreq"

Browse Source
This commit is contained in:
Rafal Slawik
2020-11-02 09:33:05 +00:00
committed by Android (Google) Code Review
parent dbf6f13897 a79cf88cb7
commit f069be7b3d
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
services/core/java/com/android/server/stats/pull/StatsPullAtomService.java
View File

@@ -247,6 +247,13 @@ public class StatsPullAtomService extends SystemService {
// 20% as a conservative estimate.
private static final int MAX_PROCSTATS_RAW_SHARD_SIZE = (int) (MAX_PROCSTATS_SHARD_SIZE * 1.20);
/**
* Threshold to filter out small CPU times at frequency per UID. Those small values appear
* because of more precise accounting in a BPF program. Discarding them reduces the data by at
* least 20% with negligible error.
*/
private static final int MIN_CPU_TIME_PER_UID_FREQ = 10;
private final Object mThermalLock = new Object();
@GuardedBy("mThermalLock")
private IThermalService mThermalService;
@@ -1509,7 +1516,7 @@ public class StatsPullAtomService extends SystemService {
int pullCpuTimePerUidFreqLocked(int atomTag, List<StatsEvent> pulledData) {
mCpuUidFreqTimeReader.readAbsolute((uid, cpuFreqTimeMs) -> {
for (int freqIndex = 0; freqIndex < cpuFreqTimeMs.length; ++freqIndex) {
if (cpuFreqTimeMs[freqIndex] != 0) {
if (cpuFreqTimeMs[freqIndex] >= MIN_CPU_TIME_PER_UID_FREQ) {
pulledData.add(FrameworkStatsLog.buildStatsEvent(
atomTag, uid, freqIndex, cpuFreqTimeMs[freqIndex]));
}