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

Commit 13ca4537 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge changes from topic "b150519505_disp_up_reason_push_atom" 

* changes:
  Suez: Implement display wake reason stats
  Suez: change display wake reason to push atom
parents 71da62dd 0850023c

cmds/statsd/src/atoms.proto

+12 −6
Original line number Diff line number Diff line
@@ -450,13 +450,14 @@ message Atom { 
        TvCasSessionOpenStatus tv_cas_session_open_status =

            280 [(module) = "framework"];

        AssistantInvocationReported assistant_invocation_reported = 281 [(module) = "framework"];

        DisplayWakeReported display_wake_reported = 282 [(module) = "framework"];



        // StatsdStats tracks platform atoms with ids upto 500.

        // Update StatsdStats::kMaxPushedAtomId when atom ids here approach that value.

    }



    // Pulled events will start at field 10000.

    // Next: 10084

    // Next: 10081

    oneof pulled {

        WifiBytesTransfer wifi_bytes_transfer = 10000 [(module) = "framework"];

        WifiBytesTransferByFgBg wifi_bytes_transfer_by_fg_bg = 10001 [(module) = "framework"];
@@ -547,7 +548,7 @@ message Atom { 
        SimSlotState sim_slot_state = 10078 [(module) = "telephony"];

        SupportedRadioAccessFamily supported_radio_access_family = 10079 [(module) = "telephony"];

        SettingSnapshot setting_snapshot = 10080 [(module) = "framework"];

        DisplayWakeReason display_wake_reason = 10081 [(module) = "framework"];

        //10081 free for use

        DataUsageBytesTransfer data_usage_bytes_transfer = 10082 [(module) = "framework"];

        BytesTransferByTagAndMetered bytes_transfer_by_tag_and_metered =

                10083 [(module) = "framework"];
@@ -9827,15 +9828,20 @@ message AccessibilityServiceReported { 
    optional android.stats.accessibility.ServiceStatus service_status = 2;

}



message DisplayWakeReason {

/**

 * Logs when display wake up.

 *

 * Logged from:

 *   services/core/java/com/android/server/power/Notifier.java

 */



message DisplayWakeReported {

    // Wake_up_reason code

    // If LOWORD(wake_up_reason) = 0

    //     reference to HIWORD(wake_up_reason) PowerManager.WAKE_REASON_XXX

    //     else reference wake_up_reason to

    //     frameworks/base/services/core/java/com/android/server/power/Notifier.java#DispWakeupReason

    //     services/core/java/com/android/server/power/Notifier.java#onWakeUp

    optional int32 wake_up_reason = 1;

    // Count of wake up by reason

    optional int32 wake_times = 2;

}



/**

services/core/java/com/android/server/power/Notifier.java

+1 −0
Original line number Diff line number Diff line
@@ -616,6 +616,7 @@ public class Notifier { 
        } catch (RemoteException ex) {

            // Ignore

        }

        FrameworkStatsLog.write(FrameworkStatsLog.DISPLAY_WAKE_REPORTED, reason);

    }



    /**

services/core/java/com/android/server/stats/pull/StatsPullAtomService.java

+0 −18
Original line number Diff line number Diff line
@@ -468,8 +468,6 @@ public class StatsPullAtomService extends SystemService { 
                        return pullAttributedAppOps(atomTag, data);

                    case FrameworkStatsLog.SETTING_SNAPSHOT:

                        return pullSettingsStats(atomTag, data);

                    case FrameworkStatsLog.DISPLAY_WAKE_REASON:

                        return pullDisplayWakeStats(atomTag, data);

                    default:

                        throw new UnsupportedOperationException("Unknown tagId=" + atomTag);

                }
@@ -656,7 +654,6 @@ public class StatsPullAtomService extends SystemService { 
        registerBatteryVoltage();

        registerBatteryCycleCount();

        registerSettingsStats();

        registerDisplayWakeStats();

    }



    private void initAndRegisterNetworkStatsPullers() {
@@ -3678,21 +3675,6 @@ public class StatsPullAtomService extends SystemService { 
        return StatsManager.PULL_SUCCESS;

    }



    private void registerDisplayWakeStats() {

        int tagId = FrameworkStatsLog.DISPLAY_WAKE_REASON;

        mStatsManager.setPullAtomCallback(

                tagId,

                null, // use default PullAtomMetadata values

                DIRECT_EXECUTOR,

                mStatsCallbackImpl

        );

    }



    int pullDisplayWakeStats(int atomTag, List<StatsEvent> pulledData) {

        //TODO: Denny, implement read/write DisplayWakeStats, b/154172964

        return 0;

    }



    // Thermal event received from vendor thermal management subsystem

    private static final class ThermalEventListener extends IThermalEventListener.Stub {

        @Override