[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8269770: nsk tests should start IOPipe channel before launch debuggee - Debugee.prepareDebu
From:       Alex Menkov <amenkov () openjdk ! java ! net>
Date:       2021-07-26 20:20:33
Message-ID: poGjZE28Fiz6E2Lv6jYpSjnbGA6IsnMN5RvIGJKId34=.b77e67b4-57c6-4617-a6b2-e657bf1a5605 () github ! com
[Download RAW message or body]

On Fri, 2 Jul 2021 19:30:11 GMT, Alex Menkov <amenkov@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


[prev in list] [next in list] [prev in thread] [next in thread]