> On May 10, 2016, at 12:00 PM, Alexandre (Shura) Iline > <alexandre.il...@oracle.com> wrote: > > Hi. > > Please review the fix: > http://cr.openjdk.java.net/~shurailine/8155993/webrev.00/ > > In the failing tests I have replaced java.management with jdk.management in > @modules. > > Besides the failed tests I have scanned all the tests which call > ManagementFactory.getPlatformMBeanServer(…). Those which were not failing > were already declaring dependency on jdk.management, sometimes together with > java.management. Declaring java.management in such case is not needed, > because java.management is a dependency of jdk.management.
In fact JDK-8155993 reveals a potential bug in the VM support for java.lang.management when jdk.management module is not present. I have no objection to change these tests to run with jdk.management and this patch looks fine. But we should keep JDK-8155993 open and provide a small test to reproduce NCDFE when running with an image with only java.management but no jdk.management module. Mandy
