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

Commit c0cb4bbd authored by Mohamad Mahmoud's avatar Mohamad Mahmoud
Browse files

Move trace pruning after stack dumping 

Call maybePruneOldTraces after calling dumpStackTraces as it introduces
unnecessary latency in some cases

Test: Tested on a device
Bug: 237671718
Change-Id: I54395c249e563d6ed445c5f1c974250ac26bafce
parent 90e81227

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

+3 −3
Original line number Diff line number Diff line
@@ -3492,9 +3492,6 @@ public class ActivityManagerService extends IActivityManager.Stub 
            }















            final File tracesDir = new File(ANR_TRACE_DIR);













            // Each set of ANR traces is written to a separate file and dumpstate will process













            // all such files and add them to a captured bug report if they're recent enough.













            maybePruneOldTraces(tracesDir);





























            // NOTE: We should consider creating the file in native code atomically once we've















            // gotten rid of the old scheme of dumping and lot of the code that deals with paths










@@ -3527,6 +3524,9 @@ public class ActivityManagerService extends IActivityManager.Stub













            if (firstPidEndOffset != null) {















                firstPidEndOffset.set(firstPidEndPos);















            }













            // Each set of ANR traces is written to a separate file and dumpstate will process













            // all such files and add them to a captured bug report if they're recent enough.













            maybePruneOldTraces(tracesDir);





























            return tracesFile;















        } finally {