Bug 1219751 - Change the the depth limit of the thread hangs stack to use the 99th percentile. r=gfritzsche a=lizzard
authorAlessio Placitelli <alessio.placitelli@gmail.com>
Thu, 29 Oct 2015 08:05:00 +0100
changeset 305494 8efe7a54bafea994bfaa1fce37e9bf84283d160f
parent 305493 4b6161cc379b78e4fef8195c3f62a1603acf7b59
child 305495 11725eb448923301ce442f83817c9fd67097a561
push id1001
push userraliiev@mozilla.com
push dateMon, 18 Jan 2016 19:06:03 +0000
treeherdermozilla-release@8b89261f3ac4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgfritzsche, lizzard
bugs1219751
milestone44.0a2
Bug 1219751 - Change the the depth limit of the thread hangs stack to use the 99th percentile. r=gfritzsche a=lizzard
xpcom/threads/BackgroundHangMonitor.cpp
--- a/xpcom/threads/BackgroundHangMonitor.cpp
+++ b/xpcom/threads/BackgroundHangMonitor.cpp
@@ -27,18 +27,18 @@
 #include "nsXULAppAPI.h"
 
 #include <algorithm>
 
 // Activate BHR only for one every BHR_BETA_MOD users.
 #define BHR_BETA_MOD 100;
 
 // Maximum depth of the call stack in the reported thread hangs. This value represents
-// the 95th pecentile of the thread hangs stack depths reported by Telemetry.
-static const size_t kMaxThreadHangStackDepth = 11;
+// the 99.9th percentile of the thread hangs stack depths reported by Telemetry.
+static const size_t kMaxThreadHangStackDepth = 30;
 
 // An utility comparator function used by std::unique to collapse "(* script)" entries in
 // a vector representing a call stack.
 bool StackScriptEntriesCollapser(const char* aStackEntry, const char *aAnotherStackEntry)
 {
   return !strcmp(aStackEntry, aAnotherStackEntry) &&
          (!strcmp(aStackEntry, "(chrome script)") || !strcmp(aStackEntry, "(content script)"));
 }