8303421: [BACKOUT] 8303133: Update ProcessTools.startProcess(...) to exit early if process exit before linePredicate is printed.

Reviewed-by: dholmes

Author: lmesnik

test/lib/jdk/test/lib/process/ProcessTools.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2013, 2023, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2013, 2022, 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
@@ -216,23 +216,15 @@ protected void processLine(String line) {
 
         try {
             if (timeout > -1) {
-                // Every second check if line is printed and if process is still alive
-                Utils.waitForCondition(() -> latch.getCount() == 0 || !p.isAlive(),
-                        unit.toMillis(Utils.adjustTimeout(timeout)), 1000);
-
-                if (latch.getCount() > 0) {
-                    if (!p.isAlive()) {
-                        // Give some extra time for the StreamPumper to run after the process completed
-                        Thread.sleep(1000);
-                        if (latch.getCount() > 0) {
-                            throw new RuntimeException("Started process " + name + " terminated before producing the expected output.");
-                        }
-                    } else {
+                if (timeout == 0) {
+                    latch.await();
+                } else {
+                    if (!latch.await(Utils.adjustTimeout(timeout), unit)) {
                         throw new TimeoutException();
                     }
                 }
             }
-        } catch (TimeoutException | RuntimeException | InterruptedException e) {
+        } catch (TimeoutException | InterruptedException e) {
             System.err.println("Failed to start a process (thread dump follows)");
             for (Map.Entry<Thread, StackTraceElement[]> s : Thread.getAllStackTraces().entrySet()) {
                 printStack(s.getKey(), s.getValue());

test/lib/jdk/test/lib/thread/ProcessThread.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2013, 2023, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2013, 2018, 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
@@ -150,21 +150,16 @@ public ProcessRunnable(ProcessBuilder pb, String name, Predicate<String> waitfor
          */
         @Override
         public void xrun() throws Throwable {
-            try {
-                this.process = ProcessTools.startProcess(name, processBuilder, waitfor);
-            } catch (Throwable t) {
-                System.out.println(String.format("ProcessThread[%s] failed: %s", name, t.toString()));
-                throw t;
-            } finally {
-                // Release when process is started or failed
-                latch.countDown();
-            }
+            this.process = ProcessTools.startProcess(name, processBuilder, waitfor);
+            // Release when process is started
+            latch.countDown();
 
+            // Will block...
             try {
-                output = new OutputAnalyzer(this.process);
-                // Will block...
                 this.process.waitFor();
+                output = new OutputAnalyzer(this.process);
             } catch (Throwable t) {
+                String name = Thread.currentThread().getName();
                 System.out.println(String.format("ProcessThread[%s] failed: %s", name, t.toString()));
                 throw t;
             } finally {