<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix"><br>
Hi Leonid,<br>
<br>
Thanks for noticing the java -X print out. I did not know about
this.<br>
<br>
If I understand this correctly java -X basically lists what is in
this file:<br>
<br>
jdk/src/share/classes/sun/launcher/resources/launcher.properties<br>
<br>
This file seems to be internationalized, so there are several
files for different languages.<br>
<br>
At the moment I don't think we need to update all these files. The
-Xincgc flag still works, it is just deprecated. But once we
remove the flag we need to remember to remove it from all these
files. Thanks for pointing this out.<br>
<br>
Bengt<br>
<br>
On 2/13/13 10:04 AM, Leonid Mesnik wrote:<br>
</div>
<blockquote cite="mid:511B5723.6060500@oracle.com" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<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 moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ebrutisso/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 moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ebrutisso/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 moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ebrutisso/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>
</blockquote>
<br>
</body>
</html>