Created attachment 2076[details]
Use OpenSSL where available
Attached is a patch that uses OpenSSL's implementation of the MD4 and MD5 digest algorithms on platforms where OpenSSL is available.
Included: Linux, the *BSDs.
Excluded: Windows, Irix, OS X and Solaris. The last two should have OpenSSL but I don't know where the library lives. Someone can probably fill me in here.
ioquake3's stock MD4 implementation turned out to be 50% slower than OpenSSL's in a quick benchmark I wrote (hash 100 MB of in-memory data 100 times). Another way of putting it, is saying that OpenSSL is 33% faster but that is not quite as spectacular, is it now? :-)
Anyway, this should have a positive impact on start up and map load times.
*** MD4 ***
$ time ./md4 ioquake3
real 0m33.866s
user 0m33.578s
sys 0m0.096s
$ time ./md4 openssl
real 0m22.933s
user 0m22.649s
sys 0m0.120s
*** MD5 ***
$ time ./md5 ioquake3
real 0m35.434s
user 0m35.042s
sys 0m0.096s
$ time ./md5 openssl
real 0m31.093s
user 0m30.738s
sys 0m0.108s
Created attachment 2076 [details] Use OpenSSL where available Attached is a patch that uses OpenSSL's implementation of the MD4 and MD5 digest algorithms on platforms where OpenSSL is available. Included: Linux, the *BSDs. Excluded: Windows, Irix, OS X and Solaris. The last two should have OpenSSL but I don't know where the library lives. Someone can probably fill me in here. ioquake3's stock MD4 implementation turned out to be 50% slower than OpenSSL's in a quick benchmark I wrote (hash 100 MB of in-memory data 100 times). Another way of putting it, is saying that OpenSSL is 33% faster but that is not quite as spectacular, is it now? :-) Anyway, this should have a positive impact on start up and map load times. *** MD4 *** $ time ./md4 ioquake3 real 0m33.866s user 0m33.578s sys 0m0.096s $ time ./md4 openssl real 0m22.933s user 0m22.649s sys 0m0.120s *** MD5 *** $ time ./md5 ioquake3 real 0m35.434s user 0m35.042s sys 0m0.096s $ time ./md5 openssl real 0m31.093s user 0m30.738s sys 0m0.108s