Title: [255314] trunk/Source/_javascript_Core
Revision
255314
Author
[email protected]
Date
2020-01-28 14:19:03 -0800 (Tue, 28 Jan 2020)

Log Message

Some website needs more stack space.
https://bugs.webkit.org/show_bug.cgi?id=206891

Reviewed by Saam Barati.

We're bumping the maximum stack usage limit (JSC_maxPerThreadStackUsage) to 5M
to give websites a little more stack space when available.  The actual useable
stack space is still the minimum of JSC_maxPerThreadStackUsage and the stack size
that the OS provisions, minus some overhead.

* runtime/OptionsList.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (255313 => 255314)


--- trunk/Source/_javascript_Core/ChangeLog	2020-01-28 22:17:49 UTC (rev 255313)
+++ trunk/Source/_javascript_Core/ChangeLog	2020-01-28 22:19:03 UTC (rev 255314)
@@ -1,3 +1,17 @@
+2020-01-28  Mark Lam  <[email protected]>
+
+        Some website needs more stack space.
+        https://bugs.webkit.org/show_bug.cgi?id=206891
+
+        Reviewed by Saam Barati.
+
+        We're bumping the maximum stack usage limit (JSC_maxPerThreadStackUsage) to 5M
+        to give websites a little more stack space when available.  The actual useable
+        stack space is still the minimum of JSC_maxPerThreadStackUsage and the stack size
+        that the OS provisions, minus some overhead.
+
+        * runtime/OptionsList.h:
+
 2020-01-27  Carlos Garcia Campos  <[email protected]>
 
         [GTK] Remote Inspector: add support for service workers targets

Modified: trunk/Source/_javascript_Core/runtime/OptionsList.h (255313 => 255314)


--- trunk/Source/_javascript_Core/runtime/OptionsList.h	2020-01-28 22:17:49 UTC (rev 255313)
+++ trunk/Source/_javascript_Core/runtime/OptionsList.h	2020-01-28 22:19:03 UTC (rev 255314)
@@ -90,7 +90,7 @@
     \
     v(Bool, reportMustSucceedExecutableAllocations, false, Normal, nullptr) \
     \
-    v(Unsigned, maxPerThreadStackUsage, 4 * MB, Normal, "Max allowed stack usage by the VM") \
+    v(Unsigned, maxPerThreadStackUsage, 5 * MB, Normal, "Max allowed stack usage by the VM") \
     v(Unsigned, softReservedZoneSize, 128 * KB, Normal, "A buffer greater than reservedZoneSize that reserves space for stringifying exceptions.") \
     v(Unsigned, reservedZoneSize, 64 * KB, Normal, "The amount of stack space we guarantee to our clients (and to interal VM code that does not call out to clients).") \
     \
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to