Code Review for hotspot-gc-print

Prepared by:filipp.zhinkin@oracle.com on Tue Sep 10 11:00:14 MSK 2013
Workspace:/home/filipp/Development/hotspot-gc-print
Compare against: http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/
Summary of changes: 924 lines changed: 924 ins; 0 del; 0 mod; 0 unchg
Patch of changes: hotspot-gc-print.patch
Author comments:
Brief overview of tests to be added:

Following tests verify that PrintGC and PrintGCDetails flags work and emit appropriate output. Tests grouped by GC in order to handle different GC logging format.
gc/arguments/print/TestCMSPrintGCFlags
gc/arguments/print/TestG1PrintGCFlags.java
gc/arguments/print/TestSerialAndParallelPrintGCFlags

Test on PrintGCTimeStamps and PrintGCDateStamps flags. It verifies that options and their combinations emit appropriate output and verifies that time and date stamps are not decreasing over time.
gc/arguments/print/TestGCStampFlags

Test on PrintGCApplicationConcurrentTime and PrintGCApplicationStoppedTime. It verifies that these options emit appropriate output.
gc/arguments/print/TestPrintGCAppTimeFlags

Test on PrintGCTaskTimeStamps flah. It verifies that when this option emit output only with parallel GC and also verifies that all timestamps not decreasing and for each GC thread verifies that number of reported taks timestamps equal to expected number of tasks.
gc/arguments/print/TestPrintGCTaskTimeStampsFlag

Test on PrintTenuringDistribution flag. It verifies that appropriate output is emitted when this option is turned on and also verifies that reported values lies between 0 and selected MaxTenuringThreshold.
gc/arguments/print/TestPrintTenuringDistributionFlag
Bug id: 8024319
Legend: Modified file
Deleted file
New file

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestCMSPrintGCFlags.java

47 lines changed: 47 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestG1PrintGCFlags.java

55 lines changed: 55 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestGCStampFlags.java

144 lines changed: 144 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestPrintGCAppTimeFlags.java

84 lines changed: 84 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestPrintGCTaskTimeStampsFlag.java

218 lines changed: 218 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestPrintGCTools.java

211 lines changed: 211 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestPrintTenuringDistributionFlag.java

115 lines changed: 115 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/arguments/print/TestSerialAndParallelPrintGCFlags.java

50 lines changed: 50 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using ../webrev.ksh (vers 23.18-hg).