From d8bdf4e049e0a87bb27befd7bd79ba447e3f29b5 Mon Sep 17 00:00:00 2001 From: Gabriel Schulhof Date: Thu, 26 Dec 2019 09:04:43 -0800 Subject: [PATCH] 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: https://github.com/nodejs/node/pull/31063#discussion_r361143460 PR-URL: https://github.com/nodejs/node/pull/31103 Reviewed-By: Richard Lau Reviewed-By: Denys Otrishko --- configure.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/configure.py b/configure.py index 6f0d30cae23434..05db673a496637 100755 --- a/configure.py +++ b/configure.py @@ -1091,6 +1091,13 @@ def configure_node(o): else: o['variables']['node_use_dtrace'] = 'false' + if options.node_use_large_pages or options.node_use_large_pages_script_lld: + warn('''The `--use-largepages` and `--use-largepages-script-lld` options + have 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']