> 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 one additional 
commit since the last revision:

  review: fixed typos in the RuntimeException messages

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/31581/files
  - new: https://git.openjdk.org/jdk/pull/31581/files/e4ff84e4..b7009228

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

  Stats: 2 lines in 1 file changed: 0 ins; 0 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