
<html>

  <head>

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

  </head>

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

    <br>

    <div class="moz-cite-prefix">On 11/4/2013 1:02 AM, Bengt Rutisson

      wrote:<br>

    </div>

    <blockquote cite="mid:527762B7.4060605@oracle.com" type="cite">

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

      <br>

      Hi Jon,<br>

      <br>

      <div class="moz-cite-prefix">On 2013-11-02 01:02, Jon Masamitsu

        wrote:<br>

      </div>

      <blockquote cite="mid:52744109.5030302@oracle.com" type="cite">

        <meta http-equiv="content-type" content="text/html;

          charset=UTF-8">

        <big>8024483 - assertion failure: (!mirror_alive ||

          loader_alive) failed<br>

          <br>

          The default value of _should_unload_classes in CMSCollector is

          false.<br>

          The default value of CMSClassUnloadingEnabled is true (changed

          in jdk8)<br>

          and is used to set the root scanning options.Â  This

          inconsistency leads to<br>

          dead class loaders without unloading classes.<br>

          <br>

          The fix for 8024954 fixes (or works around) this inconsistency

          by resetting <br>

          the root scanning options based on _should_unload_classes.<br>

          <br>

          Additionally there is a missing call to

          update_should_unload_classes() in<br>

          the CMS foreground collector.Â  Adding the call allows the test

          for<br>

        </big><big>CMSClassUnloadingEnabledÂ  to pass but may not be a

          complete fix.<br>

          <br>

          <a moz-do-not-send="true" class="moz-txt-link-freetext"

            href="http://cr.openjdk.java.net/%7Ejmasa/8024483/webrev.00/">http://cr.openjdk.java.net/~jmasa/8024483/webrev.00/</a><br>

        </big></blockquote>

      <br>

      Looks good.<br>

      <br>

      <blockquote cite="mid:52744109.5030302@oracle.com" type="cite"><big>

          <br>

        </big><big>Â <br>

          8024954: CMS: CMSClassUnloadingMaxInterval is not implemented

          correctly<br>

          <br>

          For 8024954 the classes used as roots was being set at

          initialization based on<br>

          the value of <span class="removed">CMSClassUnloadingEnabled</span>.Â 


          For a value of CMSClassUnloadingMaxInterval<br>

          greater than 1, the roots need to be set dynamically <br>

          (done in

          CMSCollector::setup_cms_unloading_and_verification_state with

          this fix).<br>

          <br>

          <a moz-do-not-send="true" class="moz-txt-link-freetext"

            href="http://cr.openjdk.java.net/%7Ejmasa/8024954/webrev.01/">http://cr.openjdk.java.net/~jmasa/8024954/webrev.01/</a></big>

      </blockquote>

      <br>

      <br>

      Looks good. One minor question. In CMSCollector::CMSCollector()

      you left this line:<br>

      <br>

      Â 792Â Â  add_root_scanning_option(SharedHeap::SO_SystemClasses);<br>

      <br>

      But in CMSCollector::setup_cms_unloading_and_verification_state()

      we now always set up the state correctly to be either

      SO_AllClasses or SO_SystemClasses. So, do we need that

      initilization in CMSCollector::CMSCollector()?<br>

    </blockquote>

    <br>

    I tested both with and without it and it is not needed as far as I

    can tell.<br>

    I left it in because I know that SO_None is always wrong (as set by

    the<br>

    constructor).Â  I didn't want to try and figure out what the best

    default<br>

    was based on the other flags because that was the source of this

    bug.<br>

    If you think leaving it initialized to SO_None makes the code more

    readable,<br>

    I can remove it. <br>

    <br>

    Jon<br>

    <br>

    <blockquote cite="mid:527762B7.4060605@oracle.com" type="cite"> <br>

      Thanks,<br>

      Bengt<br>

    </blockquote>

    <br>

  </body>

</html>