> 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 unneeded #if
 - review: replaced logging message with throwing a RuntimeException

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

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

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

  Stats: 6 lines in 2 files changed: 2 ins; 3 del; 1 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