[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