RFR: 8276429: CodeHeapState::print_names() fails with "assert(klass->is_loader_alive()) failed: must be alive"

Wed Nov 3 22:00:12 UTC 2021

On Wed, 3 Nov 2021 14:19:29 GMT, Evgeny Astigeevich <duke at openjdk.java.net> wrote:

> This PR fixes `applications/kitchensink/Kitchensink.java` regression introduced by JDK-8275729.
> The requirement for a method holder to be alive is relaxed to the holder not to be NULL.
> If holder's name is not available the format of the string used for the name is the same as for unavailable method's name and signature, instead of the default string: `<unknown>`.
> Testing:
> - `make run-test TEST=tier1_serviceability`: Passed
> - `make run-test TEST=hotspot_tier2_serviceability`: Passed
> - `make run-test TEST=serviceability/dcmd/compiler/CodeHeapAnalyticsMethodNames.java`: Passed

After infrastructure hiccup tests finally passed.
Please integrate and I will sponsor.

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

Marked as reviewed by kvn (Reviewer).

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