RFR: 8365919: Replace currentTimeMillis with nanoTime in Stresser.java [v4]

Thomas Schatzl tschatzl at openjdk.org
Tue Aug 26 08:17:37 UTC 2025


On Tue, 26 Aug 2025 07:42:56 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Mostly a mechanic change from `currentTimeMillis()` to `nanoTime()/1000000`. It also removes `finishTime` to avoid overflowing.
>> 
>> The change in `iteration()` is to ensure `currentTime` is properly initialized.
>> 
>> (Was investigating a timeout on Windows-x64, which led me to this code. I think this fix is good enough by its own.)
>>  
>> Test: tier1-5
>
> Albert Mingkun Yang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
> 
>  - review
>  - Merge branch 'master' into test-nano-time
>  - review
>  - review
>  - Merge branch 'master' into test-nano-time
>  - test-nano-time

Marked as reviewed by tschatzl (Reviewer).

Marked as reviewed by tschatzl (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/26879#pullrequestreview-3154493898
PR Review: https://git.openjdk.org/jdk/pull/26879#pullrequestreview-3154496012


More information about the hotspot-gc-dev mailing list