Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

tls: make tls a little bit faster #3629

Closed
wants to merge 1 commit into from

Conversation

indutny
Copy link
Member

@indutny indutny commented Jul 3, 2012

Compile OpenSSL with inline assembly for big numbers.

Benchmarks:

Original node

Concurrency Level:      100
Time taken for tests:   25.138 seconds
Complete requests:      10000
Failed requests:        0
Write errors:           0
Total transferred:      800000 bytes
HTML transferred:       50000 bytes
Requests per second:    397.81 [#/sec] (mean)
Time per request:       251.375 [ms] (mean)
Time per request:       2.514 [ms] (mean, across all concurrent requests)
Transfer rate:          31.08 [Kbytes/sec] received

Patched node:

Concurrency Level:      100
Time taken for tests:   18.616 seconds
Complete requests:      10000
Failed requests:        0
Write errors:           0
Total transferred:      800000 bytes
HTML transferred:       50000 bytes
Requests per second:    537.17 [#/sec] (mean)
Time per request:       186.161 [ms] (mean)
Time per request:       1.862 [ms] (mean, across all concurrent requests)
Transfer rate:          41.97 [Kbytes/sec] received

Compile OpenSSL with inline assembly for big numbers
@TooTallNate
Copy link

Will this work on ARM?

@indutny
Copy link
Member Author

indutny commented Jul 3, 2012

No, it even won't work on x86

@bnoordhuis
Copy link
Member

Easy win is easy. LGTM and nice work, Fedor. Land it in master for now.

@mranney
Copy link

mranney commented Jul 5, 2012

Woah, awesome. More speedups like this, please.

@indutny
Copy link
Member Author

indutny commented Jul 5, 2012

@bnoordhuis @isaacs are we going to merge this into 0.8?

@indutny
Copy link
Member Author

indutny commented Jul 5, 2012

Otherwise, landed there d923269

@bnoordhuis
Copy link
Member

are we going to merge this into 0.8?

Possibly. Let's use master as a testing ground.

@bnoordhuis bnoordhuis closed this Jul 5, 2012
@indutny
Copy link
Member Author

indutny commented Jul 5, 2012

Another patch for probable landing in 0.8: f210530

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants