RFR: 8366462: Test gc/z/TestCommitFailure.java#Normal failed: expected output missing
Axel Boldt-Christmas
aboldtch at openjdk.org
Sun Aug 31 10:31:40 UTC 2025
The newly introduced test is intermittently failing where it tries to verify that we got a commit failure. I suggest we simply remove this verification as an initial fix.
Trying to reproduce these failure paths reliably with a blackbox test is probably not possible. So we would have to extend the JVM with whitebox hooks for doing exactly what we want. I think we need to be very careful in such a design to not affect the maintainability and readability to the code.
For now it is enough that this test intermittently exercises the commit failure paths and intermittently would reproduce any regressions.
The risk is having to be more diligent in the future with checking that this test still works when changing ZPageAllocator policies or implementation details.
-------------
Commit messages:
- 8366462: Test gc/z/TestCommitFailure.java#Normal failed: expected output missing
Changes: https://git.openjdk.org/jdk/pull/27023/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=27023&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8366462
Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/27023.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/27023/head:pull/27023
PR: https://git.openjdk.org/jdk/pull/27023
More information about the hotspot-gc-dev
mailing list