<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi Albert,<br>
    <br>
    On 2014-09-05 15:27, Albert wrote:<br>
    <blockquote cite="mid:5409BA4F.9080404@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      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>
    </blockquote>
    <br>
    Yes - certainly. Fixed.<br>
    <br>
    <blockquote cite="mid:5409BA4F.9080404@oracle.com" type="cite"> <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>
    </blockquote>
    <br>
    Good. Yes it make more sense to print all nmethods.<br>
    <br>
    I also found that I had forgotten to add a file
    (MethodIdentifierParser.java). <br>
    <br>
    New webrev here:<br>
    <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~neliasso/8054889/webrev.04">http://cr.openjdk.java.net/~neliasso/8054889/webrev.04</a><br>
    <br>
    Thank you!<br>
    //Nils<br>
    <br>
    <blockquote cite="mid:5409BA4F.9080404@oracle.com" type="cite"> <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 moz-do-not-send="true" 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 moz-do-not-send="true" class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Eneliasso/8054889/webrev.02/">http://cr.openjdk.java.net/~neliasso/8054889/webrev.02/</a>
        <br>
        <br>
        Thanks, <br>
        Nils Eliasson <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>