On Thu, 11 Feb 2021 22:04:53 GMT, Daniel D. Daugherty <[email protected]> wrote:
> A minor fix to add a new function: > > bool Thread::is_JavaThread_protected(const JavaThread* p) > > that returns true when the target JavaThread* is protected and false > otherwise. Update JavaThread::get_thread_name() to create a > ThreadsListHandle and use the new is_JavaThread_protected(). Also > update JvmtiTrace::safe_get_thread_name() to use the new > is_JavaThread_protected(). > > This fix is tested via a Mach5 Tier[1-8] run. Hi Dan, It looks good to me modulo your your discussion with Coleen on JvmtiTrace being a friend. Thanks, Serguei ------------- Marked as reviewed by sspitsyn (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/2535
