[hdk8u40] Request for review (s) - 8031323: Optionally align objects copied to survivor spaces

Jon Masamitsu jon.masamitsu at oracle.com
Tue Aug 5 21:07:49 UTC 2014


This is a backport from jdk9.  The patch applied cleanly.

8031323: Optionally align objects copied to survivor spaces

Add the option to specify an alignment for allocations into the
survivor spaces.  Survivor space allocations will overflow if the
survivor spaces have not been increased.   The expected use
is to align survivor space allocations to cache line sizes.

This is the webrev for the patch from jdk9.

http://cr.openjdk.java.net/~jmasa/8031323/webrev.00/

I made a change to the error message in arguments.cpp
here.  This change was based on a review comment I received
after the push to jdk9

http://cr.openjdk.java.net/~jmasa/8031323/webrev_delta.00/


      if (SurvivorAlignmentInBytes < ObjectAlignmentInBytes) {
        jio_fprintf(defaultStream::error_stream(),
-          "error: SurvivorAlignmentInBytes=%d must be greater than ObjectAlignmentInBytes=%d \n",
+          "error: SurvivorAlignmentInBytes=%d must be greater than or equal to ObjectAlignmentInBytes=%d \n",
            (int)SurvivorAlignmentInBytes, (int)ObjectAlignmentInBytes);
        return false;
      }

Thanks.

Jon



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/attachments/20140805/633a935f/attachment.html>


More information about the hotspot-gc-dev mailing list