On Tue, 19 May 2026 17:51:10 GMT, Leonid Mesnik <[email protected]> wrote:

>> Please review following test fix that improve synchronization of compiled 
>> method load/unload events.
>> 
>> The CompiledMethodLoad/CompiledMethodUnload might be already executing while 
>> SetEventCallbacks removed their callbacks.
>> Thus they hit 'newEventCount' after setting it to zero and cause test to 
>> fail.
>> 
>> The simple monitor inside event doesn't help, because race might happen 
>> while callback is calling in VM or before it obtain lock.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Leonid Mesnik has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   updated after commnets

Looks good

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

Marked as reviewed by cjplummer (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/31128#pullrequestreview-4283218938

Reply via email to