<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><pre><b><font color="blue">!</font><font color="black">     return "Print all compiled methods in code cache</font><u><font color="blue"> that is alive</font></u><font color="black">";</font></b>
</pre><div>“that are alive”.</div><div><div><br></div><div>On Sep 16, 2014, at 1:16 AM, Nils Eliasson <<a href="mailto:nils.eliasson@oracle.com">nils.eliasson@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hi,<br><br>I would like review of this change that includes three fixes:<br><br>1) Let Dcmd Compiler.codelist only print alive-nmethods. We ran into crashes when listing zombies and unloaded too. Alive nmethods includes not-entrants so it still gives a pretty good idea about whats in the code cache and what has been used recently.<br><br>2) Take CompileQueue lock when printing queue. It is not enough to be at a safepoint - the compiler threads may still mutate the list causing crashes.<br><br>3) Relax the parsing of long hex-numbers in the test of codelist. High addresses (sparc) casues NumberFormatExceptions.<br><br>bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8058461">https://bugs.openjdk.java.net/browse/JDK-8058461</a><br>webrev: <a href="http://cr.openjdk.java.net/~neliasso/8058461/webrev.04/">http://cr.openjdk.java.net/~neliasso/8058461/webrev.04/</a><br><br>Thanks!<br>//Nils<br></blockquote></div><br></body></html>