Lessen partial scavenge interval on x86-64
authorsbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Apr 2018 06:52:36 +0000 (06:52 +0000)
committersbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Apr 2018 06:52:36 +0000 (06:52 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184577

Rubber-stamped by Filip Pizlo.

I initially made the scavenge interval longer because I had thought the
shorter interval caused a JetStream regression. I was mistaken though.
I was looking at the wrong commit range when analyzing perf data.

This patch shortens the interval, but still keeps x86-64 50% longer than
other architectures. We know that scavenging frequently on Mac is less
important to overall system performance than it is on iOS.

* bmalloc/Scavenger.cpp:
(bmalloc::Scavenger::threadRunLoop):

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

Source/bmalloc/ChangeLog
Source/bmalloc/bmalloc/Scavenger.cpp

index c188636..3451b9b 100644 (file)
@@ -1,5 +1,23 @@
 2018-04-12  Saam Barati  <sbarati@apple.com>
 
+        Lessen partial scavenge interval on x86-64
+        https://bugs.webkit.org/show_bug.cgi?id=184577
+
+        Rubber-stamped by Filip Pizlo.
+
+        I initially made the scavenge interval longer because I had thought the
+        shorter interval caused a JetStream regression. I was mistaken though.
+        I was looking at the wrong commit range when analyzing perf data.
+        
+        This patch shortens the interval, but still keeps x86-64 50% longer than
+        other architectures. We know that scavenging frequently on Mac is less
+        important to overall system performance than it is on iOS.
+
+        * bmalloc/Scavenger.cpp:
+        (bmalloc::Scavenger::threadRunLoop):
+
+2018-04-12  Saam Barati  <sbarati@apple.com>
+
         Raise the partial scavenge interval even more on x86-64
         https://bugs.webkit.org/show_bug.cgi?id=184551
 
index 55845c3..dfad43f 100644 (file)
@@ -408,7 +408,7 @@ void Scavenger::threadRunLoop()
             }
 
 #if BCPU(X86_64)
-            auto partialScavengeInterval = std::chrono::milliseconds(36000);
+            auto partialScavengeInterval = std::chrono::milliseconds(12000);
 #else
             auto partialScavengeInterval = std::chrono::milliseconds(8000);
 #endif