RFR: 8350705: [JMH] test security.SSLHandshake failed for 2 threads configuration

Hai-May Chao hchao at openjdk.org
Tue Mar 25 15:41:14 UTC 2025


On Mon, 24 Mar 2025 12:54:01 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

> Update the SSLHandshake benchmark to enable running in multiple threads.
> 
> This PR changes the scope of the state from per-benchmark to per-thread. The server SSLContext is still shared across all threads to simulate the scenario where multiple clients try to connect to the same server at the same time.
> 
> Before the change, running this benchmark with `make test TEST=micro:SSLHandshake MICRO=OPTIONS="-t 3"` failed with an exception. After this change the benchmark completes successfully.

Changes look good.

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

Marked as reviewed by hchao (Committer).

PR Review: https://git.openjdk.org/jdk/pull/24195#pullrequestreview-2714260936


More information about the security-dev mailing list