On Tue, 6 Jan 2026 22:47:08 GMT, Chad Rakoczy <[email protected]> wrote:

>> [JDK-8369150](https://bugs.openjdk.org/browse/JDK-8369150)
>> 
>> The test checks for JVMTI `COMPILED_METHOD_LOAD` and 
>> `COMPILED_METHOD_UNLOAD` events to be published for a relocated nmethod. It 
>> would originally intermittently fail if the JVM exited before it had time to 
>> publish the events so now it loops and forces GCs to encourage event 
>> publishing. The test fails if the events are received in the incorrect order 
>> (such as an unload before a load) or if the correct events are not received 
>> and the test times out.
>
> Chad Rakoczy has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Remove JNIenv use and update shouldExit

@chadrako  Thanks for all improvements!
Friendly reminder - you need to wait for some reasonable until review from all 
people that has concerns. Just to ensure that you all are in agreement.
Also, https://openjdk.org/guide/#hotspot-development says specific Hotspot 
rules. At least to reviews are needed for any non-trivial change.

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

Marked as reviewed by lmesnik (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28683#pullrequestreview-3637340896

Reply via email to