test/crypto: increase mbuf data payload size to 4K

Message ID 20210720132811.19425-1-suanmingm@nvidia.com (mailing list archive)
State Accepted, archived
Delegated to: akhil goyal
Headers
Series test/crypto: increase mbuf data payload size to 4K |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/github-robot success github build: passed
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS

Commit Message

Suanming Mou July 20, 2021, 1:28 p.m. UTC
  Currently, there is vendor which can support bigger crypto data size.
Increase the default mbuf data payload size to 4K as needed.

Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
---
 app/test/test_cryptodev.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Akhil Goyal July 21, 2021, 6:22 a.m. UTC | #1
> Currently, there is vendor which can support bigger crypto data size.
> Increase the default mbuf data payload size to 4K as needed.
> 
> Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
> ---
>  app/test/test_cryptodev.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Adding more people.

> diff --git a/app/test/test_cryptodev.h b/app/test/test_cryptodev.h
> index 31e278261a..1cdd84d01f 100644
> --- a/app/test/test_cryptodev.h
> +++ b/app/test/test_cryptodev.h
> @@ -19,7 +19,7 @@
>  #define DEFAULT_NUM_XFORMS              (2)
>  #define NUM_MBUFS                       (8191)
>  #define MBUF_CACHE_SIZE                 (256)
> -#define MBUF_DATAPAYLOAD_SIZE		(2048 +
> DIGEST_BYTE_LENGTH_SHA512)
> +#define MBUF_DATAPAYLOAD_SIZE		(4096 +
> DIGEST_BYTE_LENGTH_SHA512)
>  #define MBUF_SIZE			(sizeof(struct rte_mbuf) + \
>  		RTE_PKTMBUF_HEADROOM + MBUF_DATAPAYLOAD_SIZE)
> 
> --
> 2.25.1
  
Akhil Goyal July 27, 2021, 6:26 p.m. UTC | #2
> 
> > Currently, there is vendor which can support bigger crypto data size.
> > Increase the default mbuf data payload size to 4K as needed.
> >
> > Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
> > ---
> >  app/test/test_cryptodev.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> 
> Adding more people.
> 
No response from anyone.

Acked-by: Akhil Goyal <gakhil@marvell.com>

Applied to dpdk-next-crypto
  

Patch

diff --git a/app/test/test_cryptodev.h b/app/test/test_cryptodev.h
index 31e278261a..1cdd84d01f 100644
--- a/app/test/test_cryptodev.h
+++ b/app/test/test_cryptodev.h
@@ -19,7 +19,7 @@ 
 #define DEFAULT_NUM_XFORMS              (2)
 #define NUM_MBUFS                       (8191)
 #define MBUF_CACHE_SIZE                 (256)
-#define MBUF_DATAPAYLOAD_SIZE		(2048 + DIGEST_BYTE_LENGTH_SHA512)
+#define MBUF_DATAPAYLOAD_SIZE		(4096 + DIGEST_BYTE_LENGTH_SHA512)
 #define MBUF_SIZE			(sizeof(struct rte_mbuf) + \
 		RTE_PKTMBUF_HEADROOM + MBUF_DATAPAYLOAD_SIZE)