RFR(XS): 8161695 - compiler/jsr292/MHInlineTest.java can't be run on client-only platforms
leonid.mesnik at oracle.com
Tue Jul 19 17:35:25 UTC 2016
I think such check could be easily implemented now like it is done for
G1 in the:
The new property vm.supportedFlavors or vm.isServer could be added
which is true only when tested JVM could be executed in server mode and
-client/-minimal options are not set explicitly.
On 19.07.2016 19:44, Dmitrij Pochepko wrote:
> Thank you for quick reply.
> Regarding checks. This test can't check for server vm before forking
> process. The only information available is current vm properties. This
> test can be launched with client vm and then fork a process with
> server vm.
> Another usecase is when no server vm is available but test code is
> launched via client vm and it still has exit code check, skipping
> process output checks, which is weaker checks, but it still can find
> Basically, this test can be modified by adding: @require vm.flavor =
> but this will reduce test coverage by cases above. I can do it in such
> manner in case these cases are not valuable enough.
>> I see that test checks Server VM only for output analysis. I think it
>> should be done before forking process.
>> On 7/19/16 7:46 AM, Dmitrij Pochepko wrote:
>>> please review fix for 8161695 - compiler/jsr292/MHInlineTest.java can't
>>> be run on client-only platforms
>>> Test launched child process with -server, which is failed on platforms
>>> with client vm only. A fix is to remove -server, since test code checks
>>> server vm presence.
>>> webrev: http://cr.openjdk.java.net/~dpochepk/8161695/webrev.01/
>>> CR: https://bugs.openjdk.java.net/browse/JDK-8161695
More information about the hotspot-compiler-dev