Changeset 128391 in webkit

Timestamp:

Sep 12, 2012 5:49:47 PM (12 years ago)

Author:

Stephanie Lewis

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):

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/webkitpy/layout_tests/port/mac.py (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2012-09-12  Stephanie Lewis  <slewis@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  <beidson@apple.com>
 

trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py

@@ -116 +116 @@
         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()