Re: [RFC PATCH 23/28] algif: Remove hash_sendpage*()

From: Herbert Xu
Date: Thu Mar 16 2023 - 22:41:59 EST


David Howells <dhowells@xxxxxxxxxx> wrote:
> Remove hash_sendpage*() and use hash_sendmsg() as the latter seems to just
> use the source pages directly anyway.

...

> - if (!(flags & MSG_MORE)) {
> - if (ctx->more)
> - err = crypto_ahash_finup(&ctx->req);
> - else
> - err = crypto_ahash_digest(&ctx->req);

You've just removed the optimised path from user-space to
finup/digest. You need to add them back to sendmsg if you
want to eliminate sendpage.

Cheers,
--
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt