<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Hi<br>
<br>
During reviewing this I noticed that java -X was not updated. It
shows:<br>
-->/java/re/jdk/8/promoted/all/b76/binaries/linux-amd64/bin/java
-X<br>
-Xmixed mixed mode execution (default)<br>
-Xint interpreted mode execution only<br>
-Xbootclasspath:<directories and zip/jar files separated by
:><br>
set search path for bootstrap classes and
resources<br>
-Xbootclasspath/a:<directories and zip/jar files separated
by :><br>
append to end of bootstrap class path<br>
-Xbootclasspath/p:<directories and zip/jar files separated
by :><br>
prepend in front of bootstrap class path<br>
-Xdiag show additional diagnostic messages<br>
-Xnoclassgc disable class garbage collection<br>
<b> -Xincgc enable incremental garbage collection</b><br>
-Xloggc:<file> log GC status to a file with time
stamps<br>
-Xbatch disable background compilation<br>
-Xms<size> set initial Java heap size<br>
-Xmx<size> set maximum Java heap size<br>
-Xss<size> set java thread stack size<br>
-Xprof output cpu profiling data<br>
-Xfuture enable strictest checks, anticipating future
default<br>
-Xrs reduce use of OS signals by Java/VM (see
documentation)<br>
-Xcheck:jni perform additional checks for JNI functions<br>
-Xshare:off do not attempt to use shared class data<br>
-Xshare:auto use shared class data if possible (default)<br>
-Xshare:on require using shared class data, otherwise
fail.<br>
-XshowSettings show all settings and continue<br>
-XshowSettings:all<br>
show all settings and continue<br>
-XshowSettings:vm show all vm related settings and continue<br>
-XshowSettings:properties<br>
show all property settings and continue<br>
-XshowSettings:locale<br>
show all locale related settings and
continue<br>
<br>
The -X options are non-standard and subject to change without
notice. However -Xincgc warn user about deprecation.<br>
Shouldn't we remove -Xincgc fron -X options or add warning there?
Could we find replacement for -Xincgc instead iCMS in JDK 8?<br>
<br>
Leonid<br>
<br>
On 02/13/2013 12:11 PM, Bengt Rutisson wrote:<br>
</div>
<blockquote cite="mid:511B4AA9.2040809@oracle.com" type="cite">
<br>
Hi John,
<br>
<br>
Thanks for looking at this!
<br>
<br>
On 2/13/13 3:40 AM, Jon Masamitsu wrote:
<br>
<blockquote type="cite">Bengt,
<br>
<br>
Changes look good.
<br>
<br>
You include tests for UseG1GC and UseParallelGC (should
<br>
not print warnings) but not UseConcMarkSweepGC nor
<br>
UseSerialGC. Why the former but not the latter?
<br>
</blockquote>
<br>
The TestSerialGC.java is running with UseSerialGC. Am I missing
something or is this the test you meant?
<br>
<br>
For CMS I think you have a point. I was kind of thinking that the
test TestParNewCMS was anyway testing that GC combination, but we
should really have a separate CMS test as well. I just added
TestCMS.java to test UseConcMarkSweepGC explicitly. Here is an
updated webrev:
<br>
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~brutisso/8006398/webrev.01/">http://cr.openjdk.java.net/~brutisso/8006398/webrev.01/</a>
<br>
<br>
<blockquote type="cite">
<br>
Do you want to test
<br>
<br>
-XX:+UseConcMarkSweepGC -XX:-CMSIncrementalMode -version
<br>
<br>
to check that no warning is emitted?
<br>
</blockquote>
<br>
Hm. I don't know. If we start combining all the flags we will get
into an explosion of tests. I think this is a valid combination to
test. But in that case so are things like "-XX:+UseConcMarkSweepGC
-XX:-UseSerialGC". I like the way you think about this testing, I
just think I would like to keep these tests to be smoke testing
the more commonly used combinations.
<br>
<br>
Make sense?
<br>
<br>
Thanks again for looking at this!
<br>
Bengt
<br>
<br>
<blockquote type="cite">
<br>
Jon
<br>
<br>
On 2/12/2013 4:56 AM, Bengt Rutisson wrote:
<br>
<blockquote type="cite">
<br>
Hi again,
<br>
<br>
Christian has pushed his test library change and it has
propagated to the hotspot-gc repository. So, this change is
ready to be pushed. Just need some reviews. Any takers? :)
<br>
<br>
Thanks,
<br>
Bengt
<br>
<br>
On 1/16/13 1:58 PM, Bengt Rutisson wrote:
<br>
<blockquote type="cite">
<br>
Hi all,
<br>
<br>
Could I have a couple of reviews for this change?
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~brutisso/8006398/webrev.00/">http://cr.openjdk.java.net/~brutisso/8006398/webrev.00/</a>
<br>
<br>
Recently we deprecated some GC combinations. Those should
now print a warning at startup. Other GC combinations should
not print any warnings.
<br>
<br>
With the new process handling support that Christian
Törnqvist is adding to the JTREG tests for hotspot it is
very easy to write test that start a VM and checks the
output.
<br>
<br>
This changes makes use of Christian's testlibrary to verify
that warnings are printed as expected.
<br>
<br>
I'm also adding the "gc" keyword to JTREG to make it
possible to filter out GC tests. We should probably use this
for all test in the the /gc folder, but I think that should
be done as a separate change.
<br>
<br>
The webrev above is based on Christian's webrev to add the
testlibrary:
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~brutisso/8006413/webrev.00/">http://cr.openjdk.java.net/~brutisso/8006413/webrev.00/</a>
<br>
<br>
Thanks,
<br>
Bengt
<br>
<br>
</blockquote>
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Leonid Mesnik
JVM SQE</pre>
</body>
</html>