<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi Nils,<br>
    <br>
    I looked at your changes. Here is some feedback:<br>
    <br>
    - Would it make sense to add an assert in '
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    CompileQueue::print()' that checks that we are either at a safepoint
    or that the MethodCompileQueue_lock is held?<br>
      (
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    assert_locked_or_safepoint(CodeCache_lock);). This seems to make the
    implementation more robust.<br>
    <br>
    -
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    CodeCache::print_codelist() iterates over all 'alive' nmethods.
    However, the description says:<br>
      'Print all compiled methods in code cache.' I think either the
    description or the implementation should be adapted to keep things
    consistent.<br>
    <br>
    Best,<br>
    Albert <br>
    <br>
    <br>
    Best,<br>
    Albert<br>
    <div class="moz-cite-prefix">On 09/05/2014 01:12 PM, Nils Eliasson
      wrote:<br>
    </div>
    <blockquote cite="mid:54099AB8.7010601@oracle.com" type="cite">Hi,
      <br>
      <br>
      This fix introduces three new diagnostic commands and
      corresponding tests. The commands are Compiler.queue for printing
      the contents of the compiler queue, Compiler.codelist for printing
      all nmethods in the codecache and Compiler.codecache for printing
      an informational code cache summary.
      <br>
      <br>
      Description here:
      <br>
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8054889">https://bugs.openjdk.java.net/browse/JDK-8054889</a>
      <br>
      <br>
      Webrev:
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~neliasso/8054889/webrev.02/">http://cr.openjdk.java.net/~neliasso/8054889/webrev.02/</a>
      <br>
      <br>
      Thanks,
      <br>
      Nils Eliasson
      <br>
    </blockquote>
    <br>
  </body>
</html>