Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f99c289b authored by Dmitri Plotnikov's avatar Dmitri Plotnikov Committed by Automerger Merge Worker
Browse files

Merge "Address race condition in procstate time tracking" into sc-dev am: b1f548e3 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15177890

Change-Id: I322ee8c8ab5b64607c2815ae28122b9667fbc67c
parents 9471396c b1f548e3

core/java/android/os/BatteryStats.java

+3 −4
Original line number Diff line number Diff line
@@ -16,7 +16,6 @@ 


package android.os;



import static android.app.ActivityManager.PROCESS_STATE_BOUND_TOP;

import static android.os.BatteryStatsManager.NUM_WIFI_STATES;

import static android.os.BatteryStatsManager.NUM_WIFI_SUPPL_STATES;
@@ -903,9 +902,9 @@ public abstract class BatteryStats implements Parcelable { 
         * is not attributed to any non-critical process states.

         */

        public static final int[] CRITICAL_PROC_STATES = {

                PROCESS_STATE_TOP,

                PROCESS_STATE_BOUND_TOP, PROCESS_STATE_FOREGROUND_SERVICE,

                PROCESS_STATE_FOREGROUND

                Uid.PROCESS_STATE_TOP,

                Uid.PROCESS_STATE_FOREGROUND_SERVICE,

                Uid.PROCESS_STATE_FOREGROUND

        };



        public abstract long getProcessStateTime(int state, long elapsedRealtimeUs, int which);

core/tests/coretests/src/com/android/internal/os/BstatsCpuTimesValidationTest.java

+9 −3
Original line number Diff line number Diff line
@@ -584,7 +584,7 @@ public class BstatsCpuTimesValidationTest { 
                actualCpuTimeMs += cpuTimesMs[i];

            }

            assertApproximateValue("Incorrect total cpu time, " + msgCpuTimes,

                    2 * WORK_DURATION_MS, actualCpuTimeMs);

                    WORK_DURATION_MS, actualCpuTimeMs);



            batteryOffScreenOn();

        } finally {
@@ -656,8 +656,14 @@ public class BstatsCpuTimesValidationTest { 
        }

    }



    private void assertApproximateValue(String errorPrefix, long expectedValue, long actualValue) {

        assertValueRange(errorPrefix, actualValue, expectedValue * 0.5, expectedValue * 1.5);

    private void assertApproximateValue(String errorPrefix, long expectedValueMs,

            long actualValueMs) {

        // Allow the actual value to be 1 second smaller than the expected.

        // Also allow it to be up to 5 seconds larger, to accommodate the arbitrary

        // latency introduced by BatteryExternalStatsWorker.scheduleReadProcStateCpuTimes

        assertValueRange(errorPrefix, actualValueMs,

                expectedValueMs - 1000,

                expectedValueMs + 5000);

    }



    private void assertValueRange(String errorPrefix,