RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-05 Thread Cheleswer Sahu
Hi, Please review the code changes for https://bugs.openjdk.java.net/browse/JDK-8153319. Webrev link: http://cr.openjdk.java.net/~csahu/8153319/ Bug Brief: Test is failing on some platforms. Problem Identified: Newly created child thread (NamedThread) is finished its execution

Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-05 Thread Filipp Zhinkin
Hi Cheleswer, is it possible to use j.u.c.CountDownLatch for synchronization instead of sleep? Regards, Filipp. On Tue, Apr 5, 2016 at 1:23 PM, Cheleswer Sahu wrote: > Hi, > > > > Please review the code changes for > https://bugs.openjdk.java.net/browse/JDK-8153319. > > > > Webrev link: http://

Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-05 Thread Alexander Kulyakhtin
- Original Message - From: cheleswer.s...@oracle.com To: hotspot-runtime-...@openjdk.java.net, serviceability-dev@openjdk.java.net Sent: Tuesday, April 5, 2016 1:23:57 PM GMT +03:00 Iraq Subject: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-05 Thread Leonid Mesnik
Hi I don't think that sleep is a good way to ensure that thread is started. It is not reliable on the slow host / VM under stress and just waste of time on fast host. Is it possible just to add any explicit synchronization to ensure that NamedThread is started? Leonid On 05.04.2016 13:23, C

Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-06 Thread Dmitry Samersoff
Cheleswer, Some times ago I invented LingeredAppWithDeadlock for tests like this one. Please take a look at: .../jdk/test/sun/tools/jstack/DeadlockDetectionTest.java .../hotspot/test/serviceability/sa/DeadlockDetectionTest.java -Dmitry On 2016-04-05 13:23, Cheleswer Sahu wrote: > Hi, > >

RE: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-07 Thread Cheleswer Sahu
test serviceability/tmtools/jstack/JstackThreadTest.java fails Hi I don't think that sleep is a good way to ensure that thread is started. It is not reliable on the slow host / VM under stress and just waste of time on fast host. Is it possible just to add any explicit synchronizati

Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-07 Thread Leonid Mesnik
- From: Leonid Mesnik Sent: Tuesday, April 05, 2016 7:00 PM To: Cheleswer Sahu; hotspot-runtime-...@openjdk.java.net; serviceability-dev@openjdk.java.net Subject: Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails Hi I don't think that sleep i

Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-07 Thread Dmitry Samersoff
Sent: Tuesday, April 05, 2016 7:00 PM > To: Cheleswer Sahu; hotspot-runtime-...@openjdk.java.net; > serviceability-dev@openjdk.java.net > Subject: Re: RFR[9u-dev]: 8153319: new test > serviceability/tmtools/jstack/JstackThreadTest.java fails > > Hi > > I don't think

Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-07 Thread Daniel D. Daugherty
ssage- From: Leonid Mesnik Sent: Tuesday, April 05, 2016 7:00 PM To: Cheleswer Sahu; hotspot-runtime-...@openjdk.java.net; serviceability-dev@openjdk.java.net Subject: Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails Hi I don't think that sleep

RE: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails

2016-04-07 Thread Cheleswer Sahu
-...@openjdk.java.net; serviceability-dev@openjdk.java.net; Filipp Zhinkin Subject: Re: RFR[9u-dev]: 8153319: new test serviceability/tmtools/jstack/JstackThreadTest.java fails Cheleswer, When you re-sync with the current JDK9-hs-rt, you'll have to re-enable this test. It was quaran