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

Commit e7537eb3 authored by Vova Sharaienko's avatar Vova Sharaienko
Browse files

Atom ProcessStateChanged marked as deprecated 

Removed excessive logging of this atom, which is 10k+ times per day

Bug: 211823706
Test: build & boot
Change-Id: I96c62e58b828f9bb67ed3d34d1a2afcd0ef7bfbb
parent 65e05f9c

services/core/java/com/android/server/am/ProcessProfileRecord.java

+0 −13
Original line number Diff line number Diff line
@@ -17,7 +17,6 @@ 
package com.android.server.am;



import static android.app.ActivityManager.PROCESS_STATE_NONEXISTENT;

import static android.app.ActivityManager.processStateAmToProto;



import android.annotation.IntDef;

import android.app.IApplicationThread;
@@ -318,12 +317,6 @@ final class ProcessProfileRecord { 
                            origBase.setState(ProcessStats.STATE_NOTHING,

                                    tracker.getMemFactorLocked(), SystemClock.uptimeMillis(),

                                    pkgList.getPackageListLocked());

                            pkgList.forEachPackage((pkgName, holder) ->

                                    FrameworkStatsLog.write(FrameworkStatsLog.PROCESS_STATE_CHANGED,

                                        mApp.uid, mApp.processName, pkgName,

                                        processStateAmToProto(ProcessStats.STATE_NOTHING),

                                        holder.appVersion)

                            );

                        }

                        origBase.makeInactive();

                    }
@@ -362,12 +355,6 @@ final class ProcessProfileRecord { 
                        origBase.setState(ProcessStats.STATE_NOTHING,

                                tracker.getMemFactorLocked(), SystemClock.uptimeMillis(),

                                pkgList.getPackageListLocked());

                        pkgList.forEachPackage((pkgName, holder) ->

                                FrameworkStatsLog.write(FrameworkStatsLog.PROCESS_STATE_CHANGED,

                                    mApp.uid, mApp.processName, pkgName,

                                    processStateAmToProto(ProcessStats.STATE_NOTHING),

                                    holder.appVersion)

                        );

                    }

                    origBase.makeInactive();

                    setBaseProcessTracker(null);

services/core/java/com/android/server/am/ProcessRecord.java

+0 −7
Original line number Diff line number Diff line
@@ -54,7 +54,6 @@ import com.android.internal.annotations.VisibleForTesting; 
import com.android.internal.app.procstats.ProcessState;

import com.android.internal.app.procstats.ProcessStats;

import com.android.internal.os.Zygote;

import com.android.internal.util.FrameworkStatsLog;

import com.android.server.wm.WindowProcessController;

import com.android.server.wm.WindowProcessListener;
@@ -1212,12 +1211,6 @@ class ProcessRecord implements WindowProcessListener { 
                    long now = SystemClock.uptimeMillis();

                    baseProcessTracker.setState(ProcessStats.STATE_NOTHING,

                            tracker.getMemFactorLocked(), now, mPkgList.getPackageListLocked());

                    mPkgList.forEachPackage((pkgName, holder) ->

                            FrameworkStatsLog.write(FrameworkStatsLog.PROCESS_STATE_CHANGED,

                                uid, processName, pkgName,

                                ActivityManager.processStateAmToProto(ProcessStats.STATE_NOTHING),

                                holder.appVersion)

                    );

                    if (numOfPkgs != 1) {

                        mPkgList.forEachPackageProcessStats(holder -> {

                            if (holder.state != null && holder.state != baseProcessTracker) {

services/core/java/com/android/server/am/ProcessStateRecord.java

+0 −13
Original line number Diff line number Diff line
@@ -36,7 +36,6 @@ import android.util.TimeUtils; 


import com.android.internal.annotations.CompositeRWLock;

import com.android.internal.annotations.GuardedBy;

import com.android.internal.util.FrameworkStatsLog;

import com.android.server.am.PlatformCompatCache.CachedCompatChangeId;



import java.io.PrintWriter;
@@ -599,12 +598,6 @@ final class ProcessStateRecord { 
    @GuardedBy({"mService", "mProcLock"})

    void setReportedProcState(int repProcState) {

        mRepProcState = repProcState;

        mApp.getPkgList().forEachPackage((pkgName, holder) ->

                FrameworkStatsLog.write(FrameworkStatsLog.PROCESS_STATE_CHANGED,

                    mApp.uid, mApp.processName, pkgName,

                    ActivityManager.processStateAmToProto(mRepProcState),

                    holder.appVersion)

        );

        mApp.getWindowProcessController().setReportedProcState(repProcState);

    }
@@ -620,12 +613,6 @@ final class ProcessStateRecord { 
                mRepProcState = newState;

                setCurProcState(newState);

                setCurRawProcState(newState);

                mApp.getPkgList().forEachPackage((pkgName, holder) ->

                        FrameworkStatsLog.write(FrameworkStatsLog.PROCESS_STATE_CHANGED,

                            mApp.uid, mApp.processName, pkgName,

                            ActivityManager.processStateAmToProto(mRepProcState),

                            holder.appVersion)

                );

            }

        }

    }