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

Commit 19625788 authored by Hugo Benichi's avatar Hugo Benichi Committed by android-build-merger
Browse files

Merge "Wakeup packet events: addressing a few comments" am: 67d16ec1 

am: a35bbade

Change-Id: I33eeb421cff21efdaa409d27fb130ff4754ac6b0
parents 3db86833 a35bbade

core/java/android/net/metrics/WakeupStats.java

+3 −3
Original line number Diff line number Diff line
@@ -35,7 +35,7 @@ public class WakeupStats { 
    public long systemWakeups = 0;

    public long nonApplicationWakeups = 0;

    public long applicationWakeups = 0;

    public long unroutedWakeups = 0;

    public long noUidWakeups = 0;

    public long durationSec = 0;



    public WakeupStats(String iface) {
@@ -58,7 +58,7 @@ public class WakeupStats { 
                systemWakeups++;

                break;

            case NO_UID:

                unroutedWakeups++;

                noUidWakeups++;

                break;

            default:

                if (ev.uid >= Process.FIRST_APPLICATION_UID) {
@@ -80,7 +80,7 @@ public class WakeupStats { 
                .append(", system: ").append(systemWakeups)

                .append(", apps: ").append(applicationWakeups)

                .append(", non-apps: ").append(nonApplicationWakeups)

                .append(", unrouted: ").append(unroutedWakeups)

                .append(", no uid: ").append(noUidWakeups)

                .append(", ").append(durationSec).append("s)")

                .toString();

    }

proto/src/ipconnectivity.proto

+2 −2
Original line number Diff line number Diff line
@@ -501,8 +501,8 @@ message WakeupStats { 
  // between [1001, 9999]. See android.os.Process for possible uids.

  optional int64 non_application_wakeups = 6;



  // The total number of wakeup packets with no associated sockets.

  optional int64 unrouted_wakeups = 7;

  // The total number of wakeup packets with no associated socket or uid.

  optional int64 no_uid_wakeups = 7;

}



// Represents one of the IP connectivity event defined in this file.

services/core/java/com/android/server/connectivity/IpConnectivityEventBuilder.java

+1 −1
Original line number Diff line number Diff line
@@ -126,7 +126,7 @@ final public class IpConnectivityEventBuilder { 
        wakeupStats.systemWakeups = in.systemWakeups;

        wakeupStats.nonApplicationWakeups = in.nonApplicationWakeups;

        wakeupStats.applicationWakeups = in.applicationWakeups;

        wakeupStats.unroutedWakeups = in.unroutedWakeups;

        wakeupStats.noUidWakeups = in.noUidWakeups;

        final IpConnectivityEvent out = buildEvent(0, 0, in.iface);

        out.setWakeupStats(wakeupStats);

        return out;

services/core/java/com/android/server/connectivity/NetdEventListenerService.java

+2 −2
Original line number Diff line number Diff line
@@ -170,11 +170,11 @@ public class NetdEventListenerService extends INetdEventListener.Stub { 
            timestampMs = System.currentTimeMillis();

        }



        addWakupEvent(iface, timestampMs, uid);

        addWakeupEvent(iface, timestampMs, uid);

    }



    @GuardedBy("this")

    private void addWakupEvent(String iface, long timestampMs, int uid) {

    private void addWakeupEvent(String iface, long timestampMs, int uid) {

        int index = wakeupEventIndex(mWakeupEventCursor);

        mWakeupEventCursor++;

        WakeupEvent event = new WakeupEvent();

tests/net/java/com/android/server/connectivity/IpConnectivityEventBuilderTest.java

+2 −2
Original line number Diff line number Diff line
@@ -512,7 +512,7 @@ public class IpConnectivityEventBuilderTest { 
        stats.nonApplicationWakeups = 1;

        stats.rootWakeups = 2;

        stats.systemWakeups = 3;

        stats.unroutedWakeups = 3;

        stats.noUidWakeups = 3;



        IpConnectivityEvent got = IpConnectivityEventBuilder.toProto(stats);

        String want = String.join("\n",
@@ -526,11 +526,11 @@ public class IpConnectivityEventBuilderTest { 
                "  wakeup_stats <",

                "    application_wakeups: 5",

                "    duration_sec: 0",

                "    no_uid_wakeups: 3",

                "    non_application_wakeups: 1",

                "    root_wakeups: 2",

                "    system_wakeups: 3",

                "    total_wakeups: 14",

                "    unrouted_wakeups: 3",

                "  >",

                ">",

                "version: 2\n");