RFR: 8312181: CDS dynamic dump crashes when verifying unlinked class from static archive [v5]

[Ioi Lam]

On Tue, 1 Aug 2023 19:03:07 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> During CDS dynamic dump, VM crashes due to `assert(!k->is_shared()) failed` while linking an old class (class file version <= 49) loaded from the static archive. The fix mostly adjusts some assert statements based on static vs dynamic dumping. In `MetaspaceShared::try_link_class`, if linking of a shared class during dynamic dump has failed, the `shared_loading_failed` flag will be set to avoid linking the same class again.
>> 
>> Passed tiers 1 - 4 testing.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fix typo in OldClassVerifierTrouble.java

LGTM

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

Marked as reviewed by iklam (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15077#pullrequestreview-1557650153