-
Notifications
You must be signed in to change notification settings - Fork 2
Load test performance might not be relevant #8
Comments
True, not setting the same cipher was a glaring omission by me, thanks for catching that! I've ran benchmarks on my laptop using I will follow up with proper benchmarks on my main machine, but I'll first have to do some tweaking with the code (to allow setting ciphers) and the benchmark script. Hopefully I'll also update #4 then. |
Hi, As I posted on another thread where peoples start discussing about ssl vs other tls implementations: I started working on a project to compare the erlang tls libs (https://github.com/silviucpp/tls_bench). Project is still in development so might have bugs. any feedback and help is appreciated. Idea is that I found few problems that make me believes that etls benchmark is not realistic: Using my benchmark with 50 concurrent connections sending 80000, messages of 30 KB each result in : gen_tcp throughput: 3.77 GB/s (added to see the overhead of tcp) But again I'm still working to improve the benchmark and tuning and why not in the end try to compile p1_tls, fast_tls and erlang with boring ssl instead openssl. Hope this helps you. Also I will be happy to receive feedback from your side on my benchmark solution. Silviu |
Oh I forgot to mention, I got almost the same results as you for ssl, but after II tuned the erlang params It jumped to almost 320 MB/s. I suspected that something is wrong because without tuning also gen_tcp performances was very bad (almost identical with ssl) :) |
Cipher settings added as requested in #8 .
Cipher settings added as requested in #8 .
Cipher settings added as requested in #8 .
Cipher settings added as requested in #8 .
Cipher settings added as requested in #8 .
Hello,
Because etls doesn't support the cipher property where you can limit a client to a specific cipher the load test you did might not be accurate. You can't know if on both stacks erlang and etls was negotiated the same cipher. And some of them can be more cpu intensive and other less.
Silviu
The text was updated successfully, but these errors were encountered: