On Thu, 17 Dec 2020 16:16:16 +0500
Sarosh Arif <sarosh.a...@emumba.com> wrote:
> + 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.
The script should leave it up to the user to do another query about
status if they care about what the result is.
