Title: [128391] trunk/Tools
Revision
128391
Author
sle...@apple.com
Date
2012-09-12 17:49:47 -0700 (Wed, 12 Sep 2012)

Log Message

Reduce parallism on the wk2 testers.
https://bugs.webkit.org/show_bug.cgi?id=95906

Reviewed by Dirk Pranke.

The wk2 testers on Mountain Lion are getting stuck and timing out.  The problem appears to be due to
resource contention.  Reducing the number of processes alleviates the issue.

Starting by reducing 25%.

* Scripts/webkitpy/layout_tests/port/mac.py:
(MacPort.default_child_processes):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (128390 => 128391)


--- trunk/Tools/ChangeLog	2012-09-13 00:43:18 UTC (rev 128390)
+++ trunk/Tools/ChangeLog	2012-09-13 00:49:47 UTC (rev 128391)
@@ -1,3 +1,18 @@
+2012-09-12  Stephanie Lewis  <sle...@apple.com>
+
+        Reduce parallism on the wk2 testers.
+        https://bugs.webkit.org/show_bug.cgi?id=95906
+
+        Reviewed by Dirk Pranke.
+
+        The wk2 testers on Mountain Lion are getting stuck and timing out.  The problem appears to be due to
+        resource contention.  Reducing the number of processes alleviates the issue.  
+
+        Starting by reducing 25%.
+
+        * Scripts/webkitpy/layout_tests/port/mac.py:
+        (MacPort.default_child_processes):
+
 2012-09-12  Brady Eidson  <beid...@apple.com>
 
         Assert in NetscapePlugin::destroy() with async plugin init

Modified: trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py (128390 => 128391)


--- trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py	2012-09-13 00:43:18 UTC (rev 128390)
+++ trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py	2012-09-13 00:49:47 UTC (rev 128391)
@@ -115,6 +115,12 @@
 
         default_count = super(MacPort, self).default_child_processes()
 
+        # FIXME: https://bugs.webkit.org/show_bug.cgi?id=95906  With too many WebProcess WK2 tests get stuck in resource contention.
+        # To alleviate the issue reduce the number of running processes
+        # Anecdotal evidence suggests that a 4 core/8 core logical machine may run into this, but that a 2 core/4 core logical machine does not.
+        if self.get_option('webkit_test_runner') and default_count > 4:
+            default_count = int(.75 * default_count)
+
         # Make sure we have enough ram to support that many instances:
         total_memory = self.host.platform.total_bytes_memory()
         bytes_per_drt = 256 * 1024 * 1024  # Assume each DRT needs 256MB to run.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to