> 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 ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/4659/files - new: https://git.openjdk.java.net/jdk/pull/4659/files/8c46ee02..854df1c9 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4659&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4659&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/4659.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/4659/head:pull/4659 PR: https://git.openjdk.java.net/jdk/pull/4659