On Tue, 30 Jul 2024 23:13:23 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 > prevent JVMTI_ERROR_WRONG_PHASE in the `ClassPrepare` events. > > Testing: > - run the test AllowedFunctions.java locally > - TBD: submit a mach5 run for fixed test Converting the native agent to C++ would simplify the test and allow to use utility functions from the `jvmti_common.hpp`. But I wanted to make the WRONG_PHASE related fix to be clear for reviewers. So, this conversion can be done as a separate step or as a part of a separate effort covered by an enhancement. ------------- PR Comment: https://git.openjdk.org/jdk/pull/20397#issuecomment-2259356963
