[jdk17] RFR: 8268470: CDS dynamic dump asserts with JFR RecordingStream
Calvin Cheung
ccheung at openjdk.java.net
Fri Jun 11 16:10:58 UTC 2021
On Thu, 10 Jun 2021 20:14:10 GMT, Yumin Qi <minqi at openjdk.org> wrote:
>> During JFR startup, it redefines some of the builtin classes like `java/lang/Throwable`. Later, when a subclass of Throwable is loaded, it will inherit the redefined version of Throwable which will not be inside the buffer space or mapped static archive.
>>
>> To fix this bug, in `SystemDictionaryShared::check_for_exclusion_impl()`, instead of only checking if the class has been redefined, it also checks if any of its super types has been redefined. If so, exclude the class from the archive.
>>
>> - [x] mach5 tier1, tier2 (including the new test) (in progress)
>
> LGTM.
@yminqi, @iklam Thanks for the review!
-------------
PR: https://git.openjdk.java.net/jdk17/pull/9
More information about the hotspot-runtime-dev
mailing list