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