RFR: 8302155: [AIX] NUM_LCPU is not required variable

Tyler Steele tsteele at openjdk.org
Fri Feb 10 17:02:50 UTC 2023


On Thu, 9 Feb 2023 15:21:42 GMT, Amit Kumar <duke at openjdk.org> wrote:

> We don't need NUM_LCPU, it's just a small code change which removes this variable. I've tested it on AIX and cores are detected successfully with changes. 
> @backwaterred please take look at it. 
>  Suggestions are welcomed :)

I wanted to point out these changes are semantically different in the error case. If we don't get the output we expect from lparstat we potentially overwrite the NUM_CORES var with the empty string. Previously, the code protected against this by testing if the result stored in the temp variable was non-empty before committing to them.

Checking the [documentation](https://www.ibm.com/docs/en/aix/7.2?topic=l-lparstat-command) for AIX 7.1-7.3, it seems like the syntax is stable, so removing this check seems reasonable to me.

-------------

Marked as reviewed by tsteele (Committer).

PR: https://git.openjdk.org/jdk/pull/12494



More information about the build-dev mailing list