> The test does not handle exceptions, e.g. `InterruptedException`, in the 
> `Producer` and `Consumer` threads. This can cause deadlocks. This timeout 
> issue is hard to reproduce. This update is to add catch any `Throwable` and 
> log all information needed to understand the root cause.
> Also, removed unneeded fragment with the call: 
> `System.loadLibrary(agentLib);`.
> 
> Testing:
>  - Executed the test locally: 
> `serviceability/jvmti/vthread/VThreadTest/VThreadTest.java`
>  - TBD: Submit: mach5 tiers 1-3 to be completely safe
> 
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Serguei Spitsyn has updated the pull request incrementally with two additional 
commits since the last revision:

 - review: removed stale constant
 - review: fixed two typos in the logging

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/31581/files
  - new: https://git.openjdk.org/jdk/pull/31581/files/71b8c682..522e56d9

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=31581&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=31581&range=00-01

  Stats: 16 lines in 2 files changed: 12 ins; 2 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/31581.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/31581/head:pull/31581

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

Reply via email to