RRF(XS)(10): 8176797: [TESTBUG] tools/launcher/Settings.java -Xss size is too small

[Christian Tornqvist]

Hi Chris,

This looks good, thanks for fixing this.

Thanks,
Christian

-----Original Message-----
From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of Chris Plummer
Sent: Thursday, March 16, 2017 12:59 AM
To: hotspot-runtime-dev <hotspot-runtime-dev at openjdk.java.net>; core-libs-dev at openjdk.java.net
Subject: RRF(XS)(10): 8176797: [TESTBUG] tools/launcher/Settings.java -Xss size is too small

Hello,

Please review the following:

https://bugs.openjdk.java.net/browse/JDK-8176797
http://cr.openjdk.java.net/~cjplummer/8176797/webrev.00/webrev.jdk

After fixing 8175342 (see the other RFR I just posted), this test started to fail instead of assert. The problem is 256000 is too small of a stack size on the platform in question. It needs to be at least 256k.

The test has a "stackSize" local variable that is the String "256". It runs the jvm with -Xss of this size, appending "k" to it on one run and "000" in the next. The -Xss256k version is fine. The -Xss256000 version is not. I'm not sure why the test chose to specify the stackSize as a String instead of an int. As part of this fix I changed it to an int, and just multiply it by 1024 in the case where -Xss is specified with a byte size rather the an k size. The test passes now on the platform where it was failing.

thanks,

Chris