On Thu, 10 Apr 2025 20:06:25 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> Don't use includevirtualthreads=y unless the test requires it. Debuggers > don't usually use includevirtualthreads=y, so we should be doing most of our > testing without it. The only reason tests use it is because some tests need > it so they can find virtual threads in the debuggee by using vm.allThreads(). > This change limits the use of includevirtualthreads=y to just those > com/sun/jdi tests that need it. > > Tested by running com/sun/jdi tests on all supported platforms in both > platform threads mode and virtual threads mode. Also tested with tier1 CI. This pull request has now been integrated. Changeset: d1d81dd0 Author: Chris Plummer <cjplum...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/d1d81dd01ca6f3fc1e4710e6055c5a3185f43d9a Stats: 17 lines in 4 files changed: 9 ins; 0 del; 8 mod 8353953: con/sun/jdi tests should be fixed to not always require includevirtualthreads=y Reviewed-by: sspitsyn, amenkov ------------- PR: https://git.openjdk.org/jdk/pull/24583
