RFR: 8231627: ThreadsListHandleInErrorHandlingTest.java fails in printing all threads [v2]
Serguei Spitsyn
sspitsyn at openjdk.java.net
Tue Jan 5 22:43:57 UTC 2021
On Tue, 5 Jan 2021 22:39:14 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.
>
> Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:
>
> Address comments from coleenp and pchilano.
Marked as reviewed by sspitsyn (Reviewer).
-------------
PR: https://git.openjdk.java.net/jdk/pull/1891
More information about the serviceability-dev
mailing list