On 02/17/2015 02:59 AM, Florian Weimer wrote:
On 02/16/2015 10:11 PM, Anthony Scarpino wrote:
Hi,

I'm requesting a code review to intrinsify the GHASH operations for both
x86 and SPARC platforms.  This greatly increases performance over
software for AES/GCM crypto operations, details are in the bug.

The review is for two repos, hotspot and jdk:

http://cr.openjdk.java.net/~ascarpino/8073108/hotspot/webrev/

Sorry for double-posting.

I looked at generate_ghash_processBlocks() and wonder if the loop needs
to be split to introduce occasional safepoints.  The TLS record size
should limit the number bytes per invocation to 16000, so perhaps this
isn't issue for the current application.

I would think TLS limits should be handled in a JSSE or other TLS area. I think GHASH should be the pure implementation.

Tony


Reply via email to