RFR(xs): 8229778: TestJstatdDefaults.java failed due to "fatal error: LEAF method calling lock?"

[coleen.phillimore at oracle.com]

This looks good (apart from typo).  Thank you for adding the comment why.
Coleen

On 9/23/19 8:27 AM, David Holmes wrote:
> Hi Robbin,
>
> On 23/09/2019 10:14 pm, Robbin Ehn wrote:
>> Hi all, please review.
>>
>> Threads_lock is an always safepoint checked lock.
>> We can end up in block_if_vm_exited in all sorts of states.
>> But only after final safepoint started, so we just set state in vm 
>> before locking it.
>>
>> Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8229778
>>
>> Change-set:
>> http://cr.openjdk.java.net/~rehn/8229778/webrev/
>
> !     // proper tranisition, but it happens after the final safepoint 
> have begun.
>
> s/tranisition/transition/
> s/have/has/
>
> otherwise fix is fine.
>
> Thanks,
> David
> -----
>
>> Passes t1-2, x64 svc_tools and TestJstatdDefaults.
>>
>> Thanks, Robbin