On Sat, 3 Aug 2024 11:44:02 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> The test has been fixed to:
>>  - add a guard against JVMTI_ERROR_WRONG_PHASE
>>  - replace `exit(err)` with `abort()` in the `check_jvmti_error()`
>>  
>> The JVMTI `VMDeath` event is enabled and a `RawMonitor` is introduced to 
>> serialize `ClassPrepare` and `VMDeath` events, and so, to prevent 
>> JVMTI_ERROR_WRONG_PHASE in the `ClassPrepare` events.
>> 
>> Testing:
>>  - run the test AllowedFunctions.java locally
>>  - TBD: submit a mach5 run for fixed test
>
> Serguei Spitsyn has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   minor simplified tweak

Looks good.

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

Marked as reviewed by cjplummer (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20397#pullrequestreview-2219527246

Reply via email to