RFR (S): 8131734: Add free_archive_regions support to G1 for -Xshared:auto

[Tom Benson]

Hi,

Updated full and incremental webrevs are at:
   http://cr.openjdk.java.net/~tbenson/8131734/webrev.01/
   http://cr.openjdk.java.net/~tbenson/8131734/webrev.01.vs.00/

On 8/5/2015 2:05 PM, Jiangli Zhou wrote:
> Hi Tom,
>
> On Aug 5, 2015, at 10:33 AM, Tom Benson <tom.benson at oracle.com 
> <mailto:tom.benson at oracle.com>> wrote:
>
>> ... so perhaps that's a viable option.  I'll try it to ensure the CDS 
>> use does indeed occur while is_init_completed() is false.
>
> The mapping and initialization of the CDS string data happens during 
> VM initialization and before 'set_init_completed()’ is called. Just 
> verified that in gdb as well.
>
> Thanks,
> Jiangli
>

Yes, I checked as well.  I've added this to each of 
alloc_/fill_/free_archive_regions:

    assert(!is_init_completed(), "Expect to be called at JVM init time");

Tom
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20150805/8b8670d3/attachment.htm>