[10] RFR(XS): 8183534: [TEST] Make detection of compilation mode more robust

Langer, Christoph christoph.langer at sap.com
Wed Jul 5 09:33:03 UTC 2017

Hi Volker,

overall, the change looks good to me.

When you create the Exception to print the stack trace (line 129 in the new file), why not add the value of 'java.vm.info' to the message? Also, I don't like the spaces around the '.' before PrintStackTrace() - but maybe that's just my personal taste. ;-)

Best regards

> -----Original Message-----
> From: core-libs-dev [mailto:core-libs-dev-bounces at openjdk.java.net] On
> Behalf Of Volker Simonis
> Sent: Dienstag, 4. Juli 2017 11:05
> To: HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>;
> Java Core Libs <core-libs-dev at openjdk.java.net>
> Subject: [10] RFR(XS): 8183534: [TEST] Make detection of compilation mode
> more robust
> Hi,
> can you please review the following trivial change which makes the
> detection of compilation mode for JTreg more robust:
> http://cr.openjdk.java.net/~simonis/webrevs/2017/8183534/
> https://bugs.openjdk.java.net/browse/JDK-8183534
> The compilation mode in JTreg is parsed from the "java.vm.info" system
> property. The current implementation expects that "java.vm.info"
> contains exactly one of the strings "mixed mode", "compiled mode" or
> "interpreted mode" otherwise the detection will fail and JTreg will
> quit with an error.
> The detection can be done more robust by searching for the substrings
> "mixed mode", "compiled mode" or "interpreted mode" instead, thus
> allowing OpenJDK builders to store additional information in the
> "java.vm.info" system property.
> Thank you and best regards,
> Volker

More information about the core-libs-dev mailing list