RFR: 8306929: Avoid CleanClassLoaderDataMetaspaces safepoints when previous versions are shared
Stefan Johansson
sjohanss at openjdk.org
Fri Apr 28 13:10:57 UTC 2023
Hi all,
Please review this change to avoid CleanClassLoaderDataMetaspaces safepoint when there is nothing that can be cleaned up.
**Summary**
When transforming/redefining classes a previous version list is linked together in the InstanceKlass. The original class is added to this list if it is still used or shared. The difference between shared and used is not currently noted. This leads to a problem when doing concurrent class unloading, because during that we postpone some potential work to a safepoint (since we are not in one). This is the CleanClassLoaderDataMetaspaces and it is triggered by the ServiceThread if there is work to be done, for example if InstanceKlass::_has_previous_versions is true.
Since we currently does not differentiate between shared and "in use" we always set _has_previous_versions if anything is on this list. This together with the fact that shared previous versions should never be cleaned out leads to this safepoint being triggered after every concurrent class unloading even though there is nothing that can be cleaned out.
This can be avoided by making sure the _previous_versions list is only cleaned when there are non-shared classes on it. This change renames `_has_previous_versions` to `_clean_previous_versions` and only updates it if we have non-shared classes on the list.
**Testing**
* A lot of manual testing verifying that we do get the safepoint when we should.
* Added new test to verify expected behavior by parsing the logs. The test uses JFR to trigger redefinition of some shared classes (when -Xshare:on).
* Mach5 run of new test and tier 1-3
-------------
Commit messages:
- Add test wih JFR
- 8306929: Avoid CleanClassLoaderDataMetaspaces safepoints when previous versions are shared
Changes: https://git.openjdk.org/jdk/pull/13716/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=13716&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8306929
Stats: 139 lines in 6 files changed: 116 ins; 3 del; 20 mod
Patch: https://git.openjdk.org/jdk/pull/13716.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/13716/head:pull/13716
PR: https://git.openjdk.org/jdk/pull/13716
More information about the serviceability-dev
mailing list