RFR(urgent): 8221472: Fix HandshakeSuspendExitTest

[Daniel D. Daugherty]

Thumbs up.

Dan


On 3/26/19 9:27 AM, Robbin Ehn wrote:
> Hi all, please review,
>
> I apparently I didn't run the test after review updates...
>
> Errors:
> ----------direct:(18/1128)----------
> /scratch/mesos/jib-master/install/jdk13-jdk.695/src.full/open/test/hotspot/jtreg/runtime/handshake/HandshakeSuspendExitTest.java:45: 
> error: cannot find symbol
>                 if (Thread.currentThread() != thr) {
>                                               ^
>   symbol:   variable thr
>   location: class HandshakeSuspendExitTest
> /scratch/mesos/jib-master/install/jdk13-jdk.695/src.full/open/test/hotspot/jtreg/runtime/handshake/HandshakeSuspendExitTest.java:46: 
> error: cannot find symbol
>                     thr.suspend();
>                     ^
>   symbol:   variable thr
>   location: class HandshakeSuspendExitTest
> /scratch/mesos/jib-master/install/jdk13-jdk.695/src.full/open/test/hotspot/jtreg/runtime/handshake/HandshakeSuspendExitTest.java:47: 
> error: cannot find symbol
>                     thr.resume();
>                     ^
>   symbol:   variable thr
>   location: class HandshakeSuspendExitTest
>
>
> Compiles and passes locally.
>
> Thanks, Robbin
>
> diff -r 16999bd91ba6 
> test/hotspot/jtreg/runtime/handshake/HandshakeSuspendExitTest.java
> --- 
> a/test/hotspot/jtreg/runtime/handshake/HandshakeSuspendExitTest.java 
> Tue Mar 26 12:08:51 2019 +0100
> +++ 
> b/test/hotspot/jtreg/runtime/handshake/HandshakeSuspendExitTest.java 
> Tue Mar 26 14:21:20 2019 +0100
> @@ -42,9 +42,9 @@
>          while (!_exit_now) {
>              // Leave last 2 threads running.
>              for (int i = 0; i < _suspend_threads.length - 2; i++) {
> -                if (Thread.currentThread() != thr) {
> -                    thr.suspend();
> -                    thr.resume();
> +                if (Thread.currentThread() != _suspend_threads[i]) {
> +                    _suspend_threads[i].suspend();
> +                    _suspend_threads[i].resume();
>                  }
>              }
>          }