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

Commit 88be2985 authored by Florian Mayer's avatar Florian Mayer Committed by Gerrit Code Review
Browse files

Merge "Do not disable tracing in zygote."

parents 8e08b87e 5845bdf6

core/java/android/os/Trace.java

+1 −5
Original line number Diff line number Diff line
@@ -175,11 +175,7 @@ public final class Trace { 
    }



    /**

     * Set whether tracing is enabled in this process.  Tracing is disabled shortly after Zygote

     * initializes and re-enabled after processes fork from Zygote.  This is done because Zygote

     * has no way to be notified about changes to the tracing tags, and if Zygote ever reads and

     * caches the tracing tags, forked processes will inherit those stale tags.

     *

     * Set whether tracing is enabled in this process.

     * @hide

     */

    public static void setTracingEnabled(boolean enabled, int debugFlags) {

core/java/com/android/internal/os/Zygote.java

+0 −10
Original line number Diff line number Diff line
@@ -261,10 +261,8 @@ public final class Zygote { 
        int pid = nativeForkAndSpecialize(

                uid, gid, gids, runtimeFlags, rlimits, mountExternal, seInfo, niceName, fdsToClose,

                fdsToIgnore, startChildZygote, instructionSet, appDataDir);

        // Enable tracing as soon as possible for the child process.

        if (pid == 0) {

            Zygote.disableExecuteOnly(targetSdkVersion);

            Trace.setTracingEnabled(true, runtimeFlags);



            // Note that this event ends at the end of handleChildProc,

            Trace.traceBegin(Trace.TRACE_TAG_ACTIVITY_MANAGER, "PostFork");
@@ -309,9 +307,6 @@ public final class Zygote { 
        nativeSpecializeAppProcess(uid, gid, gids, runtimeFlags, rlimits, mountExternal, seInfo,

                niceName, startChildZygote, instructionSet, appDataDir);



        // Enable tracing as soon as possible for the child process.

        Trace.setTracingEnabled(true, runtimeFlags);



        // Note that this event ends at the end of handleChildProc.

        Trace.traceBegin(Trace.TRACE_TAG_ACTIVITY_MANAGER, "PostFork");
@@ -367,11 +362,6 @@ public final class Zygote { 
                uid, gid, gids, runtimeFlags, rlimits,

                permittedCapabilities, effectiveCapabilities);



        // Enable tracing as soon as we enter the system_server.

        if (pid == 0) {

            Trace.setTracingEnabled(true, runtimeFlags);

        }



        // Set the Java Language thread priority to the default value for new apps.

        Thread.currentThread().setPriority(Thread.NORM_PRIORITY);

core/java/com/android/internal/os/ZygoteInit.java

+0 −4
Original line number Diff line number Diff line
@@ -908,10 +908,6 @@ public class ZygoteInit { 
            bootTimingsTraceLog.traceEnd(); // PostZygoteInitGC



            bootTimingsTraceLog.traceEnd(); // ZygoteInit

            // Disable tracing so that forked processes do not inherit stale tracing tags from

            // Zygote.

            Trace.setTracingEnabled(false, 0);





            Zygote.initNativeState(isPrimaryZygote);