RFR(S): 8164944: Refactor ProcessTools to get rid of dependency on java.management

Igor Ignatyev igor.ignatyev at oracle.com
Wed Apr 12 19:03:15 UTC 2017


http://cr.openjdk.java.net/~iignatyev//8164944/webrev.00/index.html
> 92 lines changed: 36 ins; 36 del; 20 mod; 

Hi all,

could you please review this small fix which removes dependency on java.management module from the testlibrary?

jdk.test.lib.process.ProcessTools class has getVmInputArgs method which depend on java.management, this method is used only by j.d.t.process.ProcessTools::executeTestJvmAll and j.t.l.cli.CommandLineOptionTest class.
 - ProcessTools::executeTestJvmAll is a variation of executeTestJvm which prepends vm flags from jtreg @run.
 - CommandLineOptionTest is a library class for cli testing, it uses flags from ProcessTools::getVmInputArgs to create a new jvm with all flags
 
the fix removes ProcessTools::executeTestJvmAll method, moves getVmInputArgs to a separate class jdk.test.lib.management.InputArguments. c.c.s.scenario.Executor and j.t.l.cli.CommandLineOptionTest are updated to use getVmInputArgs from that class. c.j.compilerToVM.DebugOutputTest is refactored to pass add-exports and patch module directly. all the affected classes and obviously updated to use executeTestJvm instead of executeTestJvmAll.

after this fix many hotspot tests won't depend on java.management, hence that module might be removed from their @modules, this will be done separately by 8178416.

jbs: https://bugs.openjdk.java.net/browse/JDK-8164944
webrev: http://cr.openjdk.java.net/~iignatyev//8164944/webrev.00/index.html
testing: :hotspot_all
8178416: https://bugs.openjdk.java.net/browse/JDK-8178416

Thanks,
-- Igor


More information about the hotspot-compiler-dev mailing list