Re: RFR(S): JDK-8131694 backout the fix for JDK-8131331 when JDK-8131693 is fixed

2015-12-04 Thread Daniel D. Daugherty
I believe so... but I'm not a Serviceability expert :-) Dan On 12/4/15 11:16 AM, Dmitry Samersoff wrote: Dan, I run RBT job with vm.tmtools testset. Is it enough? -Dmitry On 2015-12-04 00:50, Daniel D. Daugherty wrote: On 12/3/15 1:01 PM, Dmitry Samersoff wrote: Everybody, Please review:

Re: RFR(S): JDK-8131694 backout the fix for JDK-8131331 when JDK-8131693 is fixed

2015-12-04 Thread Dmitry Samersoff
Dan, I run RBT job with vm.tmtools testset. Is it enough? -Dmitry On 2015-12-04 00:50, Daniel D. Daugherty wrote: > On 12/3/15 1:01 PM, Dmitry Samersoff wrote: >> Everybody, >> >> Please review: >> >> http://cr.openjdk.java.net/~dsamersoff/JDK-8131694/webrev.01/ > > src/share/vm/runtime/vframe

Re: RFR(S): JDK-8131694 backout the fix for JDK-8131331 when JDK-8131693 is fixed

2015-12-03 Thread Daniel D. Daugherty
On 12/3/15 1:01 PM, Dmitry Samersoff wrote: Everybody, Please review: http://cr.openjdk.java.net/~dsamersoff/JDK-8131694/webrev.01/ src/share/vm/runtime/vframe.cpp No comments. The backout of the fix for JDK-8131331 looks fine. Did you run the tmtools test suite to verify? Are there oth

RFR(S): JDK-8131694 backout the fix for JDK-8131331 when JDK-8131693 is fixed

2015-12-03 Thread Dmitry Samersoff
Everybody, Please review: http://cr.openjdk.java.net/~dsamersoff/JDK-8131694/webrev.01/ -Dmitry -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.