Code Review Request for 7107613, 7107616, 7185471

[Deneau, Tom]

Valerie, Max, Xuelei --

>From one who is not too familiar with the crypto architecture,
can you tell me under which provider scenario should we see
gains on 7107613 and 7107616?

-- Tom

-----Original Message-----
From: Valerie (Yu-Ching) Peng [mailto:valerie.peng at oracle.com] 
Sent: Thursday, July 26, 2012 8:14 PM
To: valerie.peng at oracle.com
Cc: OpenJDK Dev list; Deneau, Tom
Subject: Re: Code Review Request for 7107613, 7107616, 7185471

Thanks Max and Xuelei for your reviews!
I've updated the webrevs for 7107613, and 7107616:

http://cr.openjdk.java.net/~valeriep/7107613/webrev.01
http://cr.openjdk.java.net/~valeriep/7107616/webrev.01

Please let me know if you have more comments.
Thanks,
Valerie

On 07/25/12 17:21, Valerie (Yu-Ching) Peng wrote:
> Hi, Max or Xuelei,
>
> Either of you have cycles to review the following 3 perf related fixes?
> 7107613: scalability bloker in javax.crypto.CryptoPermissions
> 7107616: scalability bloker in javax.crypto.JceSecurityManager
> 7185471: Avoid key expansion when AES cipher is re-init w/ the same key
>
> Webrevs are at:
> http://cr.openjdk.java.net/~valeriep/7107613
> http://cr.openjdk.java.net/~valeriep/7107616
> http://cr.openjdk.java.net/~valeriep/7185471
>
> The changes are for JDK 8. May be backported to 7u later if necessary,
> Thanks,
> Valerie