RFR: 8151181: Add JSnap to jhsdb

Yasumasa Suenaga yasuenag at gmail.com
Thu Mar 3 15:45:57 UTC 2016


Hi Dmitry,

Thank you for your comment.
I uploaded new webrev:

  hotspot: http://cr.openjdk.java.net/~ysuenaga/JDK-8151181/webrev.01/hotspot/
      jdk: http://cr.openjdk.java.net/~ysuenaga/JDK-8151181/webrev.01/jdk/


Could you review again?


Thanks,

Yasumasa


On 2016/03/04 0:15, Dmitry Samersoff wrote:
> Yasumasa,
> 
> It's better (with upcoming changes for JDK-8148659 in mind) to create a
> separate
> 
> private static boolean jsnapHelp() {
>     System.out.println(" <no option>\tdump perfromance counters");
>     return commonHelp();
> }
> 
> and write:
> 
> System.out.println(" jsnap --help\tto get more information");
> 
> Please also update:
> 
>     jdk/test/sun/tools/jhsdb/BasicLauncherTest.java
> 
> -Dmitry
> 
> 
> On 2016-03-03 17:43, Yasumasa Suenaga wrote:
>> Hi all,
>>
>> JSnap is useful SA tool to check PerfCounter.
>> So I want to add it to jhsdb.
>>
>>    http://cr.openjdk.java.net/~ysuenaga/JDK-8151181/webrev.00/
>>
>> Could you review it?
>>
>>
>> Thanks,
>>
>> Yasumasa
>>
> 
> 


More information about the serviceability-dev mailing list