
<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    <br>

    <br>

    <div class="moz-cite-prefix">On 11/30/17 6:22 AM, Roman Kennke

      wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:243153d8-1836-9d3e-b242-52e17b2e2134@redhat.com">Hi

      David,

      <br>

      <br>

      I added the tag as you proposed:

      <br>

      <br>

      Differential:

      <br>

      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~rkennke/8191564/webrev.08.diff/">http://cr.openjdk.java.net/~rkennke/8191564/webrev.08.diff/</a>

      <br>

      Full:

      <br>

      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~rkennke/8191564/webrev.08/">http://cr.openjdk.java.net/~rkennke/8191564/webrev.08/</a>

      <br>

      <br>

    </blockquote>

    <br>

    This looks really good.  This version looks much better than an

    early version I looked at (thanks for regenerating webrev with hg

    rename).<br>

    <br>

    A minor comment that GCMemoryManager could take an enum to indicate

    the type of this  GC action (major vs minor).   This can be a future

    cleanup.<br>

    <br>

    thanks<br>

    Mandy<br>

  </body>

</html>