Commit ad2c1bf
authored
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 db0121b commit ad2c1bf
3 files changed
Lines changed: 16 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1765,14 +1765,17 @@ changes: | |||
| 1765 | 1765 | ||
| 1766 | 1766 | Specify the maximum size, in bytes, of HTTP headers. Defaults to 16 KiB. | |
| 1767 | 1767 | ||
| 1768 | - ### `--max-old-space-size-percentage=PERCENTAGE` | ||
| 1768 | + ### `--max-old-space-size-percentage=percentage` | ||
| 1769 | 1769 | ||
| 1770 | - Sets the max memory size of V8's old memory section as a percentage of available system memory. | ||
| 1770 | + Sets the maximum memory size of V8's old memory section as a percentage of available system memory. | ||
| 1771 | 1771 | This flag takes precedence over `--max-old-space-size` when both are specified. | |
| 1772 | 1772 | ||
| 1773 | - The `PERCENTAGE` parameter must be a number greater than 0 and up to 100. representing the percentage | ||
| 1773 | + The `percentage` parameter must be a number greater than 0 and up to 100, representing the percentage | ||
| 1774 | 1774 | of available system memory to allocate to the V8 heap. | |
| 1775 | 1775 | ||
| 1776 | + **Note:** This flag utilizes `--max-old-space-size`, which may be unreliable on 32-bit platforms due to | ||
| 1777 | + integer overflow issues. | ||
| 1778 | + | ||
| 1776 | 1779 | ```bash | |
| 1777 | 1780 | # Using 50% of available system memory | |
| 1778 | 1781 | node --max-old-space-size-percentage=50 index.js | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -339,14 +339,18 @@ The file used to store localStorage data. | |||
| 339 | 339 | Specify the maximum size of HTTP headers in bytes. Defaults to 16 KiB. | |
| 340 | 340 | . | |
| 341 | 341 | .It Fl -max-old-space-size-percentage Ns = Ns Ar percentage | |
| 342 | - Sets the max memory size of V8's old memory section as a percentage of available system memory. | ||
| 342 | + Sets the maximum memory size of V8's old memory section as a percentage of available system memory. | ||
| 343 | 343 | This flag takes precedence over | |
| 344 | 344 | .Fl -max-old-space-size | |
| 345 | 345 | when both are specified. | |
| 346 | 346 | The | |
| 347 | 347 | .Ar percentage | |
| 348 | 348 | parameter must be a number greater than 0 and up to 100, representing the percentage | |
| 349 | 349 | of available system memory to allocate to the V8 heap. | |
| 350 | + .Pp | ||
| 351 | + Note: This flag utilizes | ||
| 352 | + .Fl -max-old-space-size , | ||
| 353 | + which may be unreliable on 32-bit platforms due to integer overflow issues. | ||
| 350 | 354 | . | |
| 351 | 355 | .It Fl -napi-modules | |
| 352 | 356 | 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