RFR(xxs): 8144702: Using tid decorator in Unified Logging may crash VM

[Coleen Phillimore]

This looks good.
Coleen

On 12/4/15 7:45 AM, Thomas Stüfe wrote:
> Thanks, Markus!
>
> On Fri, Dec 4, 2015 at 1:12 PM, Marcus Larsson <marcus.larsson at oracle.com>
> wrote:
>
>> Hi Thomas,
>>
>>
>> On 2015-12-04 13:05, Thomas Stüfe wrote:
>>
>>> Hi all,
>>>
>>> please review this tiny fix:
>>>
>>> webrev:
>>>
>>> http://cr.openjdk.java.net/~stuefe/webrevs/8144702-Using-tid-decorator-in-Unified-Logging-may-crash-VM/webrev.00/webrev/
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8144702
>>>
>> Looks good. Thanks for fixing this!
>>
>> Regards,
>> Marcus
>>
>>
>>
>>> Unified Logging uses Thread::current() when printing out the "tid"
>>> decorator, which will crash the VM if logging is done after thread start
>>> and before thread TLS slot initialization or after thread TLS slot cleanup
>>> and before thread finish.
>>>
>>> Kind regards, Thomas
>>>
>>