RFR: 8308614: Enabling JVMTI ClassLoad event slows down vthread creation by factor 10
Serguei Spitsyn
sspitsyn at openjdk.org
Thu Nov 16 11:20:42 UTC 2023
This is a fix of a performance/scalability related issue. The `JvmtiThreadState` objects for virtual thread filtered events enabled globally are created eagerly because it is needed when the `interp_only_mode` is enabled. Otherwise, some events which are generated in `interp_only_mode` from the debugging version of interpreter chunks can be missed.
However, it has to be okay to avoid eager creation of these object if no `interp_only_mode` has ever been requested.
It seems to be an extremely important optimization to create JvmtiThreadState objects lazily in such cases.
It is done by introducing the flag `JvmtiThreadState::_seen_interp_only_mode` which indicates when the `JvmtiThreadState` objects have to be created eagerly.
Additionally, the fix includes the following related changes:
- Use condition double checking idiom for `MutexLocker mu(JvmtiThreadState_lock)` in the function `JvmtiVTMSTransitionDisabler::VTMS_mount_end` which is on a performance-critical path and looks like this:
JvmtiThreadState* state = thread->jvmti_thread_state();
if (state != nullptr && state->is_pending_interp_only_mode()) {
MutexLocker mu(JvmtiThreadState_lock);
state = thread->jvmti_thread_state();
if (state != nullptr && state->is_pending_interp_only_mode()) {
JvmtiEventController::enter_interp_only_mode();
}
}
- Add extra check of `JvmtiExport::can_support_virtual_threads()` when virtual thread mount and unmount are posted.
- Minor: Added a `ThreadsListHandle` to the `JvmtiEventControllerPrivate::enter_interp_only_mode`. It is needed because of the dynamic creation of compensating carrier threads which is racy for JVMTI `SetEventNotificationMode` implementation.
Performance mesurements:
- Without this fix the test provided by the bug submitter gives execution numbers:
- no ClassLoad events enabled: 3251 ms
- ClassLoad events are enabled: 40534 ms
- With the fix:
- no ClassLoad events enabled: 3270 ms
- ClassLoad events are enabled: 3385 ms
Testing:
- Ran mach5 tiers 1-6, no regressions are noticed
-------------
Commit messages:
- 8308614: Enabling JVMTI ClassLoad event slows down vthread creation by factor 10
Changes: https://git.openjdk.org/jdk/pull/16686/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=16686&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8308614
Stats: 37 lines in 3 files changed: 24 ins; 4 del; 9 mod
Patch: https://git.openjdk.org/jdk/pull/16686.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16686/head:pull/16686
PR: https://git.openjdk.org/jdk/pull/16686
More information about the hotspot-dev
mailing list