RFR: 8321400: java/foreign/TestStubAllocFailure.java fails with code cache exhaustion
Jorn Vernee
jvernee at openjdk.org
Mon Dec 11 13:47:30 UTC 2023
Improve the test by being more lenient to related code cache exhaustion errors. The important thing is that we don't terminate with a fatal error, which the new code now checks for explicitly. The check for that is based on what is done by `./test/hotspot/jtreg/runtime/ErrorHandling/HsErrFileUtils.java` .
The existing `UpcallTestHelper.Output` class that was previously used to assert on stdout/stderr contents did not have the capability to look for patterns in the output. So, I've taken the opportunity to replace it with the more canonical `OutputAnalyzer` which comes from the test library.
Finally, I've also added back the test for downcall stub allocation failure which was removed as part of the initial patch because it was too inconsistent [1]. With the now approach, it should pass reliably as well.
Testing: `jdk_foreign` suite (which contains all the affected tests)
[1]: https://github.com/openjdk/jdk/pull/16311/commits/9a1360598a91871ce6ec48330849c0e4e0279c64
-------------
Commit messages:
- use match
- Improve stub alloc test
Changes: https://git.openjdk.org/jdk/pull/17056/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17056&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8321400
Stats: 106 lines in 7 files changed: 47 ins; 42 del; 17 mod
Patch: https://git.openjdk.org/jdk/pull/17056.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/17056/head:pull/17056
PR: https://git.openjdk.org/jdk/pull/17056
More information about the core-libs-dev
mailing list