<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<br>
<div class="moz-cite-prefix">On 12/21/2015 4:40 PM, Poonam Bajaj
Parhar wrote:<br>
</div>
<blockquote cite="mid:56789BED.3020304@oracle.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
Hello Jon,<br>
<br>
<div class="moz-cite-prefix">On 12/21/2015 2:29 PM, Jon Masamitsu
wrote:<br>
</div>
<blockquote cite="mid:56787D5B.50806@oracle.com" type="cite">
<meta content="text/html; charset=utf-8"
http-equiv="Content-Type">
<font face="Times New Roman, Times, serif">Poonam,<br>
<br>
Some of the string list parameters allow separation by , or "
". Did<br>
you consider adding that?<br>
<br>
</font></blockquote>
<font face="Times New Roman, Times, serif">The </font>VerifySubSet
string list can accept the strings separated by comma or a space.
Before verification, we just need to check if a particular subset
is present in the VerifySubSet list or not. We don't need any
complicated processing of the strings list here.<br>
</blockquote>
<br>
Ok. Thanks for the explanation.<br>
<br>
You mention in the RFR that you have a simple test to test the<br>
change. Can you add that as a jtreg test? I'm thinking about<br>
something like<br>
<br>
test/gc/TestVerifyDuringStartup.java<br>
test/gc/TestVerifySilently.java<br>
<br>
Jon<br>
<br>
<blockquote cite="mid:56789BED.3020304@oracle.com" type="cite"> <br>
Thanks,<br>
Poonam<br>
<br>
<blockquote cite="mid:56787D5B.50806@oracle.com" type="cite"><font
face="Times New Roman, Times, serif"> Jon<br>
<br>
PS. See share/vm/compiler/compilerDirectives.cpp<br>
<br>
ccstrlist
DirectiveSet::canonicalize_disableintrinsic(ccstrlist
option_value)<br>
<br>
</font><br>
<div class="moz-cite-prefix">On 12/15/2015 03:15 PM, Poonam
Bajaj Parhar wrote:<br>
</div>
<blockquote cite="mid:56709EF7.9020003@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html;
charset=utf-8">
Hello,<br>
<br>
Please review these changes that split up the work done under
Verify*GC options. This will be very useful in debugging GC
issues/crashes where verifying the whole memory system with
Verify*GC options slows down the process and makes it
impossible to reproduce the problem.<br>
<br>
The changes introduce a new option <i>VerifySubSet</i><i> </i>that
can be used to specify the specific memory sub-systems that
one wants to verify. It can be one or more of the sub-systems
from these: threads, heap, symbol_table, string_table,
codecache, dictionary, classloader_data_graph, metaspace,
jni_handles, c-heap and codecache_oops. If nothing is
specified with <i>VerifySubSet</i> then whole of the memory
system is verified with Verify*GC options.<br>
<br>
Bug:<a moz-do-not-send="true" class="issue-link"
data-issue-key="JDK-8072725"
href="https://bugs.openjdk.java.net/browse/JDK-8072725"
id="key-val" rel="4767373"> JDK-8072725:</a> Provide more
granular levels for GC verification<br>
Webrev: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Epoonam/8072725/webrev.00/">http://cr.openjdk.java.net/~poonam/8072725/webrev.00/</a><br>
Testing: JPRT, tested the changes with a simple test program.<br>
<br>
Thanks,<br>
Poonam<br>
<br>
<br>
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>