On Tue, 8 Jul 2025 19:41:26 GMT, Alex Menkov <amen...@openjdk.org> wrote:

>> The fix updates several tests ASAN complains about.
>> Tests for RawMonitors intentionally passes bad pointer to tested function, 
>> they should not be run with ASAN.
>> Test vmTestbase/nsk/jvmti/scenarios/events/EM07/em07t002 verifies 
>> COMPILED_METHOD_LOAD/COMPILED_METHOD_UNLOAD event and uses RawMonitor for 
>> synchronization. It destroys the monitor, but some events may arrive later 
>> (before VM exits). Usually JVMTI tests do not destroy synchronization 
>> monitors, removed destruction here.
>
> Alex Menkov has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   feedback

Looks good.

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

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26176#pullrequestreview-3003213936

Reply via email to