<div dir="ltr">Hi Jon --<div><br></div><div>Sorry for the looong delay in getting to this and then of turning around the webrev once I started looking at the code.</div><div>It's been a while since I was in this code, and the code is quite complicated, so it took a while to swap back</div><div>the context and understand (the correctness of) the change.</div><div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 22, 2015 at 12:57 PM, Jon Masamitsu <span dir="ltr"><<a href="mailto:jon.masamitsu@oracle.com" target="_blank">jon.masamitsu@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This bug is causing large young GC times for ParNew when<br>
there are large objects in the heap and the logarithmic strides<br>
are being used (badly).<br>
<br>
<a href="https://bugs.openjdk.java.net/browse/JDK-8081629" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8081629</a><br>
<br>
The change in webrev_min corrects the problem<br>
with the minimum code change (least risk).<br>
<br>
<a href="http://cr.openjdk.java.net/~jmasa/8081629/webrev_min.02/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jmasa/8081629/webrev_min.02/</a></blockquote><div><br></div><div><br></div><div>The above changes look good to me. Thanks also for confirming that these</div><div>changes fix the assertion failure seen with +VerifyBlockOffsetArray. It might be</div><div>worthwhile writing (in the future) a slightly more lightweight VerifyBlockOffsetArray</div><div>so that it becomes practical to use.</div><div><br></div><div>I haven't had the opportunity to look at and understand the second version of the</div><div>webrev below yet. But the one above looks good to me. I'll try and get to the</div><div>version below in the next day or so. Sorry for the ongoing delay.</div><div><br></div><div>thanks!</div><div>-- ramki (openjdk: ysr)</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
A rewriting of split_block() in included in a second webrev<br>
<br>
<a href="http://cr.openjdk.java.net/~jmasa/8081629/webrev.02/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jmasa/8081629/webrev.02/</a><br>
<br>
I've willing to go with the minimum change but could also<br>
be encouraged to push the rewrite of split_block().<br>
<br>
Vote for the one you like with your code review.<br>
<br>
The diff between webrev_min.02 and webrev.02 is<br>
here but I don't find it that useful.<br>
<br>
<a href="http://cr.openjdk.java.net/~jmasa/8081629/webrev_delta.02/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jmasa/8081629/webrev_delta.02/</a><br>
<br>
Thanks.<span class="HOEnZb"><font color="#888888"><br>
<br>
Jon<br>
</font></span></blockquote></div><br></div></div>