DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] app/crypto-perf: change buffer size based on optype
@ 2023-04-06  8:16 Tejasree Kondoj
  2023-04-07 10:19 ` Power, Ciara
  2023-04-10  3:10 ` Anoob Joseph
  0 siblings, 2 replies; 4+ messages in thread
From: Tejasree Kondoj @ 2023-04-06  8:16 UTC (permalink / raw)
  To: Akhil Goyal, Fan Zhang, Ciara Power; +Cc: Anoob Joseph, dev

Adding digest size to buffer size in auth-then-cipher
mode with auth_op set to generate as cipher length
includes digest size also.

Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
---
 app/test-crypto-perf/cperf_options_parsing.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/app/test-crypto-perf/cperf_options_parsing.c b/app/test-crypto-perf/cperf_options_parsing.c
index 8c9f632590..849a5d6bb4 100644
--- a/app/test-crypto-perf/cperf_options_parsing.c
+++ b/app/test-crypto-perf/cperf_options_parsing.c
@@ -1064,6 +1064,10 @@ check_cipher_buffer_length(struct cperf_options *options)
 		else
 			buffer_size = options->buffer_size_list[0];
 
+		if ((options->auth_op == RTE_CRYPTO_AUTH_OP_GENERATE) &&
+				(options->op_type == CPERF_AUTH_THEN_CIPHER))
+			buffer_size += options->digest_sz;
+
 		while (buffer_size <= options->max_buffer_size) {
 			if ((buffer_size % AES_BLOCK_SIZE) != 0) {
 				RTE_LOG(ERR, USER1, "Some of the buffer sizes are "
@@ -1090,6 +1094,10 @@ check_cipher_buffer_length(struct cperf_options *options)
 		else
 			buffer_size = options->buffer_size_list[0];
 
+		if ((options->auth_op == RTE_CRYPTO_AUTH_OP_GENERATE) &&
+				(options->op_type == CPERF_AUTH_THEN_CIPHER))
+			buffer_size += options->digest_sz;
+
 		while (buffer_size <= options->max_buffer_size) {
 			if ((buffer_size % DES_BLOCK_SIZE) != 0) {
 				RTE_LOG(ERR, USER1, "Some of the buffer sizes are "
-- 
2.25.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [PATCH] app/crypto-perf: change buffer size based on optype
  2023-04-06  8:16 [PATCH] app/crypto-perf: change buffer size based on optype Tejasree Kondoj
@ 2023-04-07 10:19 ` Power, Ciara
  2023-04-10  3:10 ` Anoob Joseph
  1 sibling, 0 replies; 4+ messages in thread
From: Power, Ciara @ 2023-04-07 10:19 UTC (permalink / raw)
  To: Tejasree Kondoj, Akhil Goyal, Fan Zhang; +Cc: Anoob Joseph, dev



> -----Original Message-----
> From: Tejasree Kondoj <ktejasree@marvell.com>
> Sent: Thursday 6 April 2023 09:16
> To: Akhil Goyal <gakhil@marvell.com>; Fan Zhang
> <royzhang1980@gmail.com>; Power, Ciara <ciara.power@intel.com>
> Cc: Anoob Joseph <anoobj@marvell.com>; dev@dpdk.org
> Subject: [PATCH] app/crypto-perf: change buffer size based on optype
> 
> Adding digest size to buffer size in auth-then-cipher mode with auth_op set
> to generate as cipher length includes digest size also.
> 
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> ---
>  app/test-crypto-perf/cperf_options_parsing.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 

Acked-by: Ciara Power <ciara.power@intel.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [PATCH] app/crypto-perf: change buffer size based on optype
  2023-04-06  8:16 [PATCH] app/crypto-perf: change buffer size based on optype Tejasree Kondoj
  2023-04-07 10:19 ` Power, Ciara
@ 2023-04-10  3:10 ` Anoob Joseph
  2023-05-24  9:29   ` Akhil Goyal
  1 sibling, 1 reply; 4+ messages in thread
From: Anoob Joseph @ 2023-04-10  3:10 UTC (permalink / raw)
  To: Tejasree Kondoj, Akhil Goyal, Fan Zhang, Ciara Power
  Cc: dev, Jerin Jacob Kollanukkaran

> Adding digest size to buffer size in auth-then-cipher mode with auth_op set to
> generate as cipher length includes digest size also.
> 
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> ---
>  app/test-crypto-perf/cperf_options_parsing.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 

Acked-by: Anoob Joseph <anoobj@marvell.com>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [PATCH] app/crypto-perf: change buffer size based on optype
  2023-04-10  3:10 ` Anoob Joseph
@ 2023-05-24  9:29   ` Akhil Goyal
  0 siblings, 0 replies; 4+ messages in thread
From: Akhil Goyal @ 2023-05-24  9:29 UTC (permalink / raw)
  To: Anoob Joseph, Tejasree Kondoj, Fan Zhang, Ciara Power
  Cc: dev, Jerin Jacob Kollanukkaran

> Subject: RE: [PATCH] app/crypto-perf: change buffer size based on optype
> 
> > Adding digest size to buffer size in auth-then-cipher mode with auth_op set to
> > generate as cipher length includes digest size also.
> >
> > Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> > ---
> >  app/test-crypto-perf/cperf_options_parsing.c | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> >
> 
> Acked-by: Anoob Joseph <anoobj@marvell.com>
Applied to dpdk-next-crypto

Thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-05-24  9:29 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-06  8:16 [PATCH] app/crypto-perf: change buffer size based on optype Tejasree Kondoj
2023-04-07 10:19 ` Power, Ciara
2023-04-10  3:10 ` Anoob Joseph
2023-05-24  9:29   ` Akhil Goyal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).