<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<font face="Times New Roman, Times, serif">Thomas,<br>
<br>
Thanks for the changes.<br>
<br>
On reading it again I think I would change "capacity after which "
(second<br>
line) to "capacity above which". <br>
<br>
Regarding the meaning of "at the next GC", since it is
complicated to give<br>
a exact definition (listing all the exceptions), maybe it would be
better to<br>
drop "at the next GC". I tried to define it in terms of what a
user would see but it<br>
was a squishy definition (and not correct considering the
exceptions).<br>
<br>
Jon<br>
</font><br>
<div class="moz-cite-prefix">On 02/08/2016 02:53 AM, Thomas Schatzl
wrote:<br>
</div>
<blockquote cite="mid:1454928839.2284.4.camel@oracle.com"
type="cite">
<pre wrap="">Hi Jon,
I mixed all suggestions so far into the one provided at
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tschatzl/8143220/webrev.2/">http://cr.openjdk.java.net/~tschatzl/8143220/webrev.2/</a>
Some more comments below.
On Fri, 2016-02-05 at 13:26 -0800, Jon Masamitsu wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Thomas,
I would leave out the part about InitiatingHeapOccupancyPercent being
used as the initial value for IHOP because not enough data has been
gathered to make a better estimate. While true today it may change
and I don't think it is needed as part of the description.
</pre>
</blockquote>
<pre wrap="">
Has already been removed earlier.
</pre>
<blockquote type="cite">
<pre wrap=""> Explaining what happens when it is 100 would be good.
</pre>
</blockquote>
<pre wrap="">
Done. I just wrote that this disables concurrent marking because that
is probably the important effect that is important for a user.
</pre>
<blockquote type="cite">
<pre wrap="">I added the explanation about fragmentation (is it correct) because I
thought that it would come up sooner or later but you can drop it, if
you think it is too much. So I would suggest.
</pre>
</blockquote>
<pre wrap="">
Done.
Thanks,
Thomas
</pre>
</blockquote>
<br>
</body>
</html>