On Mon, 27 Nov 2023 02:03:30 GMT, David Holmes <dhol...@openjdk.org> 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

Reply via email to