Don't throttle GuardMalloc on 8-core Macs
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Apr 2016 06:46:49 +0000 (06:46 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Apr 2016 06:46:49 +0000 (06:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=157108

Reviewed by Alex Christensen.

This was making them slower unnecessarily.

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

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@200175 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/port/mac.py

index a25035e..b3ad90a 100644 (file)
@@ -1,3 +1,14 @@
+2016-04-27  Alexey Proskuryakov  <ap@apple.com>
+
+        Don't throttle GuardMalloc on 8-core Macs
+        https://bugs.webkit.org/show_bug.cgi?id=157108
+
+        Reviewed by Alex Christensen.
+
+        This was making them slower unnecessarily.
+
+        * Scripts/webkitpy/port/mac.py: (MacPort.default_child_processes):
+
 2016-04-26  Konstantin Tokarev  <annulen@yandex.ru>
 
         Unreviewed: add myself to the committers list.
index 8056b1b..42e1117 100644 (file)
@@ -155,12 +155,12 @@ class MacPort(ApplePort):
         should_throttle_for_wk2 = self.get_option('webkit_test_runner') and default_count > 4
         # We also want to throttle for leaks bots.
         if should_throttle_for_wk2 or self.get_option('leaks'):
-            if self.get_option('guard_malloc'):
-                # Some 12 core Macs get a lot of tests time out when running 18 WebKitTestRunner processes (it's not clear what this depends on).
-                # Running 12 processes resolves this, and doesn't seem to introduce a measurable performance degradation on other machines.
-                default_count = int(.5 * default_count)
-            else:
-                default_count = int(.75 * default_count)
+            default_count = int(.75 * default_count)
+
+        if should_throttle_for_wk2 and self.get_option('guard_malloc'):
+            # Some 12 core Macs get a lot of tests time out when running 18 WebKitTestRunner processes (it's not clear what this depends on).
+            # <rdar://problem/25750302>
+            default_count = min(default_count, 12)
 
         # Make sure we have enough ram to support that many instances:
         total_memory = self.host.platform.total_bytes_memory()