[quake3-bugzilla] [Bug 4075] New: Use native OpenSSL MD4 & MD5. It's faster!

[bugzilla-daemon at icculus.org]

http://bugzilla.icculus.org/show_bug.cgi?id=4075

           Summary: Use native OpenSSL MD4 & MD5. It's faster!
           Product: ioquake3
           Version: SVN HEAD
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P3
         Component: Misc
        AssignedTo: zakk at icculus.org
        ReportedBy: bnoordhuis at gmail.com
         QAContact: quake3-bugzilla at icculus.org


Created an attachment (id=2076)
 --> (http://bugzilla.icculus.org/attachment.cgi?id=2076)
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

-- 
Configure bugmail: http://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.