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 sspitsyn (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/31128#pullrequestreview-4349090087
