On Thu, 15 Aug 2024 21:12:20 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> There are issues with the test attaching to the debuggee too soon, and the > debug agent isn't ready. yet. This test is based on ProcessAttachTest, which > does not have this issue. I eventually realized the reason why is because > ProcessAttachTest has this little snippet of code, which I had removed from > DataDumpTest: > > // Wait for the process to start > InputStream is = p.getInputStream(); > is.read(); > > This is waiting for the start of the debug agent's "Listening..." message. > I've re-added this to DataDumpTest, and it seems to fix the issue. Tested by > running 20 times with `-Xcomp` on all supported platforms with no failures. > It used to fail about 2/3 of the time. This pull request has now been integrated. Changeset: 8635642d Author: Chris Plummer <cjplum...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/8635642dbdfb74d2ae50a51611fd2c5980fe6e74 Stats: 7 lines in 1 file changed: 5 ins; 0 del; 2 mod 8338469: com/sun/jdi/DataDumpTest.java failed with Not a debuggee, or not listening for debugger to attach Reviewed-by: dcubed ------------- PR: https://git.openjdk.org/jdk/pull/20604
