On Wed, 1 Nov 2023 17:54:00 GMT, Mandy Chung <mch...@openjdk.org> wrote:
> `TestLoadLibraryDeadlock.java` test runs `LoadLibraryDeadlock` and wait for 5 > seconds and then grab the output. Then run `jcmd` to dump the thread stacks > in case there is a deadlock. The test ignores and swallows any exception > which makes it hard to diagnose test issues. > > This PR simplifies the test to use `jdk.test.lib.process.ProcessTools` to > launch `LoadLibraryDeadlock` test so that the output and error will be > captured in the same way as other tools are run by this test. Also update > the test to propagate exceptions where appropriate. This hopes to collect > more information to diagnose the issue if this test fails next time. This pull request has now been integrated. Changeset: 5207443b Author: Mandy Chung <mch...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/5207443b360cfe3ee9c53ece55da3464c13f6a9f Stats: 108 lines in 2 files changed: 16 ins; 73 del; 19 mod 8317965: TestLoadLibraryDeadlock.java fails with "Unable to load native library.: expected true, was false" Reviewed-by: rriggs ------------- PR: https://git.openjdk.org/jdk/pull/16459
