Commit 39eba0a
cli: add --node-memory-debug option
Add a public switch that turns on features for debugging memory
leaks inside of Node.js core.
PR-URL: #35537
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>1 parent 8b9215d commit 39eba0a
3 files changed
Lines changed: 19 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -537,6 +537,14 @@ added: v6.0.0 | |||
| 537 | 537 | ||
| 538 | 538 | Silence all process warnings (including deprecations). | |
| 539 | 539 | ||
| 540 | + ### `--node-memory-debug` | ||
| 541 | + <!-- YAML | ||
| 542 | + added: REPLACEME | ||
| 543 | + --> | ||
| 544 | + | ||
| 545 | + Enable extra debug checks for memory leaks in Node.js internals. This is | ||
| 546 | + usually only useful for developers debugging Node.js itself. | ||
| 547 | + | ||
| 540 | 548 | ### `--openssl-config=file` | |
| 541 | 549 | <!-- YAML | |
| 542 | 550 | added: v6.9.0 | |
@@ -1278,6 +1286,7 @@ Node.js options that are allowed are: | |||
| 1278 | 1286 | * `--no-deprecation` | |
| 1279 | 1287 | * `--no-force-async-hooks-checks` | |
| 1280 | 1288 | * `--no-warnings` | |
| 1289 | + * `--node-memory-debug` | ||
| 1281 | 1290 | * `--openssl-config` | |
| 1282 | 1291 | * `--pending-deprecation` | |
| 1283 | 1292 | * `--policy-integrity` | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -281,6 +281,10 @@ These will still be enabled dynamically when `async_hooks` is enabled. | |||
| 281 | 281 | .It Fl -no-warnings | |
| 282 | 282 | Silence all process warnings (including deprecations). | |
| 283 | 283 | . | |
| 284 | + .It Fl -node-memory-debug | ||
| 285 | + Enable extra debug checks for memory leaks in Node.js internals. This is | ||
| 286 | + usually only useful for developers debugging Node.js itself. | ||
| 287 | + . | ||
| 284 | 288 | .It Fl -openssl-config Ns = Ns Ar file | |
| 285 | 289 | Load an OpenSSL configuration file on startup. | |
| 286 | 290 | Among other uses, this can be used to enable FIPS-compliant crypto if Node.js is built with | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -770,6 +770,12 @@ PerProcessOptionsParser::PerProcessOptionsParser( | |||
| 770 | 770 | kAllowedInEnvironment); | |
| 771 | 771 | ||
| 772 | 772 | Insert(iop, &PerProcessOptions::get_per_isolate_options); | |
| 773 | + | ||
| 774 | + AddOption("--node-memory-debug", | ||
| 775 | + "Run with extra debug checks for memory leaks in Node.js itself", | ||
| 776 | + NoOp{}, kAllowedInEnvironment); | ||
| 777 | + Implies("--node-memory-debug", "--debug-arraybuffer-allocations"); | ||
| 778 | + Implies("--node-memory-debug", "--verify-base-objects"); | ||
| 773 | 779 | } | |
| 774 | 780 | ||
| 775 | 781 | inline std::string RemoveBrackets(const std::string& host) { | |
0 commit comments