
<html>

  <head>


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

  </head>

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

    <br>

    Hi everyone,<br>

    <br>

    Can I have a couple of reviews for this change?<br>

    <br>

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

    <br>

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

    <br>

    Unlike the other collectors G1 did not use dynamic sizes for the

    TLABs that the Java threads use for allocation. That means that the

    TLABs when using G1 were all fixed size and very small. This patch

    adds dynamic sizing of TLABs to G1 too. In performance testing this

    has shown a 2-3% improvement across a range of benchmarks.<br>

    <br>

    One difference compared to other collectors is that the maximum TLAB

    size in G1 has to be limited by the region size and not by the eden

    size. Also, for the other collectors the used space for TLAB

    allocation was deducted from the maximum TLAB size because this was

    essentially what was left in eden. That is an assumption that

    happened to work out, but it kind of breaks the abstraction. So, the

    method CollectedHeap::

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

      charset=ISO-8859-1">

    tlab_used() has been added to be more consistent about the TLAB

    abstraction and to make it more straight forward to implement in G1.<br>

    <br>

    Thanks,<br>

    Bengt<br>

  </body>

</html>