Commit f0f0291
test,doc: skip --max-old-space-size-percentage on 32-bit platforms
PR-URL: #60144
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>1 parent 31a1c95 commit f0f0291
3 files changed
Lines changed: 16 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1685,14 +1685,17 @@ changes: | |||
| 1685 | 1685 | ||
| 1686 | 1686 | Specify the maximum size, in bytes, of HTTP headers. Defaults to 16 KiB. | |
| 1687 | 1687 | ||
| 1688 | - ### `--max-old-space-size-percentage=PERCENTAGE` | ||
| 1688 | + ### `--max-old-space-size-percentage=percentage` | ||
| 1689 | 1689 | ||
| 1690 | - Sets the max memory size of V8's old memory section as a percentage of available system memory. | ||
| 1690 | + Sets the maximum memory size of V8's old memory section as a percentage of available system memory. | ||
| 1691 | 1691 | This flag takes precedence over `--max-old-space-size` when both are specified. | |
| 1692 | 1692 | ||
| 1693 | - The `PERCENTAGE` parameter must be a number greater than 0 and up to 100. representing the percentage | ||
| 1693 | + The `percentage` parameter must be a number greater than 0 and up to 100, representing the percentage | ||
| 1694 | 1694 | of available system memory to allocate to the V8 heap. | |
| 1695 | 1695 | ||
| 1696 | + **Note:** This flag utilizes `--max-old-space-size`, which may be unreliable on 32-bit platforms due to | ||
| 1697 | + integer overflow issues. | ||
| 1698 | + | ||
| 1696 | 1699 | ```bash | |
| 1697 | 1700 | # Using 50% of available system memory | |
| 1698 | 1701 | node --max-old-space-size-percentage=50 index.js | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -333,14 +333,18 @@ The file used to store localStorage data. | |||
| 333 | 333 | Specify the maximum size of HTTP headers in bytes. Defaults to 16 KiB. | |
| 334 | 334 | . | |
| 335 | 335 | .It Fl -max-old-space-size-percentage Ns = Ns Ar percentage | |
| 336 | - Sets the max memory size of V8's old memory section as a percentage of available system memory. | ||
| 336 | + Sets the maximum memory size of V8's old memory section as a percentage of available system memory. | ||
| 337 | 337 | This flag takes precedence over | |
| 338 | 338 | .Fl -max-old-space-size | |
| 339 | 339 | when both are specified. | |
| 340 | 340 | The | |
| 341 | 341 | .Ar percentage | |
| 342 | 342 | parameter must be a number greater than 0 and up to 100, representing the percentage | |
| 343 | 343 | of available system memory to allocate to the V8 heap. | |
| 344 | + .Pp | ||
| 345 | + Note: This flag utilizes | ||
| 346 | + .Fl -max-old-space-size , | ||
| 347 | + which may be unreliable on 32-bit platforms due to integer overflow issues. | ||
| 344 | 348 | . | |
| 345 | 349 | .It Fl -napi-modules | |
| 346 | 350 | This option is a no-op. | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -2,7 +2,11 @@ | |||
| 2 | 2 | ||
| 3 | 3 | // This test validates the --max-old-space-size-percentage flag functionality | |
| 4 | 4 | ||
| 5 | - require('../common'); | ||
| 5 | + const common = require('../common'); | ||
| 6 | + // This flag utilizes --max-old-space-size, which is unreliable on | ||
| 7 | + // 32-bit platforms due to integer overflow issues. | ||
| 8 | + common.skipIf32Bits(); | ||
| 9 | + | ||
| 6 | 10 | const assert = require('node:assert'); | |
| 7 | 11 | const { spawnSync } = require('child_process'); | |
| 8 | 12 | const os = require('os'); | |
0 commit comments