On Sat, 10 Oct 2026 07:09:41 GMT, Yunbo Zhang <[email protected]> wrote:
> Please review this small fix. > > `SetExtensionEventCallback()` registers the callback but does not enable the > event. > > **Fix:** > > Update the user-enabled bit when setting or clearing the callback, and add > mount/unmount protection when updating the event state. > > **Testing:** > > - Removed explicit mount/unmount enabling from `libVThreadEventTest.cpp` to > verify that callback registration enables the events. > - In `libMethodExitTest.cpp`, disable these events after registration, since > the test enables them per thread at the second breakpoint. > - `serviceability/jvmti`: 264 passed, 0 failed. > - `vmTestbase/nsk/jvmti`: 586 passed, 0 failed. > > > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). I've filed this JBS issue: https://bugs.openjdk.org/browse/JDK-8394716: SetExtensionEventCallback should not enable the event ------------- PR Comment: https://git.openjdk.org/jdk/pull/33295#issuecomment-6203738755
