On Fri, 17 Jun 2022 12:24:50 GMT, Claes Redestad <redes...@openjdk.org> wrote:

>> - Reduce forks, iteration, runtime to reduce runtime while maintaining high 
>> data quality on typical benchmarking hosts.
>> 
>> Reduces runtime from estimated 10+ hours to 54 minutes.
>
> Claes Redestad has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Copyrights, apply consistent settings to PermissionsImplies

test/micro/org/openjdk/bench/java/security/SSLHandshake.java line 56:

> 54: @Warmup(iterations = 5, time = 1)
> 55: @Measurement(iterations = 5, time = 1)
> 56: @Fork(value = 3)

should the `@Warmup`, `@Measurement`, and `@Fork` be removed from the 
`doHandshake()` benchmark on lines 115-117 below so that these take effect?

https://github.com/openjdk/jdk/blob/c2cbeb3ee875936c98bb15ec32d692f7d866df76/test/micro/org/openjdk/bench/java/security/SSLHandshake.java#L111-L115

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

PR: https://git.openjdk.org/jdk/pull/9189

Reply via email to