Commit 766c2ab
build: warn upon --use-largepages config option
Emit a warning when the user configures with `--use-largepages` and/or
`--use-largepages-script-lld` informing them that the option is now
available as a Node.js runtime option once it is built.
Refs: #31063 (comment)
Backport-PR-URL: #31063
PR-URL: #31103
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>1 parent 330eb10 commit 766c2ab
1 file changed
Lines changed: 7 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1040,6 +1040,13 @@ def configure_node(o): | |||
| 1040 | 1040 | else: | |
| 1041 | 1041 | o['variables']['node_use_dtrace'] = 'false' | |
| 1042 | 1042 | ||
| 1043 | + if options.node_use_large_pages or options.node_use_large_pages_script_lld: | ||
| 1044 | + warn('''The `--use-largepages` and `--use-largepages-script-lld` options | ||
| 1045 | + have no effect during build time. Support for mapping to large pages is | ||
| 1046 | + now a runtime option of Node.js. Run `node --use-largepages` or add | ||
| 1047 | + `--use-largepages` to the `NODE_OPTIONS` environment variable once | ||
| 1048 | + Node.js is built to enable mapping to large pages.''') | ||
| 1049 | + | ||
| 1043 | 1050 | if options.no_ifaddrs: | |
| 1044 | 1051 | o['defines'] += ['SUNOS_NO_IFADDRS'] | |
| 1045 | 1052 | ||
0 commit comments