RFR: 8353953: con/sun/jdi tests should be fixed to not always require includevirtualthreads=y [v2]
Chris Plummer
cjplummer at openjdk.org
Thu Apr 10 20:31:13 UTC 2025
> 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.
Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
Fix copyright.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/24583/files
- new: https://git.openjdk.org/jdk/pull/24583/files/bcb0d8fa..69ce42c3
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=24583&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=24583&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/24583.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/24583/head:pull/24583
PR: https://git.openjdk.org/jdk/pull/24583
More information about the serviceability-dev
mailing list