Code review request: 8005936,PrintNMTStatistics doesn't work for normal JVM exit

[Zhengyu Gu]

Thanks, David.

My bad, I do mean JVM exit handler, but rushed ...
Here is updated webrev:

http://cr.openjdk.java.net/~zgu/8005936/webrev.01/

Thanks,

-Zhengyu

On 1/11/2013 12:04 AM, David Holmes wrote:
> On 11/01/2013 6:57 AM, Zhengyu Gu wrote:
>> Moved NMT shutdown code to JVM exit handler, so guarantees that NMT
>> memory statistics can be reported when PrintNMTStatistics flag is 
>> enabled.
>>
>> webrev: http://cr.openjdk.java.net/~zgu/8005936/webrev.00/
>
> print_statistics() seems the wrong place to put the shutdown code. 
> Plus you only added it to the non-product path.
>
> David
>
>> Tests:
>>
>> 1. JPRT tests
>> 2. UTE vm.quick.testlist on Windows x64, Solaris sparv9, Linux x32 
>> and x64
>>
>>
>> Thanks,
>>
>> -Zhengyu