<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Hi Stefan,<br>
<br>
Looks fine to me.<br>
<br>
One minor point - this will affect the reg test I sent out for
review yesterday which explicitly looks for "[Verifying..."<br>
<br>
JohnC<br>
<br>
<div class="moz-cite-prefix">On 4/24/2013 11:12 AM, Stefan Karlsson
wrote:<br>
</div>
<blockquote cite="mid:5178207E.1080209@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Estefank/8013132/webrev.00/">http://cr.openjdk.java.net/~stefank/8013132/webrev.00/</a><br>
<br>
This RFE adds a new flag named VerifySilently, which can be used
to turn off the verbose output that the verification code
produces. The proposed name has been chosen to match other Verify
flags.<br>
<br>
From the Bug description:<br>
---<br>
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
The HotSpot verification code is rather verbose and intrusive. I
propose that we add a flag that can be used to turn this output
off. <br>
<br>
An example of the verification output: <br>
<tt>$ java -XX:+PrintGC -XX:+UnlockDiagnosticVMOptions
-XX:+VerifyBeforeExit -XX:+VerifyAfterGC -XX:+VerifyBeforeGC
-XX:+UseConcMarkSweepGC -XX:+ExplicitGCInvokesConcurrent -cp
/localhome/tests/ HelloSystemGC </tt><tt><br>
</tt><tt> VerifyBeforeGC:[Verifying threads heap concurrent
mark-sweep generation par new generation remset syms strs zone
dict cldg metaspace chunks hand C-heap code cache ] </tt><tt><br>
</tt><tt> [GC (System.gc()) 672K->260K(121600K), 0.0060920
secs] </tt><tt><br>
</tt><tt> VerifyAfterGC:[Verifying threads heap concurrent
mark-sweep generation par new generation remset syms strs zone
dict cldg metaspace chunks hand C-heap code cache ] </tt><tt><br>
</tt><tt> [Verifying threads heap concurrent mark-sweep generation
par new generation remset syms strs zone dict cldg metaspace
chunks hand C-heap code cache ] </tt><tt><br>
</tt><tt> [GC (CMS Initial Mark) 260K(121600K), 0.0204040 secs] </tt><tt><br>
</tt><tt> [Verifying threads heap concurrent mark-sweep generation
par new generation remset syms strs zone dict cldg metaspace
chunks hand C-heap code cache ] </tt><tt><br>
</tt><tt> [Verifying threads heap concurrent mark-sweep generation
par new generation remset syms strs zone dict cldg metaspace
chunks hand C-heap code cache ] </tt><tt><br>
</tt><tt> [GC (CMS Final Remark) [Verifying CMS Marking... done]
1604K(121600K), 0.1008210 secs] </tt><tt><br>
</tt><tt> [Verifying threads heap concurrent mark-sweep generation
par new generation remset syms strs zone dict cldg metaspace
chunks hand C-heap code cache ] </tt><br>
<br>
And with the proposed flag turned on: <br>
<tt>[GC (System.gc()) 672K->269K(121600K), 0.0050710 secs] </tt><tt><br>
</tt><tt> [GC (CMS Initial Mark) 269K(121600K), 0.0203380 secs] </tt><tt><br>
</tt><tt> [GC (CMS Final Remark) 1613K(121600K), 0.1007210 secs] </tt><br>
---<br>
<br>
thanks,<br>
StefanK<br>
</blockquote>
<br>
</body>
</html>