Changeset 216960 in webkit

Timestamp:

May 16, 2017 5:12:36 PM (7 years ago)

Author:

msaboff@apple.com

Message:

REGRESSION(r216763): JetStream is 1% slower on Mac
​https://bugs.webkit.org/show_bug.cgi?id=172124

Reviewed by Filip Pizlo.

It appears that changing maxScavengeSleepDuration from 512 to 250ms in r216763 is
responsible for the regression.

• bmalloc/Sizes.h:

Location:

trunk/Source/bmalloc

Files:

• ChangeLog (modified) (1 diff)
• bmalloc/Sizes.h (modified) (1 diff)

trunk/Source/bmalloc/ChangeLog

@@ +1 @@
+2017-05-16  Michael Saboff  <msaboff@apple.com>
+
+        REGRESSION(r216763): JetStream is 1% slower on Mac
+        https://bugs.webkit.org/show_bug.cgi?id=172124
+
+        Reviewed by Filip Pizlo.
+
+        It appears that changing maxScavengeSleepDuration from 512 to 250ms in r216763 is
+        responsible for the regression.
+
+        * bmalloc/Sizes.h:
+
 2017-05-15  Geoffrey Garen  <ggaren@apple.com>
 

trunk/Source/bmalloc/bmalloc/Sizes.h

@@ -69 +69 @@
     static const size_t bumpRangeCacheCapacity = 3;
     
-    static const std::chrono::milliseconds maxScavengeSleepDuration = std::chrono::milliseconds(250);
+    static const std::chrono::milliseconds maxScavengeSleepDuration = std::chrono::milliseconds(512);
 
     static const size_t maskSizeClassCount = maskSizeClassMax / alignment;