<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Hi Mandy,<br>
<br>
<div class="moz-cite-prefix">On 08.11.2014 02:41, Mandy Chung wrote:<br>
</div>
<blockquote cite="mid:545D58BE.3030404@oracle.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
<div class="moz-cite-prefix">On 11/7/14 3:34 AM, Evgeniya
Stepanova wrote:<br>
</div>
<blockquote cite="mid:545CAE30.9010708@oracle.com" type="cite">
<meta content="text/html; charset=utf-8"
http-equiv="Content-Type">
<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eeistepan/8062536/webrev.02/">http://cr.openjdk.java.net/~eistepan/8062536/webrev.02/</a><br>
</blockquote>
<br>
What are the valid values of<span class="changed"> vm.gc? It's a
bit awk for null as a valid value and I would imagine "*"
wildcard may be a better value?<br>
</span></blockquote>
<br>
vm.gc variable is set by jtreg based on passed -XX:+Use???GC command
line vm flag.<br>
If such flag is not given vm.gc will be set to null (or "null" which
is same).<br>
<br>
So, vm.gc could be set to any value. In reality the following
strings are expected: null, "Parallel", "Serial", "G1",
"ConcMarkSweep".<br>
<br>
null means "not set", it doesn't mean "any".<br>
If a test is applicable for all GC - you don't need @requires vm.gc
== "*"<br>
<br>
<blockquote cite="mid:545D58BE.3030404@oracle.com" type="cite"><span
class="changed"> <br>
Since you are touching test/java/lang/management/** tests, we
can remove these shell tests and add multiple @run in the java
test instead. It'd be great if you can take this opportunity to
remove these shell tests as they are not really necessary.<br>
</span></blockquote>
<br>
Yes, agree, shell is not necessary here. But such refactoring is
better to be done as part of another RFE.<br>
<br>
Thanks,<br>
Dima<br>
<br>
<br>
<blockquote cite="mid:545D58BE.3030404@oracle.com" type="cite"><span
class="changed"> <br>
thanks<br>
Mandy<br>
</span> </blockquote>
<br>
</body>
</html>