RFR: 8359174: tools/jlink/JLink20000Packages.java timed out [v8]

[Jaikiran Pai]

On Tue, 2 Sep 2025 14:27:28 GMT, Henry Jen <henryjen at openjdk.org> wrote:

>> Create a jar directly from the memory instead of real file, this should reduce the I/O overhead which likely the reason for the time out.
>> The issue is not reproducible locally, and fails intermittently, so we simply trying to reduce time needed.
>> The jar file created is verified manually running jtreg with retain=all.
>
> Henry Jen has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains nine commits:
> 
>  - Merge openjdk/master
>  - Merge openjdk/master' into JDK-8359174
>  - Clean up module dependencies
>  - Clean up jtreg directives
>  - Adapt review feedbacks
>  - Update copyright year
>  - Update copyright year
>  - cleanup
>  - 8359174: tools/jlink/JLink20000Packages.java timed out

test/jdk/tools/jlink/JLink20000Packages.java line 57:

> 55:  *          jdk.jlink/jdk.tools.jimage
> 56:  * @build tests.*
> 57:  * @run main/othervm/timeout=1920 -Xlog:init=debug -XX:+UnlockDiagnosticVMOptions -XX:+BytecodeVerificationLocal JLink20000Packages

Hello Henry, I think we should remove this `timeout=1920`. It should no longer be needed after your changes in this PR. That large value was recently added when the default timeout factor was changed from 4 to 1 in mainline.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25981#discussion_r2316587656