Integrated: 8353953: con/sun/jdi tests should be fixed to not always require includevirtualthreads=y
Chris Plummer
cjplummer at openjdk.org
Thu Apr 17 15:39:56 UTC 2025
On Thu, 10 Apr 2025 20:06:25 GMT, Chris Plummer <cjplummer at 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 <cjplummer at 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
More information about the serviceability-dev
mailing list