code review for 6808439
Krishna Yenduri
bhargava.yenduri at sun.com
Wed Mar 4 11:09:15 PST 2009
Mark,
> I need a code review for
> 6808439 mode code should not special case inplace operations
>
> The webrev is at
> http://cr.opensolaris.org/~mcpowers/inplace
I see that we have less number of steps in the old code for the inplace
case
presumably for performance. For example, in modes/ecb.c, we don't do
the following for inplace case -
crypto_init_ptrs(), crypto_get_ptrs() and bcopy().
The extra bcopy concerns me.
Please check if there is any performance impact for the inplace case
with your changes. Hopefully, it is insignificant.
-Krishna
More information about the crypto-discuss
mailing list