Changeset 277326 in webkit

Timestamp:

May 11, 2021 8:52:17 AM (3 years ago)

Author:

fpizlo@apple.com

Message:

Tune number of threads for AS
​https://bugs.webkit.org/show_bug.cgi?id=225635

Reviewed by Mark Lam.

Using 4 GC markers (which really means 3 parallel GC worker threads -- the mutator thread is
the 4th), 2 DFG threads, and 2 FTL threads seems to be more optimal than going off ncpu.

~1% JetStream2 speed-up, ~1% Speedometer2 speed-up.

• runtime/Options.cpp:

(JSC::overrideDefaults):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• runtime/Options.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2021-05-10  Filip Pizlo  <fpizlo@apple.com>
+
+        Tune number of threads for AS
+        https://bugs.webkit.org/show_bug.cgi?id=225635
+
+        Reviewed by Mark Lam.
+
+        Using 4 GC markers (which really means 3 parallel GC worker threads -- the mutator thread is
+        the 4th), 2 DFG threads, and 2 FTL threads seems to be more optimal than going off ncpu.
+
+        ~1% JetStream2 speed-up, ~1% Speedometer2 speed-up.
+
+        * runtime/Options.cpp:
+        (JSC::overrideDefaults):
+
 2021-05-10  Mark Lam  <mark.lam@apple.com>
 

trunk/Source/JavaScriptCore/runtime/Options.cpp

@@ -352 +352 @@
             Options::gcIncrementScale() = 0;
     }
+
+#if PLATFORM(MAC) && CPU(ARM64)
+    Options::numberOfGCMarkers() = 4;
+    Options::numberOfDFGCompilerThreads() = 2;
+    Options::numberOfFTLCompilerThreads() = 2;
+#endif
 
 #if USE(BMALLOC_MEMORY_FOOTPRINT_API)