Integrated: 8305919: java/lang/Thread/virtual/HoldsLock.java#id0 failed, ThreadInfo.getLockInfo() return null

[Alan Bateman]

On Sun, 30 Apr 2023 18:18:09 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> HoldsLock.java#id0 has been failing intermittently recently due to threads left over from previous tests terminating. HoldsLock.java#id1 doesn't fail as it runs in /othervm mode. The test uses ThreadMXBean::getAllThreadIds to get the ID of all threads and calls ThreadMXBean::getThreadInfo on each thread. If a thread from a previous test terminates then getThreadInfo returns null and the test fails.
> 
> The test can be trivially fixed to check for null but looking at it afresh, the test can be simplified to use existing test infrastructure and to just call ThreadMXBean::getThreadInfo on the carrier. The test can also be renamed to make it clearer that it is testing a carrier threads wait for a virtual thread. The old test includes a disabled test for Thread.holdsLock but there is further VM work required before that is useful and it would be better to develop new tests at part of that work.

This pull request has now been integrated.

Changeset: 86f97fe7
Author:    Alan Bateman <alanb at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/86f97fe70c9b55d196876e1e144f26a576492ebe
Stats:     292 lines in 4 files changed: 96 ins; 194 del; 2 mod

8305919: java/lang/Thread/virtual/HoldsLock.java#id0 failed, ThreadInfo.getLockInfo() return null

Reviewed-by: jpai

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

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