-----Original Message-----
From: Herbert Xu [mailto:[email protected]] 
Sent: Tuesday, June 7, 2016 3:35 AM
To: Dey, Megha <[email protected]>
Cc: [email protected]; [email protected]; 
[email protected]; [email protected]; Yu, Fenghua 
<[email protected]>; Megha Dey <[email protected]>
Subject: Re: [PATCH V2 2/2] crypto : async implementation for sha1-mb

On Thu, Jun 02, 2016 at 07:53:50PM -0700, Megha Dey wrote:
>
> +     struct ahash_alg *shash = crypto_ahash_alg(tfm);
>  
>       /* alignment is to be done by multi-buffer crypto algorithm if 
> needed */
>  
> -     return shash->finup(desc, NULL, 0, req->result);
> +     return shash->finup(desc);

You're still poking in the guts of the API.  Now that it's a real ahash you 
don't need to do that.

Just do crypto_ahash_finup.  That way you don't need to export crypto_ahsh_alg 
either.

> I have made these changes and re-sent the patch.

Thanks,
--
Email: Herbert Xu <[email protected]> Home Page: 
http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Reply via email to