REGRESSION(r216763): JetStream is 1% slower on Mac
authormsaboff@apple.com <msaboff@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 May 2017 00:12:36 +0000 (00:12 +0000)
committermsaboff@apple.com <msaboff@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 May 2017 00:12:36 +0000 (00:12 +0000)
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:

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

Source/bmalloc/ChangeLog
Source/bmalloc/bmalloc/Sizes.h

index a238971..0f294b8 100644 (file)
@@ -1,3 +1,15 @@
+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>
 
         bmalloc: Bump the size of the deallocator log to 512
index c5af29e..b48b575 100644 (file)
@@ -68,7 +68,7 @@ namespace Sizes {
     static const size_t deallocatorLogCapacity = 512;
     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;