8222828: vmTestbase/nsk/jdi/BScenarios/multithrd/tc02x004/TestDescription.java failed

[Daniil Titov]

Thank you, Chris, JC, and Gary, for reviewing this change!

Best regards,
Daniil

On 6/7/19, 12:31 PM, "Chris Plummer" <chris.plummer at oracle.com> wrote:

    Looks good.
    
    Chris
    
    On 6/7/19 11:47 AM, Daniil Titov wrote:
    > Please review the change that fixes an intermittent failure of the test when it is run with Graal on.
    >
    > The test starts a debuggee and sets the method entry breakpoint for tc02x004aClass1 class that has only constructor defined.
    > The debuggee starts 3 threads and each of them creates a new instance of tc02x004aClass1 class. The test waits for receiving
    > 3 method entry events (posted when the constructor of tc02x004aClass1 is invoked) during a specific amount of time.
    >
    > The problem here is that with Graal on it takes more time for the test to run and occasionally the test stops listening for events
    > before all method entry events are delivered.
    >
    > The fix ensures that the time the test waits for receiving method entry events is adjusted for test.timeout.factor factor.
    >
    > The fix also makes the test waiting till the debuggee terminates before exiting.
    >
    > Webrev: http://cr.openjdk.java.net/~dtitov/8222828/webrev.01
    > Bug: https://bugs.openjdk.java.net/browse/JDK-8222828
    >
    > Thanks!
    > -Daniil
    >
    >