<html>

  <head>

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

  </head>

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

    Hi,

    <br>

    I need reviewers and a commit sponsor for changes for bug 6764713,

    which will increase the size of the age field in an object header

    from 4 bits to 5. This will allow a maximum MaxTenuringThreshold of

    31, though the default will remain at the current value of 15.

    <br>

    <br>

    This includes the same change to the 32-bit version, which would

    close bug 6719225 as well.Â  In that case, the hash field in the

    header is affected, losing one bit (25 bits -> 24), so I have

    asked for review from hotspot-runtime-dev as well as gc-dev.

    <br>

    <br>

    Webrev: <a class="moz-txt-link-freetext"

      href="http://cr.openjdk.java.net/%7Ejprovino/6764713/webrev.00">http://cr.openjdk.java.net/~jprovino/6764713/webrev.00</a>

    <br>

    JBS bug: <a class="moz-txt-link-freetext"

      href="https://bugs.openjdk.java.net/browse/JDK-6764713">https://bugs.openjdk.java.net/browse/JDK-6764713</a>

    <br>

    Testing:Â  JPRT and reference server performance tests

    <br>

    <br>

    Notes:

    <br>

    Contrary to what earlier notes in the JBS entry said, this does not

    require stronger alignment for the JavaThread structure for when

    biased locking stores that pointer in the header.Â Â  The JavaThread*

    was already being aligned 1 power of 2 more strongly than it needed

    to be, so there was an unused bit that could be stolen.

    <br>

    <br>

    In the 32-bit version, it does require taking one bit from the hash

    field, which goes from 25 to 24 bits.Â  This is something I'd

    especially like feedback on.Â  Running reference server performance

    tests, I saw no impact from this change.Â  We <b

      class="moz-txt-star"><span class="moz-txt-tag">*</span>could<span

        class="moz-txt-tag">*</span></b> make this change 64-bit-only,

    and leave the age field at 4 bits for the 32-bit version.Â  If we did

    so, we could also decrease the alignment required for the

    JavaThread* to 512 from the current 1024.

    <br>

    <br>

    The comment changes imply that the bits available for the

    JavaThread* have been reduced by 1, and that the alignment is now

    stronger, but neither is true.Â  The comments have been corrected to

    match the alignment that was already enforced.

    <br>

    <br>

    Three tests needed to be corrected to match the new limits.Â  These

    check the maximum valid values, what value represents NeverTenure,

    and so on.

    <br>

    <br>

    Thanks,

    <br>

    Tom<br>

    <br>

  </body>

</html>