Skip to content

Commit

Permalink
build: warn upon --use-largepages config option
Browse files Browse the repository at this point in the history
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: nodejs#31063 (comment)
PR-URL: nodejs#31103
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
  • Loading branch information
Gabriel Schulhof committed Mar 6, 2020
1 parent 7075e9d commit 88b76d1
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -1024,6 +1024,13 @@ def configure_node(o):
else:
o['variables']['node_use_dtrace'] = 'false'

if options.node_use_large_pages:
warn('''The `--use-largepages` option has no effect during build time.
Support for mapping to large pages is now a runtime option of Node.js.
Run `node --use-largepages` or add `--use-largepages` to the
`NODE_OPTIONS` environment variable once Node.js is built to enable
mapping to large pages.''')

if options.no_ifaddrs:
o['defines'] += ['SUNOS_NO_IFADDRS']

Expand Down

0 comments on commit 88b76d1

Please sign in to comment.