mbox series

[v2,0/6] crypto: tcrypt: fix and add multi buf speed tests

Message ID 1513499346-9047-1-git-send-email-gilad@benyossef.com
Headers show
Series crypto: tcrypt: fix and add multi buf speed tests | expand

Message

Gilad Ben-Yossef Dec. 17, 2017, 8:28 a.m. UTC
The performance of some crypto tfm providers is affected by
the amount of parallelism possible with the processing.

We already had some support for speed test of multiple concurrent
requests, dubbed multi buffer, in ahash speed tests.
                                                                                
This patch set extends said support and add similar support for skcipher
and AEAD, as well as fixes some odd bugs discovered along the way.
                                                                                
It is noted that it is possible to consolidate some of the none multi
buffer speed test code better, but given that tcrypt as a whole is a
developer testing harness rather than production code, the value of
this activity seems questionable. Do let me know if you disagree.

Signed-off-by: Gilad Ben-Yossef <gilad@benyossef.com>


Changes from v1:
- re-spin on top of latest cryptodev

Gilad Ben-Yossef (6):
  crypto: tcrypt: use multi buf for ahash mb test
  crypto: tcrypt: fix AEAD decryption speed test
  crypto: tcrypt: allow setting num of bufs
  crypto: tcrypt: add multi buf ahash jiffies test
  crypto: tcrypt: add multibuf skcipher speed test
  crypto: tcrypt: add multibuf aead speed test

 crypto/tcrypt.c | 1079 +++++++++++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 975 insertions(+), 104 deletions(-)

-- 
2.7.4

Comments

Herbert Xu Dec. 28, 2017, 7:02 a.m. UTC | #1
On Sun, Dec 17, 2017 at 08:28:59AM +0000, Gilad Ben-Yossef wrote:
> The performance of some crypto tfm providers is affected by

> the amount of parallelism possible with the processing.

> 

> We already had some support for speed test of multiple concurrent

> requests, dubbed multi buffer, in ahash speed tests.

>                                                                                 

> This patch set extends said support and add similar support for skcipher

> and AEAD, as well as fixes some odd bugs discovered along the way.

>                                                                                 

> It is noted that it is possible to consolidate some of the none multi

> buffer speed test code better, but given that tcrypt as a whole is a

> developer testing harness rather than production code, the value of

> this activity seems questionable. Do let me know if you disagree.

> 

> Signed-off-by: Gilad Ben-Yossef <gilad@benyossef.com>


All applied.  Thanks.
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt