On Fri, 16 Oct 2026 15:22:06 GMT, Serguei Spitsyn <[email protected]> wrote:

> Current implementation of JVMTI SetExtensionEventCallback() registers the 
> callback but does not enable the event. This behavior matches normal events, 
> so there is a symmetry between normal and extension events.
> 
> However, the SetExtensionEventCallback spec states this:
> 
> Sets the callback function for an extension event and enables the event. Or, 
> if the callback is NULL, disables the event. Note that unlike standard 
> events, setting the callback and enabling the event are a single operation.
> 
> This wording about enabling/disabling the events has to be removed from 
> SetExtensionEventCallback spec.
> It seems that extension events are not used by the JVMTI agents. Two 
> extension events are used in number of tests:
>  - `VirtualThreadMount` and `VirtualThreadUnmount`
> 
> Testing: N/A
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Looks good. Does this need a CSR and/or a release note?

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

Marked as reviewed by cjplummer (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/33415#pullrequestreview-5554671616

Reply via email to