[issue30102] improve performance of libSSL usage on hashing

Gustavo Serra Scalet added the comment: No worries. I thank you also for reviewing all these changesets. I'm glad it worked in the end. -- ___ Python tracker

[issue30102] improve performance of libSSL usage on hashing

Christian Heimes added the comment: Thanks for your persistence and your initial patch. -- resolution: -> fixed stage: -> resolved status: open -> closed versions: +Python 3.6 ___ Python tracker

[issue30102] improve performance of libSSL usage on hashing

Christian Heimes added the comment: New changeset 7daa45db1d60eed4e5050bf792969893d9f2c8e0 by Christian Heimes in branch '2.7': [2.7] bpo-30102: Call OPENSSL_add_all_algorithms_noconf (GH-3112) (#3343) https://github.com/python/cpython/commit/7daa45db1d60eed4e5050bf792969893d9f2c8e0

[issue30102] improve performance of libSSL usage on hashing

Christian Heimes added the comment: New changeset 2ddea0f098b42dfd74f53bcbf08c8e68c83e1049 by Christian Heimes in branch '3.6': [3.6] bpo-30102: Call OPENSSL_add_all_algorithms_noconf (GH-3112) (#3342) https://github.com/python/cpython/commit/2ddea0f098b42dfd74f53bcbf08c8e68c83e1049

[issue30102] improve performance of libSSL usage on hashing

Changes by Christian Heimes : -- pull_requests: +3354 ___ Python tracker ___ ___

[issue30102] improve performance of libSSL usage on hashing

Christian Heimes added the comment: New changeset c941e6238ab2a8caad11fe17d4723a5d5e7a2d76 by Christian Heimes in branch 'master': bpo-30102: Call OPENSSL_add_all_algorithms_noconf (#3112) https://github.com/python/cpython/commit/c941e6238ab2a8caad11fe17d4723a5d5e7a2d76 --

[issue30102] improve performance of libSSL usage on hashing

Changes by Christian Heimes : -- pull_requests: +3353 ___ Python tracker ___ ___

[issue30102] improve performance of libSSL usage on hashing

Gustavo Serra Scalet added the comment: Is there any news on this issue? The PR 3112 also seems to be frozen at the moment. Is there some kind of code freeze happening at the moment that no reviews are taking place? -- ___ Python tracker

[issue30102] improve performance of libSSL usage on hashing

Changes by Christian Heimes : -- pull_requests: +3151 ___ Python tracker ___ ___

[issue30102] improve performance of libSSL usage on hashing

Gustavo Serra Scalet added the comment: > Since OPENSSL_config() accepts a filename, maybe a first step would be > to expose the function as ssl.OPENSSL_config(filename) to allow user to > load *explicitly* a configuration file? ssl.OPENSSL_config() would call > OPENSSL_config(NULL). Would it

[issue30102] improve performance of libSSL usage on hashing

STINNER Victor added the comment: > This small change also changes behavior of OpenSSL dramatically. What do you mean by "dramatically"? What does a openssl.cnf configuration contain? Do other applications using OpenSSL call OPENSSL_config(NULL)? Since OPENSSL_config() accepts a filename,

[issue30102] improve performance of libSSL usage on hashing

Gustavo Serra Scalet added the comment: > Christian Heimes added the comment: > > This small change also changes behavior of OpenSSL dramatically. What > are your Python and OpenSSL versions? I tested by compiling my own python3 (8aaf499) against its parent (d6d344d) For python2, I made the

[issue30102] improve performance of libSSL usage on hashing

Christian Heimes added the comment: This small change also changes behavior of OpenSSL dramatically. What are your Python and OpenSSL versions? How did you profile the performance of the hashing functions? If OpenSSL_config() really improves performance on relevant platforms, then we should

[issue30102] improve performance of libSSL usage on hashing

Changes by Gustavo Serra Scalet : -- pull_requests: +1309 ___ Python tracker ___

[issue30102] improve performance of libSSL usage on hashing

New submission from Gustavo Serra Scalet: To correctly pick the best algorithm for the current architecture, libssl needs to have OPENSSL_config(NULL) called as described on: https://wiki.openssl.org/index.php/Libcrypto_API This short change lead to a speedup of 50% on POWER8 when using