On Mon, 20 Feb 2023 22:30:30 GMT, David Holmes <[email protected]> wrote:

>> Test update to make failures such as JDK-8076494 more informative.
>> 
>> Waiting for a thread to change state: give more time (to distinguish a real 
>> deadlock from some other delay), and log the thread stackframes when the 
>> expected state change is not observed.
>
> test/hotspot/jtreg/vmTestbase/nsk/monitoring/share/thread/ThreadUtils.java 
> line 232:
> 
>> 230: 
>> 231:         // Most uses of waitThreadState usually succeed without retries.
>> 232:         // Sleep time increased to help avoid spurious failures.
> 
> "increased" doesn't mean anything outside this PR as you can't tell the value 
> has changed.

Also any comment on sleep time belongs after line 233.

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

PR: https://git.openjdk.org/jdk/pull/12661

Reply via email to