8268433: serviceability/dcmd/framework/VMVersionTest.java fails with Unable to send object throw not established PipeIO Listener Thread connection

Alex Menkov · Alex Menkov · commit 280f2d57d1f7 · 2021-06-24T00:28:45.000Z
Reviewed-by: sspitsyn, cjplummer
diff --git a/test/hotspot/jtreg/ProblemList-zgc.txt b/test/hotspot/jtreg/ProblemList-zgc.txt
@@ -40,7 +40,3 @@ serviceability/sa/ClhsdbPmap.java#id1                         8268722   macosx-x
 serviceability/sa/ClhsdbPstack.java#id1                       8268722   macosx-x64
 serviceability/sa/TestJmapCore.java                           8268722,8268283   macosx-x64,linux-aarch64
 serviceability/sa/TestJmapCoreMetaspace.java                  8268722,8268636   macosx-x64,linux-x64
-
-serviceability/dcmd/framework/HelpTest.java                   8268433 windows-x64
-serviceability/dcmd/framework/InvalidCommandTest.java         8268433 windows-x64
-serviceability/dcmd/framework/VMVersionTest.java              8268433 windows-x64
diff --git a/test/hotspot/jtreg/serviceability/dcmd/framework/TestProcessLauncher.java b/test/hotspot/jtreg/serviceability/dcmd/framework/TestProcessLauncher.java
@@ -77,7 +77,11 @@ public Process launch() {
 
     public void quit() {
         if (pipe != null) {
-            pipe.println("quit");
+            if (pipe.isConnected()) {
+                pipe.println("quit");
+            } else {
+                System.out.println("WARNING: IOPipe is not connected");
+            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,11 @@ public Process launch() {`
`77`	`77`
`78`	`78`	`public void quit() {`
`79`	`79`	`if (pipe != null) {`
`80`		`- pipe.println("quit");`
	`80`	`+ if (pipe.isConnected()) {`
	`81`	`+ pipe.println("quit");`
	`82`	`+ } else {`
	`83`	`+ System.out.println("WARNING: IOPipe is not connected");`
	`84`	`+ }`
`81`	`85`	`}`
`82`	`86`	`}`
`83`	`87`