<div dir="ltr"><div dir="ltr">Thanks, Aleksey. The new webrev looks good.</div><div dir="ltr"><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-Man</div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 6, 2019 at 4:40 AM Aleksey Shipilev <<a href="mailto:shade@redhat.com">shade@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Man,<br>
<br>
Thanks for taking a look. New webrev:<br>
  <a href="http://cr.openjdk.java.net/~shade/8215221/webrev.03/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~shade/8215221/webrev.03/</a><br>
<br>
Still passes hotspot tier1, eyeballing Dacapo logs, jdk-submit is running.<br>
<br>
<br>
On 2/22/19 4:43 AM, Man Cao wrote:<br>
> I found the following could fix the problem (yes, it's ugly, but I couldn't find cleaner way that<br>
> works in all cases):<br>
> <br>
> if (!do_young_collection) {<br>
>   gc_prologue(complete);<br>
>   increment_total_collections(complete);<br>
> }<br>
> if (!complete) {<br>
>   increment_total_full_collections();<br>
> }<br>
<br>
I see, dang. Rewritten like this:<br>
<br>
    if (!do_young_collection) {<br>
      gc_prologue(complete);<br>
      increment_total_collections(complete);<br>
    }<br>
<br>
    // Accounting quirk: total full collections would be incremented when "complete"<br>
    // is set, by calling increment_total_collections above. However, we also need to<br>
    // account Full collections that had "complete" unset.<br>
    if (!complete) {<br>
      increment_total_full_collections();<br>
    }<br>
<br>
> I also found that CollectedHeap._total_collections seems wrong with or without this change, for <br>
> SerialGC and CMS. It is smaller than the actual number of collections according to the GC ID.<br>
> Reproduction recipe is the same as above.<br>
Yeah, that minor thing needs to be fixed too, in a separate change.<br>
<br>
-Aleksey<br>
<br>
<br>
</blockquote></div></div>