On Fri, 2 Jul 2021 19:30:11 GMT, Alex Menkov <amen...@openjdk.org> wrote:
>> The change fixes several hundreds tests which launch debugee by using uses >> Debugee.prepareDebugee() method or use >> debugee = Binder.bindToDebugee(...); >> IOPipe pipe = debugee.createIOPipe(); >> logic. >> Debugee.prepareDebugee() and Binder.bindToDebugee() launch debuggee by using >> CommandLineLaunch JDI connector with suspend=="true" argument, so they >> return debuggee suspended before the main class is loaded. >> The fix starts IOPipe listening before debuggee VM is resumed. >> >> Simplified isPackagePrivate/accipp001.java test to use >> Debugee.prepareDebugee() - it does exactly the same as >> Debugee.prepareDebugee() does (the only difference is using deprecated >> IOPipe ctor instead of Debugee.createIOPipe()) >> >> Tested all affected tests: >> test/hotspot/jtreg/vmTestbase/nsk/jdi >> test/hotspot/jtreg/vmTestbase/nsk/jdwp >> test/hotspot/jtreg/serviceability/dcmd > > Alex Menkov has updated the pull request incrementally with one additional > commit since the last revision: > > Fixed comment Ping. Need 2nd reviewer ------------- PR: https://git.openjdk.java.net/jdk/pull/4659