<div dir="ltr">Dear Jon,<div><br></div><div>Oops, I don't know what I was thinking when I found that ticket. I'll try to put it back in its old state. I filed a new ticket: JDK-8150013.</div><div><br></div><div>I folded the pruning of the scavengable nmethod list into CodeCache::scavenge_root_nmethods_do, thus removing the need for a separate pruning step after the young collection.</div><div><br></div><div>You can find the new webrev here: <a href="http://cr.openjdk.java.net/~cvarming/1/webrev/">http://cr.openjdk.java.net/~cvarming/1/webrev/</a></div><div><br></div><div>Carsten</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 16, 2016 at 6:53 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">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <font face="Times New Roman, Times, serif">Carsten,<br>
      <br>
      This looks like a good change, but 8080232 describes a different
      problem.<br>
      8080232 proposed early promotion for objects in nmethods.  This
      change<br>
      removes nmethods from a list when it is determined that they don't
      refer to<br>
      scavengable objects.  Can you create a new CR for this?<span class="HOEnZb"><font color="#888888"><br>
      <br>
      Jon<br>
      <br>
    </font></span></font><div><div class="h5"><br>
    <div>On 02/15/2016 12:53 PM, Carsten Varming
      wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">Dear GC members,
        <div><br>
        </div>
        <div>I would like to contribute a fix for <a href="http://JDK-8080232" target="_blank">JDK-8080232</a>.
          The bug is marked as resolved because it was withdrawn. The
          fix is simple, so I would like to reopen the ticket. At
          Twitter we have seen the time to fix relocations after a
          ParNew cycle drop by three to four orders of magnitude with
          this patch.</div>
        <div><br>
        </div>
        <div>I have put a webrev here: <a href="http://cr.openjdk.java.net/%7Ecvarming/prune_scavengable_nmethods/0/webrev/" target="_blank">http://cr.openjdk.java.net/~cvarming/prune_scavengable_nmethods/0/webrev/</a></div>
        <div><br>
        </div>
        <div>I will need a sponsor.</div>
        <div><br>
        </div>
        <div>Carsten</div>
      </div>
    </blockquote>
    <br>
  </div></div></div>

</blockquote></div><br></div>