RFR: 6973: New Method Profiling Page for JMC

Jean-Philippe Bempel github.com+4610701+jpbempel at openjdk.java.net
Sun Nov 29 17:35:52 UTC 2020


On Thu, 26 Nov 2020 14:15:03 GMT, Jason Zaugg <jzaugg at openjdk.org> wrote:

>> I would suggest leaving the pred/succ treeviews empty when the more than one row is selected in the top methods pane. Currently, the first selected element is reflected.
>
> I've tested with this .jfr file: https://drive.google.com/file/d/1OqMOFLsODHgRPKybUfw7rK7kv3j9fES3/view?usp=sharing
> 
> Selecting `scala.runtime.Statics.releaseFence()` as the Top Method (source code https://github.com/scala/scala/blob/9a716985fcfbb687a3b4cfe97e8eb1d88a65057d/src/library/scala/runtime/Statics.java#L147-L149)
> 
> I see Successors that are not called from the method:
> 
> <img width="927" alt="image" src="https://user-images.githubusercontent.com/65551/100360273-74c66300-3044-11eb-842e-7fbe7f234170.png">
> 
> I can find no CPU sample events that include this method as a non-leaf element in the stack.
> 
> Is is possible that something is wrong with the data filtering in the new UI?

Hi @retronym, there is indeed an issue in successors. 
I need to rework the stacktrace model to give an accurate successor view

-------------

PR: https://git.openjdk.java.net/jmc/pull/165


More information about the jmc-dev mailing list