RFR: 8350705: [JMH] test security.SSLHandshake failed for 2 threads configuration
Daniel Jeliński
djelinski at openjdk.org
Tue Apr 1 07:16:22 UTC 2025
On Tue, 25 Mar 2025 15:38:02 GMT, Hai-May Chao <hchao 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.
Thanks @haimaychao for the review!
Can I get a second review please?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24195#issuecomment-2768403907
More information about the security-dev
mailing list