Re: [PATCH 16/18] crypto: talitos - do hw_context DMA mapping outside the requests

From: Herbert Xu
Date: Thu Feb 22 2018 - 08:46:45 EST


On Thu, Feb 22, 2018 at 12:29:28PM +0000, Horia GeantÄ wrote:
>
> IIUC this means that there is no room for improvement.
> This patch needs to be reverted, to restore previous behaviour when the
> hw_context was mapped / unmapped for every request.

In general we should avoid trying to do batching in drivers. Such
optimisations should instead be done at a higher level. For example,
for disk encryption we want to do the aggregation at the block layer
rather than the crypto API because that has innate knowledge of the
data layout which we can only guess.

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