Code Review Request for 7107613, 7107616, 7185471

Valerie (Yu-Ching) Peng valerie.peng at oracle.com
Fri Jul 27 18:18:06 UTC 2012


The changes in 107613 and 7107616 are related to the JCE framework which means they are applied regardless which provider is used.
Valerie

On 07/27/12 08:46, Deneau, Tom wrote:
> 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
>
>




More information about the security-dev mailing list