On Mon, 27 Nov 2023 02:03:30 GMT, David Holmes <[email protected]> wrote:
>> Stefan Karlsson has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Split test and use othervm
>
> test/hotspot/jtreg/runtime/Monitor/MonitorWithDeadObjectTest.java line 86:
>
>> 84: }
>> 85:
>> 86: private static void testDumpThreadsAfterDetachBeforeJoin() {
>
> The `AfterDetach` in the name is not accurate. If you don't join the new
> native thread then you are racing with its execution and you don't know when
> it will detach.
Thanks. I'll remove the "before join" test case.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16783#discussion_r1405887536