Looks good.

Best regards,
Vladimir Ivanov

On 5/3/16 10:00 PM, Brent Christian wrote:
Hi,

Please review this change which fixes buggy behavior (including SEGV) in
the experimental LiveStackFrame feature of StackWalker.

Bug:
https://bugs.openjdk.java.net/browse/JDK-8147039
Webrev:
http://cr.openjdk.java.net/~bchristi/8147039/webrev.02/

As suggested in the bug report, stack walking is reimplemented using
javaVFrame/vframe::java_sender.  The native StackWalkerAnchor class is
updated (and renamed) with a similar interface and function to
vframeStream.  I've also removed the now-unneeded
vframeStreamCommon::java_frame() function, added by 8140450.

Thanks,
-Brent

Reply via email to