RFR 8211821: PrintStringTableStatistics crashes JVM

[Ioi Lam]

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
>