<html>

  <head>

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

  </head>

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

    <font face="Times New Roman, Times, serif">Changes look good.<br>

      <br>

      Jon<br>

    </font><br>

    <div class="moz-cite-prefix">On 12/21/2015 07:01 AM, Thomas Schatzl

      wrote:<br>

    </div>

    <blockquote cite="mid:1450710105.3214.46.camel@oracle.com"

      type="cite">

      <pre wrap="">Hi all,

  can I have reviews for this change that moves setup and

parallelization detail code for the remset scrubbing code out of

ConcurrentMark into G1CollectedHeap?

This way, ConcurrentMark does not need a direct reference to G1RemSet

and HeapRegionManager any more, and anyway, I think setting up the

gangtask by determining the number of threads to use is better suited to

be placed into G1CollectedHeap.

CR:

<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8145774">https://bugs.openjdk.java.net/browse/JDK-8145774</a>

Webrev:

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

Testing:

jprt

Thanks,

  Thomas

</pre>

    </blockquote>

    <br>

  </body>

</html>