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

Coleen Phillimore coleen.phillimore at oracle.com
Fri Jan 11 08:48:02 PST 2013


This looks good to me.
Coleen

On 1/11/2013 10:18 AM, Zhengyu Gu wrote:
> 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



More information about the hotspot-dev mailing list