RFR(XS): 8098815: Assertion failure in CDS shared string archive support on Windows

[Tom Benson]

Thanks, Ioi.
Tom

On 6/17/2015 12:28 PM, Ioi Lam wrote:
> Hi Tom, the change looks good to me (not a "R"eviewer).
>
> Thanks
> - Ioi
>
> On 6/17/15 7:52 AM, Tom Benson wrote:
>> Hi,
>> Please review this change for JDK-8098815.  The CDS code which maps 
>> the shared strings from an archive file must check for a zero length 
>> before calling the G1 fill_archive_regions routine. It already checks 
>> this before the earlier calls to check_archive_address and 
>> alloc_archive_regions, for which fill_archive_regions is the final 
>> step. The problem occurred only on Windows because the string table 
>> is not included in the archive there.
>>
>> JBS:   https://bugs.openjdk.java.net/browse/JDK-8098815
>> Webrev: http://cr.openjdk.java.net/~tbenson/8098815/webrev.00/
>> Tested: JPRT
>>
>> Thanks,
>> Tom
>>
>