<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Please review the modified fix for bug 8005076.&nbsp; The modified fix
    can be viewed at <a
      href="http://cr.openjdk.java.net/%7Ehseigel/bug_8005076_2/">http://cr.openjdk.java.net/~hseigel/bug_8005076_2/</a><br>
    <br>
    It contains two changes from the previous fix.<br>
    <br>
    As requested by Vitaly, the diagnostic in filemap.cpp has been
    changed to include the incompatible ObjectAlignmentInBytes values.&nbsp;
    Here's sample new message:<br>
    <blockquote>The shared archive file's ObjectAlignmentInBytes of 8 is
      too small for the current ObjectAlignmentInBytes of 16.<br>
    </blockquote>
    Also, the change to arguments.cpp no longer sets a minimum value for
    -XX:+ObjectAlignmentInBytes.&nbsp; (This was requested by Coleen.)<br>
    <br>
    Thanks, Harold<br>
    <br>
    <br>
    On 12/17/2012 7:10 PM, Vitaly Davidovich wrote:
    <blockquote
cite="mid:CAHjP37F5bvJHauKVOTt3qcr-Fw0cPoN_YL6P6-RsX_r3Gjw7EA@mail.gmail.com"
      type="cite">
      <p dir="ltr">Hi Harold,</p>
      <p dir="ltr">It might be useful to report the recorded alignment
        in the CDS archive in the diagnostic message that you print
        out.&nbsp; What do you think?</p>
      <p dir="ltr">Thanks</p>
      <p dir="ltr">Sent from my phone</p>
      <div class="gmail_quote">On Dec 17, 2012 10:24 AM, "harold seigel"
        &lt;<a moz-do-not-send="true"
          href="mailto:harold.seigel@oracle.com">harold.seigel@oracle.com</a>&gt;
        wrote:<br type="attribution">
        <blockquote class="gmail_quote" style="margin:0 0 0
          .8ex;border-left:1px #ccc solid;padding-left:1ex">
          <div bgcolor="#FFFFFF" text="#000000"> Please review the
            following change to fix bug 8005075.<br>
            <br>
            Summary:&nbsp; This change prevents a crash when a CDS archive is
            created with a value for -XX:+ObjectAlignmentInBytes that is
            smaller than the ObjectAlignmentInBytes value used when
            running with -Xshare:on.&nbsp; This fix stores the
            ObjectAlignmentInBytes in the CDS archive so that when the
            archive is read, hotspot can compare the archive's alignment
            with the current alignment and issue the following
            diagnostic if the archive's alignment is too small:<br>
            <blockquote>An error has occurred while processing the
              shared archive file.<br>
              The shared archive file was created with a smaller Object
              Alignment value.<br>
            </blockquote>
            This webrev also cleans up some text in globals.hpp and
            fixes a small problem with -XX:SharedReadOnlySize.&nbsp; The
            existing code was always setting SharedReadOnlySize to 14M
            regardless of what was requested.&nbsp; This prevented users from
            being able to expand the CDS archive's SharedReadOnly
            section.<br>
            <br>
            Open webrev at <a moz-do-not-send="true"
              href="http://cr.openjdk.java.net/%7Ehseigel/bug_8005076/"
              target="_blank">http://cr.openjdk.java.net/~hseigel/bug_8005076/</a><br>
            <br>
            Bug link at <a moz-do-not-send="true"
              href="http://bugs.sun.com/view_bug.do?bug_id=8005076"
              target="_blank">http://bugs.sun.com/view_bug.do?bug_id=8005076</a><br>
            <br>
            The changes were tested with JCK, JPRT, JTREG, and UTE
            tests, and with hand-run tests using different
            ObjectAlignmentInBytes values.<br>
            <br>
            Thanks, Harold<br>
          </div>
        </blockquote>
      </div>
    </blockquote>
  </body>
</html>