RFR: 8356706: Foreign tests timeout after passing on linux-x64-zero
Per Minborg
pminborg at openjdk.org
Thu May 15 09:54:29 UTC 2025
On Thu, 15 May 2025 09:49:10 GMT, Per Minborg <pminborg at openjdk.org> wrote:
> This PR improves the robustness of tests on slow machines such as "zero" variants. The timeout was increased to 10 minutes for both tests. There are many combinations in `TestAccessModes`, so the test takes some time even on a fast machine. In `StdLibTest`, there are numerous native calls, and these can be much slower on "zero" machines. In the latter test class, I made some attempts to improve performance. On my M1 machine, the test was about 10% faster with the changes (but I only did one run, so this might be in the error margin).
test/jdk/java/foreign/StdLibTest.java line 56:
> 54: static final Linker abi = Linker.nativeLinker();
> 55:
> 56: private static final StdLibHelper STD_LIB_HELPER = new StdLibHelper();
The `STD_LIB_HELPER` is now a constant.
test/jdk/java/foreign/StdLibTest.java line 145:
> 143: static final class StdLibHelper {
> 144:
> 145: static final MethodHandle strcat = abi.downcallHandle(abi.defaultLookup().findOrThrow("strcat"),
Drive-by cleanup to blessed order.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25248#discussion_r2090771644
PR Review Comment: https://git.openjdk.org/jdk/pull/25248#discussion_r2090769634
More information about the core-libs-dev
mailing list