RFR (S): 8008536: Add HotSpot support for printing class loader statistics for JMap

[Erik Helin]

Staffan,

On 02/21/2013 12:22 PM, Staffan Larsen wrote:
> All of these changes look good to me. What I didn't realize at first what the change from PermStat.java to ClassLoaderStats.java was simply a name change - no code was changed.

I've update the changes to take this into account.

On 02/21/2013 12:22 PM, Staffan Larsen wrote:
> The 	8005116 bug says that the new option should be -classstats, but the code uses -clstats. I think this is better, but you may want to update the bug.

Agree, I've updated the bug.

Thanks,
Erik

> Thanks,
> /Staffan
>
> On 20 feb 2013, at 17:01, Erik Helin <erik.helin at oracle.com> wrote:
>
>> Hi all,
>>
>> this change is the first of three changes related to renaming the jmap
>> option permstat to clstats. The change has to be done as three separate
>> changes in order to not break any repositories:
>> 1. Add support for the -clstats option to the SA agent
>> 2. Change the tool jmap to use -clstats instead of -permstat
>> 3. Remove the option -permstat from the SA agent
>>
>> This CR represents the first step. Once this change has found its way into the tl forest, then step two can be applied. Once the result of step two is back in the hotspot-gc forest, then step three can be done.
>>
>> Step 2 and 3 are also out for review.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~ehelin/8008536/webrev.00/
>>
>> Bug:
>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008536
>>
>> Testing:
>> JPRT
>>
>> Thanks,
>> Erik
>