Skip to content

Commit f5cd6d7

Browse files
Gabriel SchulhofMylesBorins
Gabriel Schulhof
authored andcommitted
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 <[email protected]> Reviewed-By: Denys Otrishko <[email protected]>
1 parent 5e232f5 commit f5cd6d7

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

configure.py

+7
Original file line numberDiff line numberDiff line change
@@ -1040,6 +1040,13 @@ def configure_node(o):
10401040
else:
10411041
o['variables']['node_use_dtrace'] = 'false'
10421042

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+
10431050
if options.no_ifaddrs:
10441051
o['defines'] += ['SUNOS_NO_IFADDRS']
10451052

0 commit comments

Comments
 (0)