Integrated: 8361706: Parallel weak klass link cleaning does not clean out previous klasses

[Thomas Schatzl]

On Fri, 11 Jul 2025 10:38:51 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this fix to parallel weak klass link cleaning.
> 
> `KlassCleaningTask::work()` misses cleaning out previous versions of klasses (created by JVMTI class redefinition) as the regular single-threaded variant would do when calling `Klass::clean_weak_klass_links()` with `clean_live_klasses = true`.
> 
> The fix moves the cleaning of weak klass links for previous versions of the klass into `InstanceKlass::clean_weak_instanceklass_links()`.
> 
> Testing: gha, tier1-5, many runs of `vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine/TestDescription.java`(with JDK-8361952 also out for review)
> 
> Thanks,
>   Thomas

This pull request has now been integrated.

Changeset: 99c299f0
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/99c299f0985c8be63b9b60e589db520d83fd8033
Stats:     24 lines in 4 files changed: 12 ins; 10 del; 2 mod

8361706: Parallel weak klass link cleaning does not clean out previous klasses

Reviewed-by: eosterlund, coleenp

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

PR: https://git.openjdk.org/jdk/pull/26263