[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: PING: JDK-8148659: Add all option to JSnap
From:       Dmitry Samersoff <dmitry.samersoff () oracle ! com>
Date:       2016-03-22 17:59:26
Message-ID: 56F187FE.10807 () oracle ! com
[Download RAW message or body]

Yasumasa,

Looks good for me.

-Dmitry


On 2016-03-22 15:27, Yasumasa Suenaga wrote:
> PING: Could you review it?
> 
> Yasumasa
> 
> 2016/03/15 23:31 "Yasumasa Suenaga" <yasuenag@gmail.com
> <mailto:yasuenag@gmail.com>>:
> 
>     Hi all,
> 
>     I updated webrev to adapt to current repos. Could you review it?
> 
>         http://cr.openjdk.java.net/~ysuenaga/JDK-8148659/webrev.01/
> 
> 
>     Thanks,
> 
>     Yasumasa
> 
> 
>     On 2016/03/03 23:46, Yasumasa Suenaga wrote:
>     > Hi Dmitry,
>     >
>     >> Overall direction is to support jhsdb as the only entry point for
>     SA and
>     >> all coredump related staff.
>     >
>     > I'm working on another issue (JDK-8151181) for it.
>     >   
>     http://mail.openjdk.java.net/pipermail/serviceability-dev/2016-March/019007.html
>     >
>     > After that, I will work for this issue (JDK-8148659) and adding
>     all option
>     > to JSnap.
>     >   
>     http://mail.openjdk.java.net/pipermail/serviceability-dev/2016-March/018962.html
>     >
>     >
>     > Thanks,
>     >
>     > Yasumasa
>     >
>     >
>     > On 2016/03/03 22:55, Dmitry Samersoff wrote:
>     >> Yasumasa,
>     >>
>     >> Overall direction is to support jhsdb as the only entry point for
>     SA and
>     >> all coredump related staff.
>     >>
>     >> Could you move JSnap "into jhsdb".
>     >>
>     >> And this change requires CCC approval (I'll file it for you)
>     >>
>     >> -Dmitry
>     >>
>     >> On 2016-01-30 18:48, Yasumasa Suenaga wrote:
>     >>> I want to add -a (all) option to JSnap because JSnap is very
>     useful to analyze core image.
>     >>>
>     >>> Currently, JSnap can show PerfDataEntry which namespace is
>     java.* or com.sun.* .
>     >>> We can use jcmd with PerfCounter.print when we want to attach
>     live process, however,
>     >>> as JDK-6224040 shows, we have to use JSnap.
>     >>>
>     >>> In terms of core image analysis, -a option is very useful for
>     troubleshooting.
>     >>>
>     >>> I uploaded a webrev for this enhancement.
>     >>> Could you review it?
>     >>>
>     >>>     http://cr.openjdk.java.net/~ysuenaga/JDK-8148659/webrev.00/
>     >>>
>     >>> I'm jdk9 committer, however I cannot access JPRT.
>     >>> So I need a sponsor.
>     >>>
>     >>>
>     >>> Thanks,
>     >>>
>     >>> Yasumasa
>     >>>
>     >>
>     >>
> 


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.
[prev in list] [next in list] [prev in thread] [next in thread]