<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.  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, &quot;harold seigel&quot; &lt;<a 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:  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. 
    This fix stores the ObjectAlignmentInBytes in the CDS archive so
    that when the archive is read, hotspot can compare the archive&#39;s
    alignment with the current alignment and issue the following
    diagnostic if the archive&#39;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.  The existing code was
    always setting SharedReadOnlySize to 14M regardless of what was
    requested.  This prevented users from being able to expand the CDS
    archive&#39;s SharedReadOnly section.<br>
    <br>
    Open webrev at <a 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 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>