RFR: 8254309: appcds GCDuringDump.java failed - class must exist [v4]

Calvin Cheung ccheung at openjdk.java.net
Sat Oct 31 00:09:00 UTC 2020


On Fri, 30 Oct 2020 23:22:11 GMT, Yumin Qi <minqi at openjdk.org> wrote:

>> Hi,  Please review
>>   When CDS at dump time initializes archived heap, some classes are loaded. If at this time system runs out of memory the class will not be loaded. This is what we saw in this bug. The fix checks if OOM happened, if so we print out log and exit gracefully not causing a crash. Added a test case for testing purpose when exception/OOM happens during this stage. Also check during preload classes when OOM happens, exit vm with proper message.
>> 
>> Tests: tier1-4
>> 
>> Thanks
>> Yumin
>
> Yumin Qi has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Break long line into 3 shorter lines

Looks good.

-------------

Marked as reviewed by ccheung (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/948


More information about the hotspot-dev mailing list