RFR: JDK-8213001: vmTestbase/nsk/jvmti/ThreadStart/threadstart002/TestDescription.java debug agent times out

[Gary Adams]

I could use another reviewer, or an ok to push as a trivial change.

On 1/8/19, 11:32 AM, Gary Adams wrote:
> A number failures of threadstart002 have been reported on 
> windows-x64-debug builds.
>
> This configuration has a jtreg timeout factor assigned so the test can 
> run 10x slower
> and not experience a timeout.
>
> Unfortunately, these older vmTestbase tests were not fully integrated 
> with the jtreg
> timeout and factor capabilities.
>
> This test in particular uses a 2 second timeout when monitoring 
> transitions
> between starting a thread and the check for the thread being resumed.
> This hard coded timeout is not scaled.
>
> As a quick workaround the timeout can be scaled up to allow 20 second 
> window,
> as if the scaling factor had been applied as jtreg harness intended.
>
>   Issue: https://bugs.openjdk.java.net/browse/JDK-8213001
>
> diff --git 
> a/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadStart/threadstart002/threadstart002.cpp 
> b/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadStart/threadstart002/threadstart002.cpp 
>
> --- 
> a/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadStart/threadstart002/threadstart002.cpp
> +++ 
> b/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadStart/threadstart002/threadstart002.cpp
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright (c) 2003, 2018, Oracle and/or its affiliates. All rights 
> reserved.
> + * Copyright (c) 2003, 2019, Oracle and/or its affiliates. All rights 
> reserved.
>   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>   *
>   * This code is free software; you can redistribute it and/or modify it
> @@ -34,7 +34,7 @@
>
>  #define PASSED 0
>  #define STATUS_FAILED 2
> -#define WAIT_TIME 2000
> +#define WAIT_TIME 20000
>
>  static jvmtiEnv *jvmti = NULL;
>  static jvmtiCapabilities caps;