RFR: 8356587: Missing object ID X in pool jdk.types.Method [v5]
Markus Grönlund
mgronlun at openjdk.org
Mon Jul 28 14:53:34 UTC 2025
> Greetings,
>
> The following change set addresses the data loss resulting in the assertion "Missing object ID X in pool jdk.types.Method".
>
> It involves three components:
>
> 1. Address a regression introduced by [JDK-835221](https://bugs.openjdk.org/browse/JDK-8352251). By locating JFR_ONLY(Jfr::check_and_process_sample_request(thread);) before the global_poll() in SafepointMechanism::process(), a stacktrace can be captured, and artifacts tagged, during a safepoint. This breaks an invariant as artifacts, i.e. methods, can be tagged in the wrong epoch (also a stacktrace can be stored in the wrong epoch). Must be moved to post global_poll().
>
> 2. Retransform/Redefine classes include a non-safe copy of Method trace flags, leading to stale bits being set onto the new Methods. Method trace flags need to be copied, but must be done under a safepoint.
>
> 3. It seems that there has been an increase in the frequency of issuing calls to InstanceKlass::purge_previous_versions(), making scratch klasses and old methods disappear before JFR gets the chance to serialize also tagged old methods. Therefore, we need to ensure that we always tag the latest version of a Method.
>
> There is also a cleanup of gratuitous type conversions, from Klass* to InstanceKlass* in the newly introduced MethodTracing subsystem.
>
> Testing: jdk_jfr, stress testing
>
> Thank you
> Markus
Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
removed unused
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/26458/files
- new: https://git.openjdk.org/jdk/pull/26458/files/e68e9f3c..6504e1b7
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=26458&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=26458&range=03-04
Stats: 2 lines in 1 file changed: 0 ins; 2 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/26458.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26458/head:pull/26458
PR: https://git.openjdk.org/jdk/pull/26458
More information about the hotspot-jfr-dev
mailing list