<html>

  <head>

    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">

  </head>

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

    <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~stefank/8013160/webrev.00/">http://cr.openjdk.java.net/~stefank/8013160/webrev.00/</a><br>

    <br>

    There's some remnants from the PermGen that's confusing in the class

    unloading code. This patch removes the unnecessary code, cleans up

    some comments and unifies the code structure over different GCs.<br>

    <br>

    From the Bug report:<br>

    ---<br>

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

      charset=ISO-8859-1">

    The PermGen Removal changed CompiledICHolders from being Java

    objects to C++ objects.

    <br>

    <br>

    CodeCache::do_unloading(...) used to take a keep_alive closure that

    was applied to CompiledICHolders deep down in nmethod::do_unloading.

    Since CompileICHolders don't move anymore, and we have other ways to

    keep them alive, we don't use the keep_alive closures in these

    functions anymore.

    <br>

    <br>

    Because of this, CodeCache::do_unloading will not populate the

    marking stacks, and we can remove the call to drain the stacks after

    the calls to CodeCache::do_unloading.

    <br>

    <br>

    This also has the neat effect that we now can verify that the

    marking has really completed before we start unloading classes,

    which makes the code easier to maintain/understand.<br>

    ---<br>

    <br>

    thanks,<br>

    StefanK<br>

  </body>

</html>