On Thu, Dec 17, 2020 at 11:19 PM Stephen Hemminger
[off-list ref] wrote:

On Thu, 17 Dec 2020 16:16:16 +0500
Sarosh Arif [off-list ref] wrote:

quoted

+    if get_hugepages(path) != pages:
+        print("Unable to reserve required pages. The pages reserved are:")
+        global SHOW_HUGEPAGES
+        SHOW_HUGEPAGES = True

Please don't add global's to this script.

The script is close to being clean according to pylint, and globals
are considered bad style and shouldn't be used.

I would just exit if huge pages could not be setup.

How about if we just print a warning message such as "Unable to
reserve required pages" before exiting, in case the pages are not
reserved due to lack of space in RAM? Then leave it upon the user to
query how many pages are actually reserved.

The script should leave it up to the user to do another query about
status if they care about what the result is.