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

[Serguei Spitsyn]

On Thu, 25 Feb 2021 23:14:01 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.
>
> Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Address dholmes-ora CR4 comments.

Hi Dan,
The update looks okay, but I have one question.
So, the fix in jvmtiTrace.cpp is not needed anymore?
Thanks,
Serguei

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

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