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 296724 6cabb1a43af6
parent 296723 12762fdf5ab6
child 296725 ffc3382d3829
push id5308
push uservdjeric@mozilla.com
push date2015-11-15 23:58 +0000
treeherdermozilla-beta@aaa5100e2085 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgfritzsche, lizzard
bugs1219751
milestone43.0
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)"));
 }