2024-10-07 18:06:47 -05:00
|
|
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
|
|
From: Omer Katz <omerkatz@chromium.org>
|
|
|
|
Date: Thu, 19 Sep 2024 10:50:09 +0200
|
|
|
|
Subject: cli: remove deprecated V8 flag
|
|
|
|
MIME-Version: 1.0
|
|
|
|
Content-Type: text/plain; charset=UTF-8
|
|
|
|
Content-Transfer-Encoding: 8bit
|
|
|
|
|
|
|
|
Remove the `--huge-max-old-generation-size` V8 flag from the
|
|
|
|
`NODE_OPTIONS` allowlist. That flag was recently deprecated (it
|
|
|
|
currently remains as nop, see crrev.com/c/5831467) and will soon be
|
|
|
|
completely removed.
|
|
|
|
|
|
|
|
PR-URL: https://github.com/nodejs/node/pull/54761
|
|
|
|
Reviewed-By: Richard Lau <rlau@redhat.com>
|
|
|
|
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
|
|
|
|
Reviewed-By: Michaël Zasso <targos@protonmail.com>
|
|
|
|
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
|
|
|
|
|
|
|
|
diff --git a/doc/api/cli.md b/doc/api/cli.md
|
2025-05-05 10:23:17 +02:00
|
|
|
index 1b42c5a7f4715e56fa5bc39cd6f78a76473406f2..114b7bbf6b1e105fc1696ed8a064065db73ff519 100644
|
2024-10-07 18:06:47 -05:00
|
|
|
--- a/doc/api/cli.md
|
|
|
|
+++ b/doc/api/cli.md
|
2025-05-05 10:23:17 +02:00
|
|
|
@@ -3350,7 +3350,6 @@ V8 options that are allowed are:
|
2024-10-07 18:06:47 -05:00
|
|
|
* `--disallow-code-generation-from-strings`
|
|
|
|
* `--enable-etw-stack-walking`
|
2024-10-08 09:54:33 -05:00
|
|
|
* `--expose-gc`
|
2024-10-07 18:06:47 -05:00
|
|
|
-* `--huge-max-old-generation-size`
|
|
|
|
* `--interpreted-frames-native-stack`
|
|
|
|
* `--jitless`
|
|
|
|
* `--max-old-space-size`
|
|
|
|
diff --git a/src/node_options.cc b/src/node_options.cc
|
2025-05-05 10:23:17 +02:00
|
|
|
index b22fbb0a285f6f323779d6ebb2b027a3990b031e..54b253aa54f5cdebdb04315f9c6c2506977555c0 100644
|
2024-10-07 18:06:47 -05:00
|
|
|
--- a/src/node_options.cc
|
|
|
|
+++ b/src/node_options.cc
|
2025-05-05 10:23:17 +02:00
|
|
|
@@ -984,11 +984,6 @@ PerIsolateOptionsParser::PerIsolateOptionsParser(
|
2024-10-07 18:06:47 -05:00
|
|
|
"disallow eval and friends",
|
|
|
|
V8Option{},
|
|
|
|
kAllowedInEnvvar);
|
|
|
|
- AddOption("--huge-max-old-generation-size",
|
|
|
|
- "increase default maximum heap size on machines with 16GB memory "
|
|
|
|
- "or more",
|
|
|
|
- V8Option{},
|
|
|
|
- kAllowedInEnvvar);
|
|
|
|
AddOption("--jitless",
|
|
|
|
"disable runtime allocation of executable memory",
|
|
|
|
V8Option{},
|
|
|
|
diff --git a/test/parallel/test-cli-node-options.js b/test/parallel/test-cli-node-options.js
|
2025-05-05 10:23:17 +02:00
|
|
|
index c5d74f40e7894980b45713c77cc36f836be73528..53bca572c3405c0357f868aae71fc2c82d973c04 100644
|
2024-10-07 18:06:47 -05:00
|
|
|
--- a/test/parallel/test-cli-node-options.js
|
|
|
|
+++ b/test/parallel/test-cli-node-options.js
|
2025-05-05 10:23:17 +02:00
|
|
|
@@ -76,7 +76,6 @@ if (common.hasCrypto) {
|
2024-10-07 18:06:47 -05:00
|
|
|
expect('--abort_on-uncaught_exception', 'B\n');
|
|
|
|
expect('--disallow-code-generation-from-strings', 'B\n');
|
2024-10-08 09:54:33 -05:00
|
|
|
expect('--expose-gc', 'B\n');
|
2024-10-07 18:06:47 -05:00
|
|
|
-expect('--huge-max-old-generation-size', 'B\n');
|
|
|
|
expect('--jitless', 'B\n');
|
|
|
|
expect('--max-old-space-size=0', 'B\n');
|
|
|
|
expect('--max-semi-space-size=0', 'B\n');
|