RFR: 8272854: split runtime/CommandLine/PrintTouchedMethods.java test
    David Holmes 
    dholmes at openjdk.java.net
       
    Tue Aug 24 07:21:25 UTC 2021
    
    
  
On Mon, 23 Aug 2021 21:30:07 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:
> Hi all,
> 
> could you please review this patch that extracts part of `runtime/CommandLine/PrintTouchedMethods.java` test to `runtime/CommandLine/PrintTouchedMethodsJcmd.java`?
> 
> from JBS:
>> part of `runtime/CommandLine/PrintTouchedMethods.java` test can be done in a driver mode and ignore external flags, while another part should be executed in an othervm mode. to make execution mode efficient, the test can be split into two.
> 
> testing: ``runtime/CommandLine/` on `{linux,windows,macos}-x64`
> 
> -- Igor
test/hotspot/jtreg/runtime/CommandLine/PrintTouchedMethodsJcmd.java line 42:
> 40:       var pid = Long.toString(ProcessHandle.current().pid());
> 41:       var pb = new ProcessBuilder();
> 42:       pb.command(new String[] {JDKToolFinder.getJDKTool("jcmd"), pid, "VM.print_touched_methods"});
Can't you just include the desired VM flags as part of the String[] here and keep everything in one file and using driver mode?
-------------
PR: https://git.openjdk.java.net/jdk/pull/5231
    
    
More information about the hotspot-runtime-dev
mailing list