8361751: Test sun/tools/jcmd/TestJcmdSanity.java timed out on Windows

Alex Menkov · Alex Menkov · commit a3843e8e6e18 · 2025-07-18T18:38:26.000Z
Reviewed-by: cjplummer, dholmes, sspitsyn
diff --git a/test/jdk/sun/tools/jcmd/JcmdBase.java b/test/jdk/sun/tools/jcmd/JcmdBase.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2013, 2020, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2013, 2025, 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
@@ -102,6 +102,8 @@ private static final OutputAnalyzer jcmd(boolean requestToCurrentProcess,
                 launcher.addVMArg(vmArg);
             }
         }
+        // Some command output may be lengthy, disable streaming output to avoid deadlocks
+        launcher.addVMArg("-Djdk.attach.allowStreamingOutput=false");
         if (requestToCurrentProcess) {
             launcher.addToolArg(Long.toString(ProcessTools.getProcessId()));
         }

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright (c) 2013, 2020, Oracle and/or its affiliates. All rights reserved.`
	`2`	`+ * Copyright (c) 2013, 2025, Oracle and/or its affiliates. All rights reserved.`
`3`	`3`	`* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.`
`4`	`4`	`*`
`5`	`5`	`* This code is free software; you can redistribute it and/or modify it`
`@@ -102,6 +102,8 @@ private static final OutputAnalyzer jcmd(boolean requestToCurrentProcess,`
`102`	`102`	`launcher.addVMArg(vmArg);`
`103`	`103`	`}`
`104`	`104`	`}`
	`105`	`+ // Some command output may be lengthy, disable streaming output to avoid deadlocks`
	`106`	`+ launcher.addVMArg("-Djdk.attach.allowStreamingOutput=false");`
`105`	`107`	`if (requestToCurrentProcess) {`
`106`	`108`	`launcher.addToolArg(Long.toString(ProcessTools.getProcessId()));`
`107`	`109`	`}`