Here's a review for these two CRs to add 64-bite ntohll() and htonll() functions to Solaris http://dan.drydog.com/reviews/5007142-bswap/
I tested the performance gain of these functions implemented in inline assembly and C, and the inline showed a 5x-7x gain over, depending on whether it's 32- or 64-bit and on AMD64 or EM64T. Unfortunately, the functions weren't used enough to show noticable performance gains in 3des, although the code is simplified and byte swapping is not re-implemented several times. Please comment by COB Wed. Aug. 27. Ref: 5007142 Add ntohll and htonll to sys/byteorder.h 6717509 Need to use bswap/bswapq for byte swap of 64-bit integer on x32/x64 PSARC/2008/474 -- This message posted from opensolaris.org
