RFR(S): 8247529: Crash in runtime/cds/appcds/dynamicArchive/LambdaProxyCallerIsHidden.java with Graal
calvin.cheung at oracle.com
calvin.cheung at oracle.com
Mon Aug 17 17:56:58 UTC 2020
Hi Ioi,
Thanks for your review!
I'll make the change before push.
thanks,
Calvin
On 8/17/20 9:43 AM, Ioi Lam wrote:
> Hi Calvin,
>
> This looks good to me. Just a small nit: the NULL assignment is not
> needed here since p is assigned immediately below
>
> 06 DumpTimeSharedClassInfo* p = NULL;
> Thanks
> Ioi
>> On Aug 14, 2020, at 9:10 AM, calvin.cheung at oracle.com wrote:
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8247529
>> webrev: http://cr.openjdk.java.net/~ccheung/jdk16/8247529/webrev.00/
>>
>> This proposed change includes:
>>
>> 1) Avoid inserting InstanceKlass into the DumpTimeSharedClassTable
>> after dynamic CDS dumping has started.
>>
>> This is done by adding a _dump_in_progress bool which will be set at
>> the beginning of dynamic CDS dumping. In
>> DumpTimeSharedClassTable::find_or_allocate_info_for, if the
>> _dump_in_progress is set, it will not create a new entry in the table.
>>
>> 2) Acquire the DumpTimeTable_lock before iterations on
>> _dumptime_table to ensure memory order correctness.
>>
>> Testing: passed tier1 - 4 tests.
>>
>> thanks,
>>
>> Calvin
>>
>>
>>
More information about the hotspot-runtime-dev
mailing list