RFR: 8241403: JavaThread::get_thread_name() should be ThreadSMR-aware

[Serguei Spitsyn]

On Thu, 11 Feb 2021 22:04:53 GMT, Daniel D. Daugherty <dcubed at openjdk.org> 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