> Can I please get a review of these test-only changes which proposes to remove > the `test/jdk/sun/tools/jcmd/JcmdOutputEncodingTest.java` and > `test/jdk/sun/tools/jstack/BasicJStackTest.java` tests from > `ProblemList-Virtual.txt`? > > When jtreg was enhanced to allow running the tests from within a virtual > (main) thread, some tests were problem listed since they either were failing > at that time or the test code would require additional work to make it work > when the current thread is a virtual thread. The > `test/jdk/sun/tools/jcmd/JcmdOutputEncodingTest.java` and > `test/jdk/sun/tools/jstack/BasicJStackTest.java` are 2 such threads which > require special handling when the current thread is a virtual thread. > > `test/jdk/sun/tools/jcmd/JcmdOutputEncodingTest.java` has been updated to use > a different command to dump stacktraces when the test runs in a virtual > thread. I went back and looked at the original issue for which this test was > introduced and based on that, using a different command to dump the > stacktraces shouldn't impact what the test was originally intended to test. > > `test/jdk/sun/tools/jstack/BasicJStackTest.java` has been updated to be > skipped when the current thread is the virtual thread. This is because the > test exercises the `jstack` tool which doesn't print stacktraces of virtual > threads and thus the test isn't applicable for virtual threads. > > I've run these tests with this change, both with platform threads and virtual > threads in our CI and the tests complete without failures.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: address additional problem listed tests in hotspot/jtreg/serviceability ------------- Changes: - all: https://git.openjdk.org/jdk/pull/19016/files - new: https://git.openjdk.org/jdk/pull/19016/files/201be489..6193cfbb Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=19016&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19016&range=00-01 Stats: 20 lines in 3 files changed: 12 ins; 4 del; 4 mod Patch: https://git.openjdk.org/jdk/pull/19016.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/19016/head:pull/19016 PR: https://git.openjdk.org/jdk/pull/19016
