summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Olivier Gaillard <gaillard@google.com> 2023-02-24 12:39:37 +0000
committer Olivier Gaillard <gaillard@google.com> 2023-02-28 17:36:39 +0000
commit3fab5c257ae8f74bf1b66a36416cb3242079346f (patch)
tree07c92256db4e8d8b34ac3eb52d5a0f195c0c105d
parent829a3b61440b60ea61cc9606118bc4b72766cc1d (diff)
Further limit the number of extra pids dumped as this is expensive.
In addition to being expensive, it can delay the thread dump of consecutive ANRs which will make the thread dumps less accurate. The top 2 processes should be enough to debug the ANR. Change-Id: I1a10c4e8501d705dcc1b36190db53d334f773b23 Bug: 254634348 Test: unit tests
-rw-r--r--services/core/java/com/android/server/am/ActivityManagerService.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/am/ActivityManagerService.java b/services/core/java/com/android/server/am/ActivityManagerService.java
index 7df761371b8b..fc11cd0f24d6 100644
--- a/services/core/java/com/android/server/am/ActivityManagerService.java
+++ b/services/core/java/com/android/server/am/ActivityManagerService.java
@@ -3537,7 +3537,7 @@ public class ActivityManagerService extends IActivityManager.Stub
// We'll take the stack crawls of just the top apps using CPU.
final int workingStatsNumber = processCpuTracker.countWorkingStats();
- for (int i = 0; i < workingStatsNumber && extraPids.size() < 5; i++) {
+ for (int i = 0; i < workingStatsNumber && extraPids.size() < 2; i++) {
ProcessCpuTracker.Stats stats = processCpuTracker.getWorkingStats(i);
if (lastPids.indexOfKey(stats.pid) >= 0) {
if (DEBUG_ANR) {