Integrated: 8231627: ThreadsListHandleInErrorHandlingTest.java fails in printing all threads

Daniel D.Daugherty dcubed at openjdk.java.net
Wed Jan 6 14:40:02 UTC 2021


On Thu, 24 Dec 2020 17:33:21 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

> A small robustness fix in ThreadsSMRSupport::print_info_on() to reduce the
> likelihood of crashes during error reporting. Uses Threads_lock->try_lock()
> for safety and restricts some reporting to when the Threads_lock has been
> acquired (depends on JDK-8256383). Uses a ThreadsListHandle to make
> the current ThreadsList safe for reporting (depends on JDK-8258284). Also
> detects when the system ThreadsList (_java_thread_list) has changed and
> will warn that some of the reported info may now be stale.
> 
> Two existing tests have been updated to reflect the use of a ThreadsListHandle
> in ThreadsSMRSupport::print_info_on(). Mach5 Tier[1-6] testing has no regressions.

This pull request has now been integrated.

Changeset: c0540ffd
Author:    Daniel D. Daugherty <dcubed at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/c0540ffd
Stats:     95 lines in 3 files changed: 32 ins; 9 del; 54 mod

8231627: ThreadsListHandleInErrorHandlingTest.java fails in printing all threads

Reviewed-by: eosterlund, coleenp, pchilanomate, sspitsyn

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

PR: https://git.openjdk.java.net/jdk/pull/1891


More information about the serviceability-dev mailing list