<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Derek,<br>
Looks good to me.<br>
And now I learned how TEST.groups works ;)<br>
Eric<br>
<br>
<br>
<div class="moz-cite-prefix">On 4/28/2015 5:31 PM, Derek White
wrote:<br>
</div>
<blockquote cite="mid:553FFC31.8030101@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
Please review this simple for GC tests that require certain
collectors (e.g. shouldn't run in embedded).<br>
<br>
Certain GC regression tests require specific collectors to be
enabled in the JVM (by using -XX:+UseParallelGC, etc). But these
tests were not listed in TEST.groups.<br>
<br>
This webrev adds the following tests to the appropriate
needs_g1gc, needs_serialgc, needs_parallelgc, and needs_cmsgc
lists.
<ul>
<li>gc/TestSmallHeap.java</li>
<li>gc/TestCardTablePageCommits.java</li>
<li>gc/arguments/TestParallelHeapSizeFlags.java
gc/ergonomics/TestDynamicNumberOfGCThreads.java</li>
<li>gc/logging/TestGCId.java</li>
</ul>
CR:<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8078673">https://bugs.openjdk.java.net/browse/JDK-8078673</a><br>
<br>
Webrev:<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Edrwhite/8078673/webrev.00/">http://cr.openjdk.java.net/~drwhite/8078673/webrev.00/</a><br>
<br>
Testing: <br>
JPRT<br>
<br>
Thanks,<br>
- Derek<br>
</blockquote>
<br>
</body>
</html>