RFR 8211821: PrintStringTableStatistics crashes JVM

[Harold David Seigel]

Thanks Ioi!

Harold


On 10/10/2018 5:26 PM, Ioi Lam wrote:
> Hi Harold,
>
> The fix looks good to me.
>
> Thanks
>
> - Ioi
>
>
> On 10/10/2018 01:44 PM, Harold David Seigel wrote:
>> Hi,
>>
>> Please review this fix for JDK-8211821.  The change fixes the problem 
>> by moving the call to exit_globals() in Threads::destroy_vm() to 
>> before the thread gets deleted. Although exit_globals() deletes tty, 
>> this is not a problem because neither deleting thread nor freeing the 
>> jvmci_old_thread_counters use tty.
>>
>> Open Webrev: http://cr.openjdk.java.net/~hseigel/bug_8211821/webrev/
>>
>> JBS Bug:  https://bugs.openjdk.java.net/browse/JDK-8211821
>>
>> The fix was tested by running the new test and Mach5 tiers 1 and 2 
>> tests and builds on Linux-x64, Windows, and Mac OS X, running tiers 
>> 3-5 tests on Linux-x64, and by running JCK-12 API, Lang and VM tests 
>> on Linux-x64.
>>
>> Thanks, Harold
>>
>