RFR 8166642: serviceability/dcmd/framework/* timeout (original) (raw)

daniil.x.titov at oracle.com daniil.x.titov at oracle.com
Wed Mar 14 16:47:17 UTC 2018


Hi Chris and David,

Could you please say is anything else required or you are OK with these changes?

As Chris already replied there are only 3 tests that will be affected and each of them takes less than 5 seconds to complete.

On 3/13/18 10:50 PM, Chris Plummer wrote:

Hi Danill,

The fix looks good. Were you able to reproduce this problem, and then after the fix run the tests enough times to be confident this really resolves the issue? I was able to reproduce this problem with Mach5 . There were about 1-3 failures per 100 runs of hotspot_serviceability suite. After the fix the tests were run more then 1000 times without failures. Are you going to close JDK-8194057 as a dup? Yes. I plan to close JDK-8194057 as a duplicate. thanks, Chris

Thanks!

Best regards, Daniil

On 3/13/18 10:26 PM, Daniil Titov wrote:

Please review the changes that fix intermittent timeout failure of serviceability/dcmd/framework/* tests.

The problem here is that these tests invoke jcmd in different ways and one of such ways is when a main class is passed to the jcmd as a VM identifier. The main class for jtreg test is com.sun.javatest.regtest.agent.MainWrapper and in some cases more than one test are running in parallel and there are multiple Java processes with com.sun.javatest.regtest.agent.MainWrapper as a main class . When it happens jcmd iterates over all Java processes that match the condition (the main class equals to com.sun.javatest.regtest.agent.MainWrapper) and executes the command for each of them. That results in the jcmd invokes the given command multiple times and attaches to Java processes not related to the current test. The fix makes serviceability/dcmd/framework/* tests non-concurrent to ensure that they don't interact with other tests. Bug: https://bugs.openjdk.java.net/browse/JDK-8166642 Webrev: http://cr.openjdk.java.net/~dtitov/8166642/webrev.01 The tests ran successfully with Mach5. Best regards, Daniil



More information about the serviceability-dev mailing list