RFR(S): 8155964 - Create a set of tests for verifying the Minimal VM

[David Holmes]

On 23/08/2016 8:47 PM, Christian Tornqvist wrote:
> Hi Chris,
>
>> Why does Instrumentation.java not have @requires minimal?
> It should, I'll correct this.
>
>> Why does JMX.java have:
>>
>> 28  * @run main/othervm -minimal JMX
> This was done to make sure the parent process was running minimal for the jcmd check at line 50. This isn't needed, I'll remove it.

Hmm I thought it was needed else the parent VM will show for the jcmd 
request. ??

David
-----

> Thanks,
> Christian
>
> -----Original Message-----
> From: Chris Plummer [mailto:chris.plummer at oracle.com]
> Sent: Monday, August 22, 2016 5:15 PM
> To: Christian Tornqvist <christian.tornqvist at oracle.com>; hotspot-runtime-dev at openjdk.java.net
> Subject: Re: RFR(S): 8155964 - Create a set of tests for verifying the Minimal VM
>
> Hi Christian,
>
> Overall it looks good. Thanks for adding these tests. Just a few questions:
>
> Why does Instrumentation.java not have @requires minimal?
>
> Why does JMX.java have:
>
> 28  * @run main/othervm -minimal JMX
>
> Have you tested with a jre that only has minimalvm and java.base?
>
> thanks,
>
> Chris
>
> On 8/22/16 12:03 PM, Christian Tornqvist wrote:
>> Hi everyone,
>>
>>
>>
>> Please review this change that adds a set of tests for the Minimal
>> variant of the JVM. The Minimal JVM is a subset that excludes some
>> functionality, the tests here are intended to test verify that trying
>> to use this excluded functionality doesn't lead to any unexpected errors/crashes.
>>
>>
>>
>> Verified by running the tests on Linux ARMv7 and Linux x86.
>>
>>
>>
>> Webrev:
>>
>> http://cr.openjdk.java.net/~ctornqvi/webrev/8155964/webrev.00/
>>
>>
>>
>> Bug (unfortunately not visible):
>>
>> https://bugs.openjdk.java.net/browse/JDK-8155964
>>
>>
>>
>> Thanks,
>>
>> Christian
>>
>>
>>
>
>