Hello

Thanks for looking into this.

Where can I find jstack/tmtools tests to run them ?

Thanks, Vladimir.

On 14.07.2015 19:21, Daniel D. Daugherty wrote:
> Webrev: http://cr.openjdk.java.net/~vkempik/8048353/webrev.00/

src/share/vm/classfile/javaClasses.cpp
    No comments. (Thanks for including the "<null>" case.)

src/share/vm/classfile/javaClasses.hpp
    No comments.

src/share/vm/runtime/vframe.cpp
    No comments.

Thumbs up (modulo running the proper tests).

Dan


On 7/14/15 9:56 AM, Daniel D. Daugherty wrote:
Adding serviceability-dev@... since jstack belongs to the
Serviceability team...

The jstack and/or tmtools tests should also be run...

Dan


On 7/14/15 9:51 AM, Vladimir Kempik wrote:
Hello,

Please review the patch to fix 8048353.
Customer has an issue with this bug when running jstack on jdk7.
He tested FVB and cofirmed it has fixed the issue.
Before pushing jdk7 backport I need to get the fix to 9 and 8.

Webrev: http://cr.openjdk.java.net/~vkempik/8048353/webrev.00/
JBS bug: https://bugs.openjdk.java.net/browse/JDK-8048353

testing: JPRT.

Thanks, Vladimir.



Reply via email to