8168414: Various timeouthandler fixes

Reviewed-by: mlarsson

test/failure_handler/Makefile

@@ -94,6 +94,7 @@ test: require_env build
         -agentvm                                                            \
         -thd:"${TARGET_JAR}"                                                \
         -th:jdk.test.failurehandler.jtreg.GatherProcessInfoTimeoutHandler   \
+        -thtimeout:0                                                        \
         -od:"${TARGET_JAR}"                                                 \
         -o:jdk.test.failurehandler.jtreg.GatherDiagnosticInfoObserver       \
         -w:"${RUN_DIR}/JTwork"                                              \

test/failure_handler/src/share/classes/jdk/test/failurehandler/action/ActionHelper.java

@@ -164,7 +164,8 @@ public class ActionHelper {
         Stopwatch stopwatch = new Stopwatch();
         stopwatch.start();
 
-        log.printf("%s%n[%tF %<tT] %s%n%1$s%n", line, new Date(), pb.command());
+        log.printf("%s%n[%tF %<tT] %s timeout=%s%n%1$s%n", line, new Date(), pb.command(), params.timeout);
+
         Process process;
         KillerTask killer;
 
@@ -178,22 +179,20 @@ public class ActionHelper {
                     out);
             try {
                 result = new ExitCode(process.waitFor());
-                killer.cancel();
             } catch (InterruptedException e) {
-                Thread.currentThread().interrupt();
+                log.println("WARNING: interrupted when waiting for the tool:%n");
-                if (!killer.cancel()) {
-                    log.println(
-                            "WARNING: interrupted when waiting for the tool:");
                 e.printStackTrace(log);
-                }
+            } finally {
+                killer.cancel();
             }
             if (killer.hasTimedOut()) {
                 log.printf(
                         "WARNING: tool timed out: killed process after %d ms%n",
-                        TimeUnit.MILLISECONDS.toMicros(params.timeout));
+                        params.timeout);
                 result = ExitCode.TIMED_OUT;
             }
         } catch (IOException e) {
+            log.printf("WARNING: caught IOException while running tool%n");
             e.printStackTrace(log);
             result = ExitCode.LAUNCH_ERROR;
         }
@@ -201,7 +200,7 @@ public class ActionHelper {
         stopwatch.stop();
         log.printf("%s%n[%tF %<tT] exit code: %d time: %d ms%n%1$s%n",
                 line, new Date(), result.value,
-                TimeUnit.MILLISECONDS.toSeconds(stopwatch.getElapsedTimeNs()));
+                TimeUnit.NANOSECONDS.toMillis(stopwatch.getElapsedTimeNs()));
         return result;
     }
 
@@ -247,7 +246,7 @@ public class ActionHelper {
             log.printf("WARNING: can't run jps : %s%n", e.getMessage());
             e.printStackTrace(log);
         } catch (InterruptedException e) {
-            Thread.currentThread().interrupt();
+            log.printf("WARNING: interrupted%n");
             e.printStackTrace(log);
         }
         return result;
@@ -266,8 +265,7 @@ public class ActionHelper {
             try {
                 process.exitValue();
             } catch (IllegalThreadStateException e) {
-                // !prepareProcess.isAlive()
+                process.destroyForcibly();
-                process.destroy();
                 timedOut = true;
             }
         }
@@ -301,13 +299,17 @@ public class ActionHelper {
                             exitCode.value);
                     break;
                 }
+                // sleep, if this is not the last iteration
+                if (i < n - 1) {
                     try {
                         Thread.sleep(params.pause);
                     } catch (InterruptedException e) {
-                    Thread.currentThread().interrupt();
+                        sectionWriter.printf(
+                                "WARNING: interrupted while sleeping between invocations");
                         e.printStackTrace(sectionWriter);
                     }
                 }
+            }
         } else {
             run(section.getWriter(), section.getWriter(), process, params);
         }