> Please review this small test fix. > > With `-XX:ActiveProcessorCount=1`, `HeapDumpParallelTest` incorrectly expects > serial dumping, even though it configures two GC workers. The heap dump > succeeds, but the test fails because it rejects the parallel dump output. > > **Fix:** > > 1. Identify the target VM's GC from its startup log, including builds that > select Serial GC ergonomically. > 2. Remove the processor-count check, since two workers can perform a parallel > dump even on a single CPU. > 3. Request two threads explicitly for the compressed parallel dump case. > > **Testing:** > > - Passed with G1, Parallel, Serial, Epsilon, ZGC, and Shenandoah, each with > `ActiveProcessorCount=1` and `2`. > - `serviceability/dcmd/gc`: 13 passed, 1 skipped, 0 failed. > > > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Yunbo Zhang has updated the pull request incrementally with one additional commit since the last revision: 8384612: Test serviceability/dcmd/gc/HeapDumpParallelTest.java failed on a single core machine ------------- Changes: - all: https://git.openjdk.org/jdk/pull/33212/files - new: https://git.openjdk.org/jdk/pull/33212/files/9be6e91c..3b6941a3 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=33212&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=33212&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/33212.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/33212/head:pull/33212 PR: https://git.openjdk.org/jdk/pull/33212
