Review for SHA2 x86_64 assembler optimizations (due March 19)

Krishna Yenduri bhargava.yenduri at sun.com
Wed Mar 19 17:54:11 PDT 2008


Dan Anderson wrote:
> Here's a review for CR 6665607 Need a SHA256/SHA384/SHA512 implementation optimized for 64-bit x86
> http://dan.drydog.com/reviews/6665607-sha2/
>   

 usr/src/common/crypto/sha2/amd64/sha512-x86_64.pl
KY-1    lines 220, 314  T3
 No need for these lines as this is just a digest algorithm.

usr/src/lib/libmd/amd64/Makefile
KY-2    lines 44-59     T4
 Consider making this a single rule using a pattern matching
 rule to match all the file names (I think it is %).

usr/src/lib/pkcs11/pkcs11_softtoken/Makefile.com
KY-3    lines 135-136
 Does this mean we have been linking sha1 and sha2 objects
 in to softtoken library so far! Good catch.

usr/src/uts/intel/sha2/Makefile
KY-4    lines 105-118   T4
 You don't need these rules if the SHA2_OBJS entry
 is moved to usr/src/uts/intel/Makefile.files.
 If they need to stay here, consider having a single rule
 for sha512-x86_64.o and sha256-x86_64.o by using
 pattern matching rules (I think it is %).


-Krishna


More information about the crypto-discuss mailing list