On Wed, 6 Sep 2023 20:02:44 GMT, Alex Menkov <amen...@openjdk.org> wrote:
> To test ForceEarlyReturn command for NO_MORE_FRAMES case the test creates > ThreadStartEventRequest with SUSPEND_ALL policy and requests debuggee to > start new thread. > If debuggee JVM starts some internal threads before the request is cleared > (i.e. we have several ThreadStart events), 2nd event suspends debuggee again > and the test fails with timeout. > The change adds THREAD_ONLY modifier to the ThreadStartEventRequest to > generate event only for desired thread. > It requires thread ID, so debuggee was updated to create Thread object in > advance, debugger reads the thread ID from static field (it does not need to > be static, but Debugee class has convenient methods to retrieve class ID and > static field value). > > Testing: 100 runs of the test on > windows-x64-debug,linux-x64-debug,macosx-x64-debug with > JTREG_TEST_THREAD_FACTORY=Virtual, with and without "-XX:+UseZGC > -XX:+ZGenerational" This pull request has now been integrated. Changeset: 041510dc Author: Alex Menkov <amen...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/041510dc21df36d9860f4f0048241c2cabb55ee7 Stats: 46 lines in 2 files changed: 36 ins; 7 del; 3 mod 8315486: vmTestbase/nsk/jdwp/ThreadReference/ForceEarlyReturn/forceEarlyReturn002/forceEarlyReturn002.java timed out Reviewed-by: cjplummer, lmesnik ------------- PR: https://git.openjdk.org/jdk/pull/15601
