On Wed, 27 Sep 2023 08:18:45 GMT, Matthias Baesken <[email protected]> wrote:
> Running jtreg tests with make test, for example > make test TEST="jtreg:test/jdk/java/util/prefs" fails currently on AIX > without setting the number of JOBS manually. > We get this error message: > Error: Bad use of -concurrency > > Log of cmdargs shows : > -vmoption:-Xmx768m > -agentvm > -verbose:fail,error,summary > -retain:fail,error > -concurrency:7.000000 > .... > > So currently a float is used for the concurrency value, seems this is not > supported. > Reason is an old gawk printing float values instead of the needed ints, so > better print ints in the makefile by using %d . As was pointed out in the bug, this changes the meaning of the expression to always round down rather than rounding to nearest integer, which was my intention when I originally added this logic. For higher cpu counts, this doesn't matter much, but for smaller systems, it can make a big difference if we run with 1 or 2 jobs. To work around this, perhaps add `0.5` to c before converting to int? ------------- PR Comment: https://git.openjdk.org/jdk/pull/15941#issuecomment-1737782985
