dot-asm / cryptogams

CRYPTOGAMS distribution repository
Other
56 stars 16 forks source link

Performance regression in SHA1 #10

Closed dwmw2 closed 3 years ago

dwmw2 commented 3 years ago

I updated to the latest Cryptogams code, from whatever I'd lifted from OpenSSL last time I was playing with this: https://gitlab.com/openconnect/openconnect/-/commit/4abf01865dea0feb43c328c3a5311533fca8640f

The update reduces the performance of the stitched AES+SHA1 encryption code by about 10%, and decryption by even more.

Ryzen 5

The test case is part of the OpenConnect test suite in my 'hacks2' branch which (currently) contains the commit linked above.

dwmw2 commented 3 years ago

On Haswell, it was using sha1_block_data_order_avx2 before. Now it's just sha1_block_data_order.

dwmw2 commented 3 years ago

Oh, this is just because sha1-x86_64.pl hasn't been updated in cryptogams since the repository was created. The one in OpenSSL is much newer.