RFR: JDK-8192985: SA: Test cases for the clhsdb 'inspect', 'scanoops' and 'printas' commands

Sharath Ballal sharath.ballal at oracle.com
Mon Dec 11 10:11:10 UTC 2017


Hi Jini,
Looks Good. Some nits:

http://cr.openjdk.java.net/~jgeorge/8192985/webrev.00/test/hotspot/jtreg/serviceability/sa/ClhsdbInspect.java.html

Since you are not passing any new VM options, the following lines 

   28 	    import jdk.test.lib.Utils;
  47             List<String> vmArgs = new ArrayList<String>();
  48             vmArgs.addAll(Utils.getVmOptions());
  49 
  50             theApp = new LingeredAppWithLock();
  51             LingeredApp.startApp(vmArgs, theApp);

Can be replaced by

theApp = new LingeredAppWithLock();
LingeredApp.startApp(null, theApp);

http://cr.openjdk.java.net/~jgeorge/8192985/webrev.00/test/hotspot/jtreg/serviceability/sa/ClhsdbScanOops.java.html

41     public static void testWithGcType
If you are not planning on this method being called from elsewhere, you can make it private.

Thanks,
Sharath


-----Original Message-----
From: Jini George 
Sent: Friday, December 08, 2017 12:33 PM
To: serviceability-dev at openjdk.java.net
Subject: RFR: JDK-8192985: SA: Test cases for the clhsdb 'inspect', 'scanoops' and 'printas' commands

Hello,

Requesting reviews for:

JBS Id: https://bugs.openjdk.java.net/browse/JDK-8192985
Webrev: http://cr.openjdk.java.net/~jgeorge/8192985/webrev.00/

These are the new test cases for the following clhsdb commands:
1. inspect
2. scanoops
3. printas

These tests have been verified through the Mach5 and jprt systems.

Thanks,
Jini.



More information about the serviceability-dev mailing list